[PATCH v4 1/2] musb-new: ti-musb: Handle usb dual-role feature as peripheral

[Marek Vasut]

On 8/21/23 14:16, Roger Quadros wrote:
> Hi Julien,
> 
> On 21/08/2023 11:28, Julien Panis wrote:
>> On 8/14/23 19:18, Tom Rini wrote:
>>> On Thu, Jul 06, 2023 at 12:40:22PM +0200, Julien Panis wrote:
>>>
>>>> This prevents from getting some 'No USB device found' error,
>>>> in usb_ether_init() function for instance.
>>>>
>>>> Signed-off-by: Julien Panis <jpanis at baylibre.com>
>>>> Reviewed-by: Tony Lindgren <tony at atomide.com>
>>>> Reviewed-by: Nishanth Menon <nm at ti.com>
>>>> ---
>>>>    drivers/usb/musb-new/ti-musb.c | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/drivers/usb/musb-new/ti-musb.c b/drivers/usb/musb-new/ti-musb.c
>>>> index 3be3f93dd85d..44697dc31387 100644
>>>> --- a/drivers/usb/musb-new/ti-musb.c
>>>> +++ b/drivers/usb/musb-new/ti-musb.c
>>>> @@ -302,6 +302,7 @@ static int ti_musb_wrapper_bind(struct udevice *parent)
>>>>            dr_mode = usb_get_dr_mode(node);
>>>>            switch (dr_mode) {
>>>>            case USB_DR_MODE_PERIPHERAL:
>>>> +        case USB_DR_MODE_OTG:
>>>>                /* Bind MUSB device */
>>>>                ret = device_bind_driver_to_node(parent,
>>>>                                 "ti-musb-peripheral",
>>> Julien, why don't we support OTG mode here instead?
>>>
>>
>> Roger, can this be achieved with this controller ?
> 
> The Controller supports OTG mode but I'm not sure of the u-boot driver.
> 
> Bin / Stephan might know.

Surely the controller supports both Host and Peripheral mode.