Re: [PATCH 2/2] gpio: pca953x: avoid logically dead code

[Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>]

On Fri, Dec 09, 2022 at 11:05:16PM +0200, Andy Shevchenko wrote:
> On Fri, Dec 09, 2022 at 03:50:40PM +0800, haibo.chen@xxxxxxx wrote:
> > From: Haibo Chen <haibo.chen@xxxxxxx>

...

> > -	if (reg >= 0x30) {
> > +	if (reg >= 0x30 && reg < 0x54) {
> >  		/*
> >  		 * Reserved block between 14h and 2Fh does not align on
> >  		 * expected bank boundaries like other devices.
> 
> Can we convert this to a switch case with ranges?

Looking into code more, I think we can accept this (patch 2) as a fix for now
and clean up later on.

The problem is that it really depends on the patch 1 which we need to figure
out what the correct fix would be.

-- 
With Best Regards,
Andy Shevchenko