| From | Linus Walleij <> | Date | Wed, 21 Feb 2024 14:45:08 +0100 | Subject | Re: [PATCH 18/23] gpio: nomadik: support mobileye,eyeq5-gpio |
| |
Hi Theo,
thanks for your patch!
On Wed, Feb 14, 2024 at 5:24 PM Théo Lebrun <theo.lebrun@bootlin.com> wrote:
> We create a custom compatible for the STA2X11 IP block as integrated > into the Mobileye EyeQ5 platform. Its wake and alternate functions have > been disabled, we want to avoid touching those registers. > > We both do: (1) early return in functions that do not support the > platform, but with warnings, and (2) avoid calling those functions in > the first place. > > We ensure that pinctrl-nomadik is not used with this STA2X11 variant. > > Signed-off-by: Théo Lebrun <theo.lebrun@bootlin.com> (...) >+ bool quirk_mbly;
Compulsive abbreviation? I would just rename it:
bool is_mobileye_soc;
Nevermind the long name, it makes it crystal clear for readers what is going on. (Rusty Russell's API naming guidelines.)
With that changed: Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours, Linus Walleij
|