On Mon 2014-12-08 17:55:20, Jacek Anaszewski wrote:
On 12/06/2014 01:43 PM, Pavel Machek wrote:
The format of a sysfs attribute should be concise.
The error codes are generic and map directly to the V4L2 Flash
Actually I'd like to see those flash fault code defined in LED
subsystem. And V4L2 will just include LED flash header file to use it.
Because flash fault code is not for V4L2 specific but it's a feature
of LED flash devices.
For clearing error code of flash devices, I think it depends on the
hardware. If most of our LED flash is using reading to clear error
code, we probably can make it simple as this now. But what if some
other LED flash devices are using writing to clear error code? we
should provide a API to that?
Actually, we should provide API that makes sense, and that is easy to
use by userspace.
I believe "read" is called read because it does not change anything,
and it should stay that way in /sysfs. You may want to talk to sysfs
maintainers if you plan on doing another semantics.
How would you proceed in case of devices which clear their fault
register upon I2C readout (e.g. AS3645)? In this case read does have
a side effect. For such devices attribute semantics would have to be
different than for the devices which don't clear faults on readout.
No, semantics should be same for all devices.
If device clears fault register during I2C readout, kernel will simply
gather faults in an variable, and clear them upon write to sysfs file.