On Fri, Feb 15, 2008 at 01:17:16AM -0600, Manish Ahuja wrote: > Tony Breeds wrote: > > Any reason this sysfs attribute can't be write only? The show method > > doesn't seem needed. > > yes, its used later in the code.
I see that now, thanks. From my point of view it would make reviewing these patches easier if each patch was a correct and simple as possible. In this case it would have made the review easier if the sysfs attribute was write only now and then modified to add the read side when it's actually implemented. The same goes for fixing typosi, cosmetic changes and reference counting. Looking forward to a respin of this patch series. Yours Tony linux.conf.au http://linux.conf.au/ || http://lca2008.linux.org.au/ Jan 28 - Feb 02 2008 The Australian Linux Technical Conference! _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
