greybus: power_supply: fix name setting location
authorRui Miguel Silva <rui.silva@linaro.org>
Fri, 26 Aug 2016 10:52:05 +0000 (11:52 +0100)
committerGreg Kroah-Hartman <gregkh@google.com>
Fri, 26 Aug 2016 11:16:11 +0000 (13:16 +0200)
commit6ac9166d4e91a6dea03b64b64f128e0f927aa7ec
treeabad7e484d3962ab440cffb570eebc88de27ddce
parentcc43368a3cde151739ad20cbf71139530bd53f1a
greybus: power_supply: fix name setting location

We were checking for existing power supply names in the wrong place,
i.e, we were checking before any of the module power supply were
registered, because of that of course no name collision was detected.

Move the check to the register loop and with that we guarantee that this
mechanism works for greybus power supply naming.

Tested: using gbsim and using power supply with the same name and check
that: 1. no problems creating sysfs entries; 2. naming is done following
the desired rules.

Signed-off-by: Rui Miguel Silva <rui.silva@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/power_supply.c