[AMD Official Use Only - AMD Internal Distribution Only]

> -----Original Message-----
> From: Hung, Alex <[email protected]>
> Sent: Monday, April 6, 2026 9:15 PM
> To: SHANMUGAM, SRINIVASAN <[email protected]>;
> Pillai, Aurabindo <[email protected]>
> Cc: [email protected]; Wentland, Harry <[email protected]>;
> Lin, Wayne <[email protected]>; Li, Roman <[email protected]>; Chung,
> ChiaHsuan (Tom) <[email protected]>
> Subject: Re: [PATCH] drm/amd/display: Fix do_mccs handling (MCCS/DDC) in
> amdgpu_dm_update_freesync_caps
>
>
> On 4/4/26 20:31, Srinivasan Shanmugam wrote:
> > Fix do_mccs parameter usage in amdgpu_dm_update_freesync_caps.
> >
> > This function checks FreeSync support from the display and updates
> > driver state.  MCCS (Monitor Control Command Set) over DDC (Display
> > Data
> > Channel) is used for communication between GPU and monitor.
>
> This changes comments but the title and the commit descriptions sound like
> changing the code. Can you clarify them?

Thanks for the review.

The change only adds a missing description for the do_mccs parameter in
the function kdoc. There is no functional impact.

I’ll reword the title and commit message to avoid confusion.

Best regards,
Srini


Reply via email to