On Wed, Dec 17, 2025 at 9:26 AM Lazar, Lijo <[email protected]> wrote:
>
>
>
> 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 <[email protected]>
> >
> > Reviewed-by: Lijo Lazar <[email protected]>
> >
> > 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.
Right. Thanks!
>
> 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{
> >
>
