Hello!
On 7/29/20 8:49 PM, Alexandre Belloni wrote:
> The STOTG04 phy is used as a drop-in replacement of the ISP1301 but some
> bits doesn't have exactly the same meaning and this can lead to issues.
> Detect the phy dynamically and avoid writing to reserved bits.
>
> Signed-off-by: Alexandre Belloni <alexandre.belloni@xxxxxxxxxxx>
> ---
>
> Hi Trevor, this is totally untested but at least it builds ;)
>
> drivers/usb/host/ohci-nxp.c | 21 +++++++++++++++------
> 1 file changed, 15 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/usb/host/ohci-nxp.c b/drivers/usb/host/ohci-nxp.c
> index 85878e8ad331..36ab1501c28f 100644
> --- a/drivers/usb/host/ohci-nxp.c
> +++ b/drivers/usb/host/ohci-nxp.c
> @@ -55,6 +55,15 @@ static struct clk *usb_host_clk;
>
> static void isp1301_configure_lpc32xx(void)
> {
> + u8 value, atx_is_stotg = 0;
Why the flag is not *bool*?
> + s32 vendor, product;
> +
> + vendor = i2c_smbus_read_word_data(isp1301_i2c_client, 0x00);
> + product = i2c_smbus_read_word_data(isp1301_i2c_client, 0x02);
> +
> + if (vendor == 0x0483 && product == 0xa0c4)
> + atx_is_stotg = 1;
> +
> /* LPC32XX only supports DAT_SE0 USB mode */
> /* This sequence is important */
>
[...]
MBR, Sergei
