Thread (13 messages) 13 messages, 4 authors, 2016-02-01

[rtc-linux] Re: [PATCH V5 5/5] regulator: max77620: add regulator driver for max77620/max20024

From: Krzysztof Kozlowski <hidden>
Date: 2016-01-28 06:42:48
Also in: linux-devicetree, linux-gpio, lkml

On 28.01.2016 15:24, Laxman Dewangan wrote:
On Thursday 28 January 2016 09:25 AM, Krzysztof Kozlowski wrote:
quoted
On 27.01.2016 23:10, Laxman Dewangan wrote:
quoted
MAXIM Semiconductor's PMIC, MAX77620 and MAX20024 have the
multiple DCDC and LDOs. This supplies the power to different
components of the system.
Also these rails has configuration for ramp time, flexible
power sequence, slew rate etc.

Add regulator driver to access these rails via regulator APIs.

Signed-off-by: Laxman Dewangan <ldewangan@nvidia.com>
Signed-off-by: Mallikarjun Kasoju <redacted>
---
Changes from V1:
- Cleanup code based on comment received on mfd/rtc.
- Avoid duplication on error message.

Changes form V2:
- Run coccicheck and checkpatch in strict mode for the alignment.
- Refactor Regulator driver to use core API for DT parsing.
- Update based on API changes.

Changes from V3:
- Change all sys initcall to module driver.
- change the max77620_read argument to unisgned int from u8.

Changes from V4:
- Added DT binding document as
devicetree/bindings/regulator/regulator-max77620.txt
- Detail out properties.
What does it mean "Detail out properties"? What have changed>
Based on review comment from Linus W, I added more description on the
properties.
Ah, ok.
quoted
I already reviewed the patchset but because of adding bindings here, you
could not add my reviewed-by tag (which makes my effort not tracked...).
Yaah, This patch include the dt binding and so wanted your review again.
This si only reason to not add reviewed-by.

Now I am going to separate out DT binding and driver,  I will add
reviewed-by tag for you in driver file.
Yeah, but the driver nor the bindings do not change so it really does
not need my additional review.

Best regards,
Krzysztof

-- 
-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups "rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email to rtc-linux+unsubscribe@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help