[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <87tt41bk9v.fsf@minerva.mail-host-address-is-not-set>
Date: Fri, 27 Jun 2025 11:34:20 +0200
From: Javier Martinez Canillas <javierm@...hat.com>
To: Maxime Ripard <mripard@...nel.org>
Cc: Neil Armstrong <neil.armstrong@...aro.org>, Jessica Zhang
<quic_jesszhan@...cinc.com>, Maarten Lankhorst
<maarten.lankhorst@...ux.intel.com>, Thomas Zimmermann
<tzimmermann@...e.de>, David Airlie <airlied@...il.com>, Simona Vetter
<simona@...ll.ch>, Luca Ceresoli <luca.ceresoli@...tlin.com>, Anusha
Srivatsa <asrivats@...hat.com>, Francesco Dolcini <francesco@...cini.it>,
dri-devel@...ts.freedesktop.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/5] drm/mipi-dsi: Add dev_is_mipi_dsi function
Maxime Ripard <mripard@...nel.org> writes:
> On Fri, Jun 27, 2025 at 11:04:16AM +0200, Javier Martinez Canillas wrote:
>> Maxime Ripard <mripard@...nel.org> writes:
>>
[...]
>>
>> Usually I prefer to have static inline functions instead of macros to have
>> type checking. I see that this header has a mix of both, so I don't have a
>> strong opinion on what to use in this case.
>>
>> Reviewed-by: Javier Martinez Canillas <javierm@...hat.com>
>
> Thanks for your review!
>
> For the record, it's also how the platform bus defines its equivalent
> macro, so that's why I went with it.
>
Got it. I think is OK.
> Maxime
--
Best regards,
Javier Martinez Canillas
Core Platforms
Red Hat
Powered by blists - more mailing lists