On Mon, Dec 04, 2023 at 12:01:51PM -0500, Hugo Villeneuve wrote: > Mark Brown <broonie@xxxxxxxxxx> wrote: > > I don't understand what you mean here - you say that the addresses both > > have addresses 0x1 and 0x81 but map to address 0x1. What does the 0x81 > > refer to? The comments in the driver seemed to indicate that there was > > a single address which mapped to multiple underlying registers... > I was referring to an example in da9063-i2c.c where they have > these two registers: > #define DA9063_REG_STATUS_A 0x01 > #define DA9063_REG_SEQ 0x81 > To access one or the other, you must select page 0 or 1 in page config > selection register at address 0x00. It makes sense to me for this case. That appears to be a bit confused in that they've mapped the window through which you view the paged registers on top of the physical register map - I suppose that will work but more through luck than design. The window is the physical address range through which the actual registers can be accessed, the range is the virtual register numbers through which users access the regmap. It'll make things clearer if they don't overlap. > But for the sc16is7xx, for example you have these two > independent registers, sharing the exact same address: > #define SC16IS7XX_IIR_REG (0x02) /* Interrupt Identification */ > #define SC16IS7XX_FCR_REG (0x02) /* FIFO control */ > I am not sure if regmap range can be used with this configuration. > Assuming regmap range would be properly setup, when we call > regmap_read(regmap, SC16IS7XX_IIR_REG, &val), how does regmap would > know that we want to access SC16IS7XX_IIR_REG and not SC16IS7XX_FCR_REG? This is the exact situation this feature is supposed to handle, your window is address 2 and then you should pick some random non-physical numbers to map the two registers to for access by users. The core will then do appropriate physical accesses transparently to manage the window. The whole point here is to assign new, virtual addresses to the two registers you're trying to access. > > Searching for struct regmap_range_cfg should show a lot of users in > > mainline. > Yes, I am trying to find a good example but I must download and read the > datasheet for each one. If you could point to an IC/driver that uses > regmap_range similar to IC sc16is7xx, it would really help. Essentially all of them should be fine.
Attachment:
signature.asc
Description: PGP signature
