Hi Jean-Jacques,
On 11/04/2018 18:17, Jean-Jacques Hiblot wrote:
>
>
> On 11/04/2018 17:08, Neil Armstrong wrote:
>> This is a port of the dwc3-of-simple driver from Linux to enable/deassert
>> clock and resets of a simple DWC3 Controller HW glue.
>>
>> Signed-off-by: Neil Armstrong <narmstr...@baylibre.com>
>> ---
>> drivers/usb/host/Kconfig | 7 +++
>> drivers/usb/host/Makefile | 1 +
>> drivers/usb/host/dwc3-of-simple.c | 109
>> ++++++++++++++++++++++++++++++++++++++
>> 3 files changed, 117 insertions(+)
>> create mode 100644 drivers/usb/host/dwc3-of-simple.c
>>
>> diff --git a/drivers/usb/host/Kconfig b/drivers/usb/host/Kconfig
>> index a7249b7..6caa615 100644
>> --- a/drivers/usb/host/Kconfig
>> +++ b/drivers/usb/host/Kconfig
>> @@ -21,6 +21,13 @@ config USB_XHCI_DWC3
[...]
>> +
>> +static int dwc3_of_simple_clk_init(struct udevice *dev,
>> + struct dwc3_of_simple *simple)
>> +{
>> + int ret;
>> +
>> + ret = clk_get_bulk(dev, &simple->clks);
>> + if (ret == -ENOTSUPP)
>
> Must be ENOSYS instead of ENOTSUPP, otherwise probe fails on platform not
> using the clk framework
You are right, I naively used the same between reset and clock...
>
> tested-by: Jean-Jacques hiblot <jjhib...@ti.com>
>
>> + return 0;
>> + if (ret)
>> + return ret;
>> +
>> +#if CONFIG_IS_ENABLED(CLK)
>> + ret = clk_enable_bulk(&simple->clks);
>> + if (ret) {
>> + clk_release_bulk(&simple->clks);
>> + return ret;
>> + }
>> +#endif
>> +
>> + return 0;
>> +}
[...]
Thanks,
Neil
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
