Re: [PATCH v2 7/9] drm/bridge: analogix_dp: Rename and simplify is_rockchip()

[Damon Ding]

Hi Luca,

On 3/24/2026 2:12 PM, Damon Ding wrote:
> On 3/24/2026 12:14 AM, Luca Ceresoli wrote:
> > On Thu Mar 19, 2026 at 11:40 AM CET, Damon Ding wrote:
> > > Rename is_rockchip() to analogix_dp_is_rockchip() for naming consistency
> > > and readability, and simplify the code with switch.
> > >
> > > Signed-off-by: Damon Ding <damon.ding@xxxxxxxxxxxxxx>
> > > Suggested-by: Nicolas Frattaroli <nicolas.frattaroli@xxxxxxxxxxxxx>
> >
> > The patch content is OK:
> > Reviewed-by: Luca Ceresoli <luca.ceresoli@xxxxxxxxxxx>
> >
> > However this won't apply without [0], correct?
> >
> > So, worth mentioning in the cover letter that this series depends on the
> > [0] series.
> >
> > [0] https://lore.kernel.org/all/20260319071452.1961274-8- 
> > damon.ding@xxxxxxxxxxxxxx/
>
> Yes, I will add it in v3.

BTW: Could you also help review the following two patches [0][1] when 
you get a chance? Your review would be much appreciated.

[0] 
https://lore.kernel.org/all/20251110085823.1197472-1-damon.ding@xxxxxxxxxxxxxx/
[1] 
https://lore.kernel.org/all/20251111022103.1350183-1-damon.ding@xxxxxxxxxxxxxx/

Best regards,
Damon