RE: [PATCH] platform:x86: add Intel Broxton PMC IPC driver

["Zha, Qipeng" <qipeng.zha@intel.com>]

Hi Alan

Thanks.
The register  access ipc command is only used to r/w registers on PMIC which is controlled by PMC, Based on current pmic core driver design and Fei's suggestion, we implement regmap interface to Access such registers based on PMC ipc command.
Then pmic device driver(or other driver need to access pmic registers) can use generic regmap_read/_write Directly.




Best wishes
Qipeng

-----Original Message-----
From: One Thousand Gnomes [mailto:gnomes@lxorguk.ukuu.org.uk]
Sent: Wednesday, April 22, 2015 6:37 PM
To: Zha, Qipeng
Cc: "platform-driver-x86@vger.kernel.org\" <platform-driver-x86@vger.kernel.org>, " dvhart@infradead.org " <dvhart@infradead.org>, " Yang; Fei " <fei.yang@intel.com>, " Zhong@lxorguk.ukuu.org.uk; Huiquan " <huiquan.zhong@intel.com>, " Chen@lxorguk.ukuu.org.uk; Jason.CJ " <jason.cj.chen@intel.com>, " Zheng@lxorguk.ukuu.org.uk; Qi " <qi.zheng@intel.com>"@lxorguk.ukuu.org.uk
Subject: Re: [PATCH] platform:x86: add Intel Broxton PMC IPC driver

On Wed, 22 Apr 2015 10:32:44 +0000
"Zha, Qipeng" <qipeng.zha@intel.com> wrote:

> Hi Alan
> 
> a) Register access is one of ipc commands, and we do provide more generic APIs based on regmap subsystem in other patch,
>     with that, there will be not necessary to check platform(Broxton) for register access.

Ok that is good... but can I ask why you have two layers of IPC commands and then regmap - not just a single file that solely provides the regmap interface ?

> b) we separate scu and pmc ipc in two drivers, is because they are many difference in hw itself,  it composite several sub devices
> in hardware layer, and supported ipc commands are different,    I expected user driver will not check platform(Broxston...) to do
> ipc with scu or pmc when no common commands.

Ok

Alan