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

[Alex Deucher]

On Wed, Dec 17, 2025 at 9:17 AM Lazar, Lijo <lijo.la...@amd.com> 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

Thanks, I'll squash that one in too.

Alex

>
> 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{
>