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.