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