Re: [PATCH] gpio: pca953x: Fix NBANK calculation for PCA9536

From: Linus Walleij
Date: Mon Jun 13 2016 - 03:22:34 EST


On Thu, Jun 9, 2016 at 7:32 AM, Vignesh R <vigneshr@xxxxxx> wrote:

> NBANK() macro assumes that ngpios is a multiple of 8(BANK_SZ) and
> hence results in 0 banks for PCA9536 which has just 4 gpios. This is
> wrong as PCA9356 has 1 bank with 4 gpios. This results in uninitialized
> PCA953X_INVERT register. Fix this by using DIV_ROUND_UP macro in
> NBANK().
>
> Cc: stable@xxxxxxxxxxxxxxx
> Signed-off-by: Vignesh R <vigneshr@xxxxxx>

Patch applied.

Is this really a regression or affecting systems in development?

Which machine/device tree makes this problematic?

Yours,
Linus Walleij