Re: [PATCH v2 0/4] usb: typec: tipd: add patch update support for tps6598x

From: Javier Carrasco
Date: Thu Jan 04 2024 - 15:15:45 EST


On 04.01.24 19:52, Dhruva Gole wrote:

>
> This series also broke boot on TI SK-AM62x [0].
>

>>
>> This looks like a correct fix to me.
>> Could you please send a proper PATCH with Fixes tag? Thanks!
>
> Thanks for reviewing this Roger, the same patch above worked for me to
> fix SK-AM62x as well [1].
>
> [0] https://storage.kernelci.org/next/master/next-20240103/arm64/defconfig/gcc-10/lab-ti/baseline-nfs-am62xx_sk-fs.txt
> [1] https://gist.github.com/DhruvaG2000/326b5d7fab4be95f20cd0aac4125f577
>
Hi Dhruva,

I am glad that you guys found a fix that quickly.

it seems that you guys work for the device manufacturer (because of your
email addresses), so I was wondering if you could explain (or provide
the documentation) why the tps6598x should not receive the GAID command
and a reset crashes the system. Everything looks exactly the same as for
the tps25750, but in that case there are no complaints from sending a
cold reset.

Thanks again and best regards,
Javier Carrasco