On Wed, Mar 30, 2016 at 05:45:27PM +0100, Ben Hutchings wrote:
> On Wed, 2016-03-30 at 17:42 +0100, Ben Dooks wrote:
> > On 30/03/16 17:27, Ben Hutchings wrote:
> > > sysfs attributes should use the same format for reads and writes,
> > > rather than pretty-printing on read.
> > >
> > > Fixes: 50a5ba876908 ("i2c: mux: demux-pinctrl: add driver")
> > > Signed-off-by: Ben Hutchings <ben.hutchings@xxxxxxxxxxxxxxx>
> >
> > should that be moved to a debugfs entry or a read-only sysfs?
>
> Probably another read-only attribute (like "scaling_governor" vs
> "scaling_available_governors" in cpufreq).
I agree. "current_master" and "available_masters" is probably the way to
go. Can you do it or shall I?
Attachment:
signature.asc
Description: PGP signature
