On Mon, Mar 4, 2019 at 1:08 PM Linus Walleij <[email protected]> wrote:
>
> On Fri, Mar 1, 2019 at 4:15 PM Marek Behun <[email protected]> wrote:
>
> > On Fri, 1 Mar 2019 15:34:26 +0100
> > Linus Walleij <[email protected]> wrote:
> >
> > > > +What:
> > > > /sys/bus/moxtet/devices/moxtet-<name>.<addr>/output_value
> > > > +Date: March 2019
> > > > +KernelVersion: 5.2
> > > > +Contact: Marek Behún <[email protected]>
> > > > +Description: (RW) Read last written value or write value to the
> > > > module on
> > > > + this Moxtet address. Format: %x
> > >
> > > What is the usecase for these?
> > >
> > > If this is for debugging it should be in debugfs rather than in sysfs.
> > >
> > > If it is here for random userspace drivers, we need some good
> > > explanation as to why they can't just be kernel drivers.
I had the same thought, and then saw you already commented.
> > Hmm, it was for debugging purposes but the ability to write there is
> > not needed anymore... Shall I make it read/only? Or completely remove
> > it?
>
> I would either remove it or move it to debugfs.
How about removing it for the initial submission? Then we
can see if it's still needed later and discuss how it should be
done when we need it.
Arnd
