On 04/11/2017 11:12 AM, Sebastian Reichel wrote:
Hi,
On Mon, Apr 03, 2017 at 02:52:57PM -0500, David Lechner wrote:
This fixes the TODO to parse strings and convert them to enum values
when writing to a power_supply class property sysfs attribute.
There is at least one driver that has a writable enum property that
previously could only be written as an integer, so a fallback to writing
enums as integers instead of strings is provided so we don't break existing
userspace programs.
Signed-off-by: David Lechner <[email protected]>
---
drivers/power/supply/power_supply_sysfs.c | 124 ++++++++++++++++++++----------
1 file changed, 85 insertions(+), 39 deletions(-)
drivers/power/supply/power_supply_sysfs.c: In function
‘power_supply_store_property’:
drivers/power/supply/power_supply_sysfs.c:138:9: error: implicit declaration of
function ‘sysfs_match_string’ [-Werror=implicit-function-declaration]
This is something I found in linux-next. I did not realize it was new.
It looks like it is part of a patch series for USB-C connectors.[1]
It greatly simplifies things, so I think we should wait for that patch
to land.
[1]: https://patchwork.kernel.org/patch/9636485/
