From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Thu, 27 Feb 2014 23:03:32 +0000 Subject: Re: [PATCH 4/6] phy-rcar-usb-gen2: add device tree support Message-Id: <530FD24A.7050206@cogentembedded.com> List-Id: References: <1390057420-13226-1-git-send-email-ben.dooks@codethink.co.uk> <1390057420-13226-5-git-send-email-ben.dooks@codethink.co.uk> <52DAD3B3.1050807@cogentembedded.com> <52DCFF31.2040603@codethink.co.uk> <530F5FC2.1000608@cogentembedded.com> <530F61C4.7070706@codethink.co.uk> In-Reply-To: <530F61C4.7070706@codethink.co.uk> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Ben Dooks Cc: linux-kernel@lists.codethink.co.uk, linux-sh@vger.kernel.org, Magnus Damm , Simon Horman , "open list:OPEN FIRMWARE AND..." Hello. On 02/27/2014 07:03 PM, Ben Dooks wrote: >>>>> Signed-off-by: Ben Dooks >>>>> Reviewed-by: Ian Molton >>>>> --- >>>>> Cc: linux-usb@vger.kernel.org (open list:USB PHY LAYER) >>>>> Cc: linux-sh@vger.kernel.org (open list:ARM/SHMOBILE ARM...) >>>>> Cc: Magnus Damm (supporter:ARM/SHMOBILE ARM...) >>>>> Cc: Simon Horman (supporter:ARM/SHMOBILE ARM...) >>>>> Cc: devicetree@vger.kernel.org (open list:OPEN FIRMWARE AND...) >>>>> --- >>>>> drivers/usb/phy/phy-rcar-gen2-usb.c | 35 >>>>> ++++++++++++++++++++++++++++++----- >>>>> 1 file changed, 30 insertions(+), 5 deletions(-) >>>>> diff --git a/drivers/usb/phy/phy-rcar-gen2-usb.c >>>>> b/drivers/usb/phy/phy-rcar-gen2-usb.c >>>>> index db3ab34..906b74b 100644 >>>>> --- a/drivers/usb/phy/phy-rcar-gen2-usb.c >>>>> +++ b/drivers/usb/phy/phy-rcar-gen2-usb.c >>>> [...] >>>>> @@ -203,16 +212,31 @@ static int rcar_gen2_usb_phy_probe(struct >>>>> platform_device *pdev) >>>> [...] >>>>> + if (of_id) { >> [...] >>>>> + int len = 0; >>>>> + >>>>> + if (of_get_property(dev->of_node, "renesas,usb0-hs", &len)) >>>>> + priv->ugctrl2 = USBHS_UGCTRL2_USB0_HS; >>>>> + else >>>>> + priv->ugctrl2 = USBHS_UGCTRL2_USB0_PCI; >>>>> + >>>>> + if (of_get_property(dev->of_node, "renesas,usb2-ss", &len)) >>>>> + priv->ugctrl2 |= USBHS_UGCTRL2_USB2_SS; >>>> Where is the bindings file you document these properties in? >>> Should have been in another patch in the series. >> I didn't see it. > I thought it went out, I will need to go check. It turned out that I'd already replied to that email that I had seen all 6 patches but not the binding patch, so most probably it didn't went out that time. I've seen the latter patch (just not posted to linux-usb), so no problem here. WBR, Sergei