Thread (14 messages) 14 messages, 3 authors, 2014-08-01

Re: [PATCH v3 1/4] mfd: pm8xxx-spmi: add support for Qualcomm SPMI PMICs

From: Lee Jones <hidden>
Date: 2014-08-01 11:24:07
Also in: linux-arm-kernel, linux-arm-msm, lkml

On Fri, 01 Aug 2014, Stanimir Varbanov wrote:
On 07/31/2014 11:48 AM, Stanimir Varbanov wrote:
quoted
On 07/30/2014 12:54 AM, David Collins wrote:
quoted
On 07/24/2014 05:45 AM, Stanimir Varbanov wrote:
quoted
From: Josh Cartwright <redacted>

The Qualcomm SPMI PMIC chips are components used with the
Snapdragon 800 series SoC family.  This driver exists
largely as a glue mfd component, it exists to be an owner
of an SPMI regmap for children devices described in
device tree.

Signed-off-by: Josh Cartwright <redacted>
Signed-off-by: Stanimir Varbanov <redacted>
Acked-by: Lee Jones <redacted>
---
 drivers/mfd/Kconfig       |   16 +++++++++++
 drivers/mfd/Makefile      |    1 +
 drivers/mfd/pm8xxx-spmi.c |   65 +++++++++++++++++++++++++++++++++++++++++++++
Would it be possible to rename this driver: qcom-spmi-pmic.c?  The driver
will be supporting several PMICs that do not fit the pm8xxx naming scheme.
 One of which is even specified in the compatible list of this driver
(pma8084).  There is presently downstream support for the following PMICs:
PM8019, PM8110, PM8226, PM8841, PM8916, PM8941, PM8994, PMA8084, PMD9635,
PMI8962, and PMI8994 [1].  Four of these do not fit the "PM8XXX" template.
I haven't strong opinion on the file names. The qcom prefix is the one
which annoying me. If you look at /drivers/mfd the company name prefixes
are very few.

The *compatible* strings are the important thing here. So If MFD
maintainer is fine with this name I'm fine too.
Lee, are you OK with suggested names qcom-spmi-pmic and qcom-ssbi-pmic?
Sounds fine to me.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help