Re: [PATCH] drm/radeon: Remove __counted_by from ClockInfoArray.clockInfo[]

[Lazar, Lijo]

On 17-Dec-25 7:47 PM, Lazar, Lijo wrote:

On 17-Dec-25 7:31 PM, Alex Deucher wrote:

clockInfo[] is a generic uchar pointer to variable sized structures
which vary from ASIC to ASIC.

Closes: https://gitlab.freedesktop.org/drm/amd/-/issues/4374
Signed-off-by: Alex Deucher <alexander.deuc...@amd.com>

Reviewed-by: Lijo Lazar <lijo.la...@amd.com>

There is one more -

https://github.com/torvalds/linux/blob/master/drivers/gpu/drm/radeon/ pptable.h#L463

Please ignore. False alarm, has the right datatype.

Thanks,
Lijo

Thanks,
Lijo

---
  drivers/gpu/drm/radeon/pptable.h | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/radeon/pptable.h b/drivers/gpu/drm/ radeon/pptable.h

index 969a8fb0ee9e0..f4e71046dc91c 100644
--- a/drivers/gpu/drm/radeon/pptable.h
+++ b/drivers/gpu/drm/radeon/pptable.h
@@ -450,7 +450,7 @@ typedef struct _ClockInfoArray{
      //sizeof(ATOM_PPLIB_CLOCK_INFO)
      UCHAR ucEntrySize;
-    UCHAR clockInfo[] __counted_by(ucNumEntries);
+    UCHAR clockInfo[] /*__counted_by(ucNumEntries)*/;
  }ClockInfoArray;
  typedef struct _NonClockInfoArray{