Re: [PATCH v6 2/5] pinctrl: qcom: Use qcom_scm_io_update_field()

From: andy . shevchenko
Date: Fri May 26 2023 - 18:24:10 EST


Wed, Mar 29, 2023 at 01:16:49PM +0530, Mukesh Ojha kirjoitti:
> Use qcom_scm_io_update_field() exported function introduced
> in last commit.

...

> u32 val;
> + u32 mask;

Swapping their order may help with readability.

...

> + mask = (7 << g->intr_target_bit);

Why not GENMASK(2, 0) at the same time?

--
With Best Regards,
Andy Shevchenko