Re: [PATCH v6 1/2] platform: Add driver for RAVE Supervisory Processor

Pavel Machek
2017-08-31T09:15:48Z

[-- Attachment #1: Type: text/plain, Size: 1450 bytes --]

On Thu 2017-08-31 12:01:01, Nikita Yushchenko wrote:
> >> I think that trying to make this generic is purely synthetic. This
> >> information is board-specific per it's nature, it comes from what board
> >> is designed for, different boards have quite different sets of possible
> >> reset reasons. What is needed is - pass this board-specific information
> >> to board-specific user space.
> >>
> >> What's proper API for that, if not a sysfs attribute?
> >
> > Please go through the thread.
> >
> > Sysfs attribute is okay, but:
> >
> > 1) it should probably be a string
> >
> > 2) it should certainly be superset of all the reasons
> >
> > 3) it should be in generic place, say /sys/power/reset_reason
> >
> > 4) it should be documented what each state means
>
> What I'm concerned here is that a requirement appears for kernel driver
> to keep and maintain knowledge of what all that codes mean. For me,

There's no way around that. Kernel interfaces need to be
documented. If you are passing codes between kernel and application,
_those codes need to be documented_.

> So question is - is there any proper API to communicate
> application-private information from hardware through kernel to
> userspace without any in-kernel interpretation?

No.

Pavel
--
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]