Re: [PATCH 0/3] pinctrl: s32: driver improvements and generic struct use

From: Andy Shevchenko
Date: Tue Mar 14 2023 - 13:22:36 EST


On Tue, Mar 14, 2023 at 3:46 PM Chester Lin <clin@xxxxxxxx> wrote:
>
> Hello,
>
> This patch series contains some improvements for s32 pinctrl drivers suggested
> by upstream[1], such as
>
> - Fix error shadowings and improve return value handlings.
> - Fix print format.
> - Remove unnecessary blanks.
> - Use proper macros and helpers to simplify codes.
> - Refactor config param parsing and remove config arguments that are never used.
> - Use generic struct pingroup and struct pinfunction to describe pin data.

Overall it looks not bad, thank you for doing this.
Individual patches have been reviewed and commented accordingly.

--
With Best Regards,
Andy Shevchenko