Re: [PATCH v2 03/14] mfd: Add Microchip ZL3073x support

[Ivan Vecera]

On 10. 04. 25 9:19 dop., Krzysztof Kozlowski wrote:
> On 09/04/2025 17:43, Andy Shevchenko wrote:
> > > +/*
> > > + * Regmap range configuration
> > > + *
> > > + * The device uses 7-bit addressing and has 16 register pages with
> > > + * range 0x00-0x7f. The register 0x7f in each page acts as page
> > > + * selector where bits 0-3 contains currently selected page.
> > > + */
> > > +static const struct regmap_range_cfg zl3073x_regmap_ranges[] = {
> > > +	{
> > > +		.range_min	= 0,
> >
> > This still has the same issue, you haven't given a chance to me to reply
> > in v1 thread. I'm not going to review this as it's not settled down yet.
> > Let's first discuss the questions you have in v1.

Sorry for that but I don't understand where the issue is... Many drivers 
uses this the same way.
E.g.
drivers/leds/leds-aw200xx.c
drivers/mfd/rsmu_i2c.c
sound/soc/codecs/tas2562.c
...and many others

All of them uses selector register that is present on all pages, wide 
range for register access <0, num_pages*window_size> and window <0, 
window_size>

Do they also do incorrectly or am I missing something?

I.
> I already started reviewing v2, so now we have simultaneous discussions
> in v1 and v2...
>
> Best regards,
> Krzysztof