[PATCH] regulator: qcom-rpm: Make it possible to specify supply

[PATCH] regulator: qcom-rpm: Make it possible to specify supply

[Bjorn Andersson]

Make it possible to specify the supply of a regulator, through the
vin-supply property in dt.

Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
---
 drivers/regulator/qcom_rpm-regulator.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/regulator/qcom_rpm-regulator.c b/drivers/regulator/qcom_rpm-regulator.c
index e3120bf..00c5cc3 100644
--- a/drivers/regulator/qcom_rpm-regulator.c
+++ b/drivers/regulator/qcom_rpm-regulator.c
@@ -679,6 +679,7 @@ static int rpm_reg_probe(struct platform_device *pdev)
 	vreg->desc.owner = THIS_MODULE;
 	vreg->desc.type = REGULATOR_VOLTAGE;
 	vreg->desc.name = pdev->dev.of_node->name;
+	vreg->desc.supply_name = "vin";
 
 	vreg->rpm = dev_get_drvdata(pdev->dev.parent);
 	if (!vreg->rpm) {
-- 
1.9.1

Re: [PATCH] regulator: qcom-rpm: Make it possible to specify supply

[Srinivas Kandagatla]

Hi Bjorn,
Thanks for sending this patch.. I almost forgot about this.

Shouldn’t we update the DT Bindings with this info?

Other than that:
Acked-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

--srini

On 30/01/15 00:22, Bjorn Andersson wrote:
> Make it possible to specify the supply of a regulator, through the
> vin-supply property in dt.
>
> Signed-off-by: Bjorn Andersson <bjorn.andersson@sonymobile.com>
> ---
>   drivers/regulator/qcom_rpm-regulator.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/drivers/regulator/qcom_rpm-regulator.c b/drivers/regulator/qcom_rpm-regulator.c
> index e3120bf..00c5cc3 100644
> --- a/drivers/regulator/qcom_rpm-regulator.c
> +++ b/drivers/regulator/qcom_rpm-regulator.c
> @@ -679,6 +679,7 @@ static int rpm_reg_probe(struct platform_device *pdev)
>   	vreg->desc.owner = THIS_MODULE;
>   	vreg->desc.type = REGULATOR_VOLTAGE;
>   	vreg->desc.name = pdev->dev.of_node->name;
> +	vreg->desc.supply_name = "vin";
>
>   	vreg->rpm = dev_get_drvdata(pdev->dev.parent);
>   	if (!vreg->rpm) {
>

Re: [PATCH] regulator: qcom-rpm: Make it possible to specify supply

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 323 bytes --]

On Fri, Jan 30, 2015 at 08:57:29AM +0000, Srinivas Kandagatla wrote:
> Hi Bjorn,
> Thanks for sending this patch.. I almost forgot about this.
> 
> Shouldn’t we update the DT Bindings with this info?
> 
> Other than that:
> Acked-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>

Please don't top post.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

Re: [PATCH] regulator: qcom-rpm: Make it possible to specify supply

[Bjorn Andersson]

On Fri 30 Jan 00:57 PST 2015, Srinivas Kandagatla wrote:

> Hi Bjorn,
> Thanks for sending this patch.. I almost forgot about this.
> 
> Shouldn’t we update the DT Bindings with this info?
> 

At this time we don't have any bindings for the subnodes merged. So I also sent
out [1] yesterday, including this property. Seems I forgot linux-arm-msm
though.

> Other than that:
> Acked-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
> 

Thanks

[1] https://lkml.org/lkml/2015/1/29/1077

Regards,
Bjorn