Re: [PATCH] usb: cdns3: Do not access memory after free

[Marek Vasut <marek.vasut@mailbox.org>]

On 8/23/25 4:07 AM, Siddharth Vadapalli wrote:

Hi,

>>>>> I was planning to test this patch but the change being made is only
>>>>> applicable to Controller Versions:
>>>>> 	#define DEV_VER_NXP_V1		0x00024502
>>>>> 	#define DEV_VER_TI_V1		0x00024509
>>>>> and not to:
>>>>> 	#define DEV_VER_V2		0x0002450C
>>>>> 	#define DEV_VER_V3		0x0002450d
>>>>>
>>>>> Since I don't have an SoC and a Board with DEV_VER_TI_V1, I cannot test
>>>>> it. However, the change looks correct to me.
>>>>>
>>>>> Reviewed-by: Siddharth Vadapalli <s-vadapalli@ti.com>
>>>> The change does indeed look correct.
>>>>
>>>> Do you know who might still have that board and could test ? (and which
>>>> board/soc is that) ?
>>>
>>> None of the boards that I have worked with have a DEV_VER_TI_V1 version
>>> of the controller. I also tried to use the Linux device-tree to check if
>>> I could identify the SoC/board but I was unable to do so.
>> Do you know which SoC is V2 and V3 ?
> 
> I spent more time on this and found out that J721E SR 1.0 has the
> controller with DEV_VER_TI_V1 version but other revisions of J721E as
> well as all of the following SoCs have DEV_VER_V3 version of the
> controller:
> AM64, AM68, AM69, J7200, J721S2, J722S, J742S2 and J784S4.
> 
> I will try to find an SR 1.0 J721E SoC and test the patch on it and
> share the results here.
This is awesome, thank you !