Re: [PATCH] ASoC: amd: acp: Fix incorrect retrival of acp_chip_info

[Cezary Rojewski <cezary.rojewski@intel.com>]

On 2025-09-10 3:39 PM, Prasad, Prasad wrote:
> 
> On 9/10/2025 2:41 PM, Cezary Rojewski wrote:
>> On 2025-09-09 8:19 AM, Venkata Prasad Potturu wrote:
>>> Use dev_get_drvdata(dev->parent) instead of dev_get_platdata(dev)
>>> to correctly get acp_chip_info members in acp I2S driver.
>>> This resolves issues where some members were zero due to incorrect
>>> data access.
>>
>> Hi,
>>
>> 'some members were zero' meaning null-ptr-deref? If so, please reword
>> to make it more explicit.
> It's not a null-ptr-deref, members were not updated properly, will
> rephrase the commit description
>>
>> Given the history of this file, mainly Commit 6d9b64156d84 ("ASoC:
>> amd: acp: Fix NULL pointer deref in acp_i2s_set_tdm_slot") it's kind
>> of surprising that the issue is addressed in staggered fashion. Why
>> was set_tdm_slot() fixed separately?
> This fix was missed earlier at the time of this commit 6d9b64156d84.

Let's mention that in the commit message then. It's clear now that the 
earlier fix is insufficient and null-ptr-derefs may still occur.

>>
>>> Fixes: e3933683b25e ("ASoC: amd: acp: Remove redundant acp_dev_data
>>> structure")
>>>
>>> Signed-off-by: Venkata Prasad Potturu <venkataprasad.potturu@amd.com>
>>
>> No newline between the tags, please.
> Okay, will send v2 patch.