RE: [PATCH] usb: host: xhci-plat: Iterate over parent nodes for finding quirks
From: Anurag Kumar Vulisha
Date: Mon Aug 06 2018 - 07:14:36 EST
Hi Mathais,
>>
>>>>
>>>> - if (device_property_read_bool(sysdev, "usb3-lpm-capable"))
>>>> - xhci->quirks |= XHCI_LPM_SUPPORT;
>>>> + if (device_property_read_bool(tmpdev, "usb2-lpm-disable"))
>>>> + xhci->quirks |= XHCI_HW_LPM_DISABLE;
>>>>
>>>> - if (device_property_read_bool(&pdev->dev, "quirk-broken-port-ped"))
>>>> - xhci->quirks |= XHCI_BROKEN_PORT_PED;
>>>> + if (device_property_read_bool(tmpdev, "usb3-lpm-capable"))
>>>> + xhci->quirks |= XHCI_LPM_SUPPORT;
>>>>
>>>> - /* imod_interval is the interrupt moderation value in nanoseconds. */
>>>> - xhci->imod_interval = 40000;
>>>
>>> Setting the default imod_interval could be moved before the for()
>>> loop
>>>
>>
>> I thought it would be better if everything related to imod_interval is
>> in one place, so kept it as is. I can fix this in v2 if you suggest.
>
>Default imod_interval needs to be set before the for loop to avoid overwriting a value
>got from device property
>
>So a v2 is needed
>
Thanks for correcting, will fix and send v2
Thanks,
Anurag Kumar Vulisha
>-Mathias