Re: [PATCH 4/5] firmware: add SmPL report for custom fallback mechanism
From: Milo Kim
Date: Tue Dec 13 2016 - 20:50:36 EST
Hi Jacek,
On 12/13/2016 06:44 PM, Jacek Anaszewski wrote:
Could you please verify if leds-lp55xx-common.c driver
really needs a custom firmware loading fallback mechanism?
Thanks for sharing this. The lp55xx-common uses this mechanism to load
and run LED effect manually, so this could be a misuse case.
I think the right solution is providing device attributes.
At this moment, four drivers use lp55xx-common code.
- lp5521, lp5523: OK if we do not support FW loading fallback mechanism
- lp5562, lp8501: need to create additional sysfs alternatively.
However, we should be careful because I'm not sure this modification
will generate the regression (breaking the user-space) or not.
Best regards,
Milo