lkml.org 
[lkml]   [2020]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] [PATCH] Firmware security information in SYSFS
On Tue, Jul 21, 2020 at 8:26 PM Greg Kroah-Hartman
<gregkh@linuxfoundation.org> wrote:
> On Tue, Jul 21, 2020 at 01:27:27PM -0300, Daniel Gutson wrote:
> > On Tue, Jul 21, 2020 at 7:52 AM Greg Kroah-Hartman <gregkh@linuxfoundation.org> wrote:
> > > > > If so, great, it should be a "class", as that way it is independent of
> > > > > any hardware type, right? Classes show how devices talk to userspace
> > > in
> > > > > a common way (input, tty, led, block, etc.) So why is this any
> > > > > different from that?
> > > > >
> > > >
> > > > Are you suggesting to create a new class, or use an existing one?
> > >
> > > Probably a new one, unless you can find an existing one that would fit?
> > >
> >
> > IIUC, Arnd Bergmann proposed that I create a class for each device driver
> > (in this case,
> > for the intel-spi, though I think the class would be spi-nor) and add
> > attributes to it.
> > In such a case, your proposal and his are different, mutually exclusive.
>
> Classes should be driver agnositic, and you should not have a single
> class per driver, as that is pretty pointless.

Yes, that was of course my suggestion as well. The class is how the
interface is shown to user space, and each driver that can provide the
functionality would have to register an instance of the class device per
physical device.

> > For me it's easier and makes more sense to have a class for, say,
> > firmware-security (if I understood you correctly).
>
> I still think that's a horrible name, as that is not what you are
> describing here, but sure, a single class is good.

Right, neither the word "firmware" nor the word "security" gives it
any sufficiently specific meaning.

Arnd

\
 
 \ /
  Last update: 2020-07-21 22:44    [W:2.966 / U:0.764 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site