Re: [PATCH RESEND v4 3/8] phy: sunxi: Rework phy initialization

From: Kishon Vijay Abraham I
Date: Thu May 15 2014 - 05:45:22 EST




On Wednesday 14 May 2014 11:18 AM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Tuesday 13 May 2014 09:14 PM, Maxime Ripard wrote:
>> Move the phy initialization and variables declaration to the loop itself, since
>> it is where it really belongs. Also remove all the temporary variables, we can
>> use the structure members directly.
>>
>> Signed-off-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxxxxxxxxx>
>> Reviewed-by: Hans de Goede <hdegoede@xxxxxxxxxx>
>
> let me know if I have to queue this and the next patch in my PHY tree.

queued this and the next patch in my tree.

Cheers
Kishon

>
> Cheers
> Kishon
>
>> ---
>> drivers/phy/phy-sun4i-usb.c | 42 ++++++++++++++++++------------------------
>> 1 file changed, 18 insertions(+), 24 deletions(-)
>>
>> diff --git a/drivers/phy/phy-sun4i-usb.c b/drivers/phy/phy-sun4i-usb.c
>> index e6e6c4ba7145..66a87d50512b 100644
>> --- a/drivers/phy/phy-sun4i-usb.c
>> +++ b/drivers/phy/phy-sun4i-usb.c
>> @@ -224,13 +224,8 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
>> struct sun4i_usb_phy_data *data;
>> struct device *dev = &pdev->dev;
>> struct device_node *np = dev->of_node;
>> - void __iomem *pmu = NULL;
>> struct phy_provider *phy_provider;
>> - struct reset_control *reset;
>> - struct regulator *vbus;
>> struct resource *res;
>> - struct phy *phy;
>> - char name[16];
>> int i;
>>
>> data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
>> @@ -262,42 +257,41 @@ static int sun4i_usb_phy_probe(struct platform_device *pdev)
>>
>> /* Skip 0, 0 is the phy for otg which is not yet supported. */
>> for (i = 1; i < data->num_phys; i++) {
>> + struct sun4i_usb_phy *phy = data->phys + i;
>> + char name[16];
>> +
>> snprintf(name, sizeof(name), "usb%d_vbus", i);
>> - vbus = devm_regulator_get_optional(dev, name);
>> - if (IS_ERR(vbus)) {
>> - if (PTR_ERR(vbus) == -EPROBE_DEFER)
>> + phy->vbus = devm_regulator_get_optional(dev, name);
>> + if (IS_ERR(phy->vbus)) {
>> + if (PTR_ERR(phy->vbus) == -EPROBE_DEFER)
>> return -EPROBE_DEFER;
>> - vbus = NULL;
>> + phy->vbus = NULL;
>> }
>>
>> snprintf(name, sizeof(name), "usb%d_reset", i);
>> - reset = devm_reset_control_get(dev, name);
>> - if (IS_ERR(reset)) {
>> + phy->reset = devm_reset_control_get(dev, name);
>> + if (IS_ERR(phy->reset)) {
>> dev_err(dev, "failed to get reset %s\n", name);
>> - return PTR_ERR(reset);
>> + return PTR_ERR(phy->reset);
>> }
>>
>> if (i) { /* No pmu for usbc0 */
>> snprintf(name, sizeof(name), "pmu%d", i);
>> res = platform_get_resource_byname(pdev,
>> IORESOURCE_MEM, name);
>> - pmu = devm_ioremap_resource(dev, res);
>> - if (IS_ERR(pmu))
>> - return PTR_ERR(pmu);
>> + phy->pmu = devm_ioremap_resource(dev, res);
>> + if (IS_ERR(phy->pmu))
>> + return PTR_ERR(phy->pmu);
>> }
>>
>> - phy = devm_phy_create(dev, &sun4i_usb_phy_ops, NULL);
>> - if (IS_ERR(phy)) {
>> + phy->phy = devm_phy_create(dev, &sun4i_usb_phy_ops, NULL);
>> + if (IS_ERR(phy->phy)) {
>> dev_err(dev, "failed to create PHY %d\n", i);
>> - return PTR_ERR(phy);
>> + return PTR_ERR(phy->phy);
>> }
>>
>> - data->phys[i].phy = phy;
>> - data->phys[i].pmu = pmu;
>> - data->phys[i].vbus = vbus;
>> - data->phys[i].reset = reset;
>> - data->phys[i].index = i;
>> - phy_set_drvdata(phy, &data->phys[i]);
>> + phy->index = i;
>> + phy_set_drvdata(phy->phy, &data->phys[i]);
>> }
>>
>> dev_set_drvdata(dev, data);
>>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-usb" 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-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/