On 03/08/2013 05:33 PM, Felipe Balbi wrote:
> On Thu, Feb 28, 2013 at 11:57:04AM +0100, Marc Kleine-Budde wrote:
>> @@ -147,19 +146,20 @@ static int ci13xxx_imx_probe(struct platform_device 
>> *pdev)
>>              return ret;
>>      }
>>  
>> -    phy_np = of_parse_phandle(pdev->dev.of_node, "fsl,usbphy", 0);
>> -    if (phy_np) {
>> -            data->phy_np = phy_np;
>> -            phy_pdev = of_find_device_by_node(phy_np);
>> -            if (phy_pdev) {
>> -                    struct usb_phy *phy;
>> -                    phy = pdev_to_phy(phy_pdev);
>> -                    if (phy &&
>> -                        try_module_get(phy_pdev->dev.driver->owner)) {
>> -                            usb_phy_init(phy);
>> -                            data->phy = phy;
>> -                    }
>> +    phy = devm_usb_get_phy_by_phandle(&pdev->dev, "fsl,usbphy", 0);
> 
> very nice, but should be done at chipidea core.

Any suggestions for the phandle name? "chipidea,usbphy"?

Marc
-- 
Pengutronix e.K.                  | Marc Kleine-Budde           |
Industrial Linux Solutions        | Phone: +49-231-2826-924     |
Vertretung West/Dortmund          | Fax:   +49-5121-206917-5555 |
Amtsgericht Hildesheim, HRA 2686  | http://www.pengutronix.de   |

Attachment: signature.asc
Description: OpenPGP digital signature

Reply via email to