linux-kernel - Re: [PATCH v5 1/2] platform: Add driver for RAVE Supervisory Processor

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAHQ1cqFYEGAcCxh7fM+VtvoDxt6phxoqD9KYUOLX4ZXLTWWtLw@mail.gmail.com>
Date:   Mon, 28 Aug 2017 09:23:10 -0700
From:   Andrey Smirnov <andrew.smirnov@...il.com>
To:     Pavel Machek <pavel@....cz>
Cc:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        linux-kernel <linux-kernel@...r.kernel.org>,
        Chris Healy <cphealy@...il.com>,
        Lucas Stach <l.stach@...gutronix.de>,
        Nikita Yushchenko <nikita.yoush@...entembedded.com>,
        Lee Jones <lee.jones@...aro.org>
Subject: Re: [PATCH v5 1/2] platform: Add driver for RAVE Supervisory Processor

On Thu, Aug 24, 2017 at 9:04 AM, Pavel Machek <pavel@....cz> wrote:
> On Fri 2017-07-28 07:27:03, Andrey Smirnov wrote:
>> Add a driver for RAVE Supervisory Processor, an MCU implementing
>> varoius bits of housekeeping functionality (watchdoging, backlight
>> control, LED control, etc) on RAVE family of products by Zodiac
>> Inflight Innovations.
>>
>> This driver implementes core MFD/serdev device as well as
>> communication subroutines necessary for commanding the device.
>
> This introduces new /sysfs userland interfaces, right? That needs documenting,
> and some review.
>

Correct and yes, good point. I was originally asked to implement all
of the exposed attributes via sysfs, but, after more discussion, I now
have green light to move a large chunk of those into debugfs which is
what I intend to do in v6, I'll also update it to document all of the
remaining sysfs entries.

> For example... bootreason is there on Nokia N900, too. We want to use same interface...
>

I tried grepping Documentation/ABI and kernel tree in general for that
sysfs interface, but wasn't able to find it. If you don't mind, could
you point me to where it's implemented?

Thanks,
Andrey Smirnov