On 11/15/2012 09:11 PM, Laxman Dewangan wrote:
> On 11/15/2012 09:56 AM, Roger Quadros wrote:
>> The device tree node will provide the input supply name as
>> a string. Use that to populate the supply_name parameter of
>> the regulator descriptor.
>>
>> Also correct the documentation to reflect the same.
>>
>> Signed-off-by: Roger Quadros <rogerq@xxxxxx>
>> CC: Laxman Dewangan <ldewangan@xxxxxxxxxx>
>> CC: Mark Brown <broonie@xxxxxxxxxxxxxxxxxxxxxxxxxxx>
>> ---
>> regulator-boot-on;
>> gpio-open-drain;
>> - vin-supply = <&parent_reg>;
>> + vin-supply = "input-supply-name";
>
> This is not correct as per the regulator binding. It says
> - <name>-supply: phandle to the parent supply/regulator node
>
> So we need to pass the phandle rather than string.
>
If you see of_get_fixed_voltage_config() in drivers/regulator/fixed.c,
all it does (without my patch) is
if (of_find_property(np, "vin-supply", NULL))
config->input_supply = "vin";
How is this supposed to work? How does phandle supplied in vin-supply
map to config->input_supply?
This config->input_supply is a string which is used in
reg_fixed_voltage_probe() like below
if (config->input_supply) {
drvdata->desc.supply_name = kstrdup(config->input_supply,
GFP_KERNEL);
So I don't understand how supplying phandle would work. Maybe I missed
something?
> Just for curiosity, why do you want this change? What is the issue are
> you facing.
>
Because I could not understood how the original approach would work. But
if you can explain how it works maybe my patch is redundant.
> This change will creates regression on many platform.
>
Did you test your platform on 3.7-rc5?
regards,
-roger
>
> -----------------------------------------------------------------------------------
>
> This email message is for the sole use of the intended recipient(s) and
> may contain
> confidential information. Any unauthorized review, use, disclosure or
> distribution
> is prohibited. If you are not the intended recipient, please contact
> the sender by
> reply email and destroy all copies of the original message.
> -----------------------------------------------------------------------------------
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-omap" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
