pinctrl: cy8c95x0: Yet another cleanup series and a fix

[PATCH v1 0/8] pinctrl: cy8c95x0: Yet another cleanup series and a fix

Posted by Andy Shevchenko 1 month, 3 weeks ago

While playing on with Intel Galileo board again, I found that I messed up a bit
with the driver. Here is a fix followed by a set of ad-hoc cleanups. Obviously
tested on the above mentioned board.

Andy Shevchenko (8):
  pinctrl: cy8c95x0: Don't miss reading the last bank registers
  pinctrl: cy8c95x0: Use devm_mutex_init() for mutex initialization
  pinctrl: cy8c95x0: remove duplicate error message
  pinctrl: cy8c95x0: Unify messages with help of dev_err_probe()
  pinctrl: cy8c95x0: Move driver data to the local variable in ->probe()
  pinctrl: cy8c95x0: Drop unused 'name' in struct cy8c95x0_pinctrl
  pinctrl: cy8c95x0: Eliminate fragile use of I²C ID table
  pinctrl: cy8c95x0: Gather ID tables in one place

 drivers/pinctrl/pinctrl-cy8c95x0.c | 97 ++++++++++++------------------
 1 file changed, 40 insertions(+), 57 deletions(-)

-- 
2.50.1

Re: [PATCH v1 0/8] pinctrl: cy8c95x0: Yet another cleanup series and a fix

Posted by Linus Walleij 1 month, 2 weeks ago

On Mon, Feb 23, 2026 at 7:09 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:

> While playing on with Intel Galileo board again, I found that I messed up a bit
> with the driver. Here is a fix followed by a set of ad-hoc cleanups. Obviously
> tested on the above mentioned board.
>
> Andy Shevchenko (8):
>   pinctrl: cy8c95x0: Don't miss reading the last bank registers

This patch applied for fixes!

> pinctrl: cy8c95x0: Use devm_mutex_init() for mutex initialization
> pinctrl: cy8c95x0: remove duplicate error message
> pinctrl: cy8c95x0: Unify messages with help of dev_err_probe()
> pinctrl: cy8c95x0: Move driver data to the local variable in ->probe()
> pinctrl: cy8c95x0: Drop unused 'name' in struct cy8c95x0_pinctrl
> pinctrl: cy8c95x0: Eliminate fragile use of I²C ID table
> pinctrl: cy8c95x0: Gather ID tables in one place

The rest 7 for next.

Yours,
Linus Walleij