* [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip variant
@ 2014-02-19 16:32 Opensource [Anthony Olech]
2014-02-19 16:47 ` Mark Brown
0 siblings, 1 reply; 3+ messages in thread
From: Opensource [Anthony Olech] @ 2014-02-19 16:32 UTC (permalink / raw)
To: Liam Girdwood, Mark Brown
Cc: Lee Jones, linux-kernel, Samuel Ortiz, David Dajun Chen
Add support for a new BC variant of the DA9053 PMIC.
There is one difference between it and the AA, BA and BB.
Signed-off-by: Anthony Olech <anthony.olech.opensource@diasemi.com>
---
This patch is relative to linux-next repository tag next-20140219
This patch depends on patch number 1 of this patch
series being applied first or it will not compile.
drivers/regulator/da9052-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9052-regulator.c b/drivers/regulator/da9052-regulator.c
index 889c7c9..599f2ad 100644
--- a/drivers/regulator/da9052-regulator.c
+++ b/drivers/regulator/da9052-regulator.c
@@ -354,6 +354,7 @@ static inline struct da9052_regulator_info *find_regulator_info(u8 chip_id,
case DA9053_AA:
case DA9053_BA:
case DA9053_BB:
+ case DA9053_BC:
for (i = 0; i < ARRAY_SIZE(da9053_regulator_info); i++) {
info = &da9053_regulator_info[i];
if (info->reg_desc.id == id)
--
end-of-patch 3/3 for PATCH V1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip variant
2014-02-19 16:32 [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip variant Opensource [Anthony Olech]
@ 2014-02-19 16:47 ` Mark Brown
2014-02-20 9:44 ` Opensource [Anthony Olech]
0 siblings, 1 reply; 3+ messages in thread
From: Mark Brown @ 2014-02-19 16:47 UTC (permalink / raw)
To: Opensource [Anthony Olech]
Cc: Liam Girdwood, Lee Jones, linux-kernel, Samuel Ortiz, David Dajun Chen
[-- Attachment #1: Type: text/plain, Size: 253 bytes --]
On Wed, Feb 19, 2014 at 04:32:47PM +0000, Opensource [Anthony Olech] wrote:
> Add support for a new BC variant of the DA9053 PMIC.
Acked-by: Mark Brown <broonie@linaro.org>
> There is one difference between it and the AA, BA and BB.
Which is...? :)
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip variant
2014-02-19 16:47 ` Mark Brown
@ 2014-02-20 9:44 ` Opensource [Anthony Olech]
0 siblings, 0 replies; 3+ messages in thread
From: Opensource [Anthony Olech] @ 2014-02-20 9:44 UTC (permalink / raw)
To: Mark Brown, Opensource [Anthony Olech]
Cc: Liam Girdwood, Lee Jones, linux-kernel, Samuel Ortiz, David Dajun Chen
> -----Original Message-----
> From: Mark Brown [mailto:broonie@kernel.org]
> Sent: 19 February 2014 16:48
> To: Opensource [Anthony Olech]
> Cc: Liam Girdwood; Lee Jones; linux-kernel@vger.kernel.org; Samuel Ortiz;
> David Dajun Chen
> Subject: Re: [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip
> variant
> On Wed, Feb 19, 2014 at 04:32:47PM +0000, Opensource [Anthony Olech]
> wrote:
> > Add support for a new BC variant of the DA9053 PMIC.
> Acked-by: Mark Brown <broonie@linaro.org>
> > There is one difference between it and the AA, BA and BB.
> Which is...? :)
Hi Mark,
it is easier to see in the diff -C15 output, that the BC chip does not need
a 'safe read':
--- drivers/mfd/da9052-i2c.c Thu Feb 20 09:20:43 2014
***************
*** 63,92 ****
--- 63,93 ----
static int da9052_i2c_fix(struct da9052 *da9052, unsigned char reg)
{
int val;
switch (da9052->chip_id) {
case DA9052:
case DA9053_AA:
case DA9053_BA:
case DA9053_BB:
/* A dummy read to a safe register address. */
if (!i2c_safe_reg(reg))
return regmap_read(da9052->regmap,
DA9052_PARK_REGISTER,
&val);
break;
+ case DA9053_BC:
default:
/*
* For other chips parking of I2C register
* to a safe place is not required.
*/
break;
}
return 0;
}
Tony Olech
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2014-02-20 9:44 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-02-19 16:32 [PATCH V1 3/3] REGULATOR: da9052: Add new DA9053 BC chip variant Opensource [Anthony Olech]
2014-02-19 16:47 ` Mark Brown
2014-02-20 9:44 ` Opensource [Anthony Olech]
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.