Messages in this thread | | | Date | Thu, 18 Jul 2013 17:01:27 -0700 | From | Greg Kroah-Hartman <> | Subject | Re: [PATCH v3 01/10] Driver core and sysfs changes for attribute groups |
| |
On Thu, Jul 18, 2013 at 04:54:17PM -0700, Guenter Roeck wrote: > On Sun, Jul 14, 2013 at 04:05:50PM -0700, Greg Kroah-Hartman wrote: > > Hi all, > > > > Here is the third iteration of the patchset to add better attribute > > group support to the driver core and sysfs. > > > > Others have now tested it, and we got 3 more patches fixing a warning > > that was showing up when binary attributes were in a group with no > > "normal" attributes, and some more helper macros were added and cleaned > > up. > > > > I've booted this successfully against 3.11-rc1 with no problems (am > > sending these from that machine.) > > > > Ian, I didn't add a DEVICE_PATTR() macro yet, that can come later. I > > considered it, but that would also mean a BUS_PATTR() and the like, which I > > guess works, but I hate even a _hint_ of "Hungarian" notation, which this > > implies. > > > > Anyone else have a better name for this macro: > > #define DEVICE_PATTR(_name) &dev_attr_##_name.attr > > that they can think of? It will cut down on the typing done for > > attribute lists. > > > > changes from v3: > > - tested by Guenter > > - patches from Oliver added > > - refreshed on 3.11-rc1 (minor fuzz) > > - Ian glanced at them. > > > > changes from v2: > > - actually boots > > - 7th patch added properly > > - added BUS_ATTR, CLASS_ATTR, and DRIVER_ATTR RW and RO macros > > to help with converting code to use attributes properly. > > > Hi Greg, > > thanks a lot for this series of patches, and for pushing it upstream. > > Do you by any chance have a repository for the cleanup patches depending on this > series ? It looks like a good area to get involved with when I am tired/annoyed > by my other work, but I would not want to duplicate your work.
Give me a week or so to get them out, I have 1500+ "normal" patches to wade through, and 150+ stable patches to deal with first.
But, if you want to look, I do keep my patches in a quilt series on git.kernel.org in my patches.git repo, feel free to grab those to play with.
thanks,
greg k-h
| |