* regulator: mpq7920: Some inconsistencies in current driver
@ 2020-01-14 12:49 Axel Lin
2020-01-14 17:44 ` saravanan sekar
0 siblings, 1 reply; 2+ messages in thread
From: Axel Lin @ 2020-01-14 12:49 UTC (permalink / raw)
To: Saravanan Sekar; +Cc: Mark Brown, Liam Girdwood, LKML
Hi Saravanan,
There are a few inconsistencies in current driver:
I don't have the datasheet, so I'm not 100% sure.
Maybe you can help check it.
1. It's unlikely MPQ7920_LDO1_REG_B and MPQ7920_REG_CTL0 have the same address.
I think this needs double check.
2. The MPQ7920_DISCHARGE_ON seems wrong because it does not match
MPQ7920_MASK_DISCHARGE.
I guess MPQ7920_DISCHARGE_ON should be BIT(5).
3. The MPQ7920_MASK_BUCK_ILIM seems wrong. I guess it should be 0xC0.
4. Not sure why define both MPQ7920_REG_REGULATOR_EN1 and
MPQ7920_REG_REGULATOR_EN.
Regards,
Axel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: regulator: mpq7920: Some inconsistencies in current driver
2020-01-14 12:49 regulator: mpq7920: Some inconsistencies in current driver Axel Lin
@ 2020-01-14 17:44 ` saravanan sekar
0 siblings, 0 replies; 2+ messages in thread
From: saravanan sekar @ 2020-01-14 17:44 UTC (permalink / raw)
To: Axel Lin; +Cc: Mark Brown, Liam Girdwood, LKML
On 14/01/20 1:49 pm, Axel Lin wrote:
> Hi Saravanan,
>
> There are a few inconsistencies in current driver:
> I don't have the datasheet, so I'm not 100% sure.
> Maybe you can help check it.
>
> 1. It's unlikely MPQ7920_LDO1_REG_B and MPQ7920_REG_CTL0 have the same address.
> I think this needs double check.
There is no REG_B register for MPQ7920_LDO1, it is a tweak for MACRO expansion consistence of MPQ7920LDO.
However LDO1 doesn't not access this register since mpq7920_ldortc_ops discharge api is set to NULL
> 2. The MPQ7920_DISCHARGE_ON seems wrong because it does not match
> MPQ7920_MASK_DISCHARGE.
> I guess MPQ7920_DISCHARGE_ON should be BIT(5).
Yes, you are correct and thanks for pointing out
>
> 3. The MPQ7920_MASK_BUCK_ILIM seems wrong. I guess it should be 0xC0.
Yes, you are correct and thanks for pointing out
> 4. Not sure why define both MPQ7920_REG_REGULATOR_EN1 and
> MPQ7920_REG_REGULATOR_EN.
There is EN1 register exist but later decided that cannot be handled by
driver so its stray which
shall be removed.
>
> Regards,
> Axel
Thanks,
Saravanan
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-01-14 17:45 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-01-14 12:49 regulator: mpq7920: Some inconsistencies in current driver Axel Lin
2020-01-14 17:44 ` saravanan sekar
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox