From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Subject: Re: [PATCH v3 2/2] usb: renesas_usbhs: Add device tree support for
 R-Car H2 and M2
Date: Thu, 2 Oct 2014 09:05:11 +0900
Message-ID: <542C96B7.4030407@renesas.com>
References: <5406A664.2050501@renesas.com> <54298A38.5030703@cogentembedded.com> <542A7D29.3010805@renesas.com> <542B21DE.4010903@cogentembedded.com> <542C547C.1080601@cogentembedded.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="windows-1252"
Content-Transfer-Encoding: 7bit
Return-path: <linux-sh-owner@vger.kernel.org>
In-Reply-To: <542C547C.1080601@cogentembedded.com>
Sender: linux-sh-owner@vger.kernel.org
To: Sergei Shtylyov <sergei.shtylyov@cogentembedded.com>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Felipe Balbi <balbi@ti.com>
Cc: Rob Herring <robh+dt@kernel.org>, "pawel.moll@arm.com" <pawel.moll@arm.com>, "mark.rutland@arm.com" <mark.rutland@arm.com>, "ijc+devicetree@hellion.org.uk" <ijc+devicetree@hellion.org.uk>, "galak@codeaurora.org" <galak@codeaurora.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, "linux-usb@vger.kernel.org" <linux-usb@vger.kernel.org>, SH-Linux <linux-sh@vger.kernel.org>, kuninori morimoto <kuninori.morimoto.gx@renesas.com>, Ulrich Hecht <ulrich.hecht@gmail.com>
List-Id: devicetree@vger.kernel.org

Hello.

(2014/10/02 4:22), Sergei Shtylyov wrote:
> On 10/01/2014 01:34 AM, Sergei Shtylyov wrote:
> 
>>>>> This driver supports other SoCs, but they need boards/Soc depend code.
>>>>> So, this patch adds device tree support for R-Car H2 and M2 initially.
> 
>>>>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
>>>>> Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
>>>>> ---
>>>>>    drivers/usb/renesas_usbhs/common.c |   44
>>>>> ++++++++++++++++++++++++++++++++++++
>>>>>    1 file changed, 44 insertions(+)
> 
>>>>> diff --git a/drivers/usb/renesas_usbhs/common.c
>>>>> b/drivers/usb/renesas_usbhs/common.c
>>>>> index 1b9bf8d..b3b6813 100644
>>>>> --- a/drivers/usb/renesas_usbhs/common.c
>>>>> +++ b/drivers/usb/renesas_usbhs/common.c
>> [...]
>>>>> @@ -438,6 +440,43 @@ static int usbhsc_drvcllbck_notify_hotplug(struct
>>>>> platform_device *pdev)
>>>> [...]
>>>>> +static struct renesas_usbhs_platform_info *usbhs_parse_dt(struct device
>>>>> *dev)
>>>>> +{
>>>>> +    struct renesas_usbhs_platform_info *info;
>>>>> +    struct renesas_usbhs_driver_param *dparam;
>>>>> +    const struct of_device_id *of_id = of_match_device(usbhs_of_match, dev);
>>>>> +    u32 tmp;
>>>>> +    int gpio;
>>>>> +
>>>>> +    info = devm_kzalloc(dev, sizeof(*info), GFP_KERNEL);
>>>>> +    if (!info)
>>>>> +        return NULL;
>>>>> +
>>>>> +    dparam = &info->driver_param;
>>>>> +    dparam->type = of_id ? (u32)of_id->data : 0;
>>>>> +    if (!of_property_read_u32(dev->of_node, "renesas,buswait", &tmp))
>>>>> +        dparam->buswait_bwait = tmp;
>>>>> +    gpio = of_get_named_gpio_flags(dev->of_node, "renesas,enable-gpio", 0,
>>>>> +                       NULL);
>>>>> +    if (gpio > 0)
>>>>> +        dparam->enable_gpio = gpio;
>>>>> +
>>>>> +    return info;
> 
>>>>      I don't see where you parse the optional "phy" property. It should exist
>>>> for the Lager and Koelsch boards you target.
> 
> [...]
> 
>>> About using the phy driver in renesas_usbhs driver, I intend to modify
>>> the drivers/usb/renesas_usbhs/rcar.c.
> 
>>     Hm, it slready supports 'struct usb_phy'. Also PHYs are not limited to
>> R-Car Gen2 SoCs...
> 
>     OK, let's do this locally to rcar2.c for now. But since you still haven't 
> provided the patches and I'm tasked with the USBHS support, I have to take the 
> further development in my hands.

Thank you for the reply. And, I'm sorry for the deleyed response.
I will submit the patches (add support generic phy in rcar.c and device node
for some SoCs) today.

Best regards,
Yoshihiro Shimoda

>>> Best regards,
>>> Yoshihiro Shimoda
> 
> WBR, Sergei
>