linux-kernel - RE: [PATCH v3 2/3] firmware: zynqmp: Add sysfs entry for runtime features

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <BYAPR02MB44884D1746D719EB47A96BD2A46A9@BYAPR02MB4488.namprd02.prod.outlook.com>
Date:   Fri, 3 Dec 2021 10:17:42 +0000
From:   Ronak Jain <ronakj@...inx.com>
To:     Greg KH <gregkh@...uxfoundation.org>
CC:     Michal Simek <michals@...inx.com>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Rajan Vaja <RAJANV@...inx.com>,
        "corbet@....net" <corbet@....net>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "arnd@...db.de" <arnd@...db.de>,
        Sai Krishna Potthuri <lakshmis@...inx.com>
Subject: RE: [PATCH v3 2/3] firmware: zynqmp: Add sysfs entry for runtime
 features

Hi Greg,

Thanks for reviewing the patch.

> -----Original Message-----
> From: Greg KH <gregkh@...uxfoundation.org>
> Sent: Friday, December 3, 2021 2:55 PM
> To: Ronak Jain <ronakj@...inx.com>
> Cc: Michal Simek <michals@...inx.com>; linux-kernel@...r.kernel.org; Rajan
> Vaja <RAJANV@...inx.com>; corbet@....net; linux-arm-
> kernel@...ts.infradead.org; arnd@...db.de; Sai Krishna Potthuri
> <lakshmis@...inx.com>
> Subject: Re: [PATCH v3 2/3] firmware: zynqmp: Add sysfs entry for runtime
> features
> 
> On Fri, Dec 03, 2021 at 01:18:13AM -0800, Ronak Jain wrote:
> > Create sysfs entry for runtime feature configuration. The support  is
> > added for an over temperature and external watchdog feature.
> >
> > The below listed files are used for runtime features configuration:
> > /sys/devices/platform/firmware\:zynqmp-firmware/feature_config_id
> > /sys/devices/platform/firmware\:zynqmp-firmware/feature_config_value
> >
> > In order to configure an over temperature or external watchdog
> > features, first the user need to select the valid config id and then
> > the user can configure the value for selected feature config id.
> >
> > Signed-off-by: Ronak Jain <ronak.jain@...inx.com>
> 
> Ah, here's the documentation.
> 
> You should say so in the Subject line, and the changelog text is not correct, this
> is a documentation update.
I will fix the suggested changes.

> 
> 
> > ---
> > Changes in v3:
> > - None
> >
> > Changes in v2:
> > - Update commit message
> > ---
> >  .../ABI/stable/sysfs-driver-firmware-zynqmp   | 84 +++++++++++++++++++
> >  1 file changed, 84 insertions(+)
> >
> > diff --git a/Documentation/ABI/stable/sysfs-driver-firmware-zynqmp
> > b/Documentation/ABI/stable/sysfs-driver-firmware-zynqmp
> > index f5724bb5b462..2fde354715a5 100644
> > --- a/Documentation/ABI/stable/sysfs-driver-firmware-zynqmp
> > +++ b/Documentation/ABI/stable/sysfs-driver-firmware-zynqmp
> > @@ -113,3 +113,87 @@ Description:
> >  		    # echo 0 >
> > /sys/devices/platform/firmware\:zynqmp-firmware/health_status
> >
> >  Users:		Xilinx
> > +
> > +What:		/sys/devices/platform/firmware\:zynqmp-
> firmware/feature_config_*
> > +Date:		Aug 2021
> > +KernelVersion:	5.14
> 
> 5.14?
Will fix it.
> 
> Are these sysfs files already in the kernel tree?
No
> 
> And can you break this up into one entry per sysfs file?
> 
> One for the feature_config_id file and one for feature_config_value?
Yes, I will make a separate documentation for the sysfs files.

Thanks,
Ronak
> 
> thanks,
> 
> greg k-h