Re: [PATCH] regmap: add virtual PMIC IPC bus support

[Mark Brown]

On Tue, May 19, 2015 at 04:57:00AM +0000, Zha, Qipeng wrote:

> >Why is this being added to the regmap core rather than being implemented
> >in using the reg_write() and reg_read() callbacks in the core driver for
> >the PMIC?

> Thanks. I don't receive this message before in eamil.
> Current regmap only support register access  using several bus:  i2c, spi, ac97 ...
> But register access for intel pmic are done by hardware ipc, so we add such support in regmap core,
> To make pmic subdevice drivers can use generic regmap r/w API to access registers.

That doesn't really address my concern - we already have the reg_read()
and reg_write() interface so devices can use regmap without a bus in the
core.  The question is if this is something that's going to be used by
many devices so there's code to share or if it's something that's only
going to be used by this PMIC in which case keeping it in the driver
should be easier.

Attachment: signature.asc
Description: Digital signature