Re: [PATCH v2 1/4] max8903: adds documentation for device tree bindings.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On 16/06/2016 4:35 PM, Krzysztof Kozlowski wrote:
On 06/14/2016 03:54 AM, Chris Lapa wrote:
On 10/06/2016 11:51 PM, Krzysztof Kozlowski wrote:
On 06/10/2016 02:32 PM, Chris Lapa wrote:
From: Chris Lapa <chris@xxxxxxxxxxx>

Signed-off-by: Chris Lapa <chris@xxxxxxxxxxx>
---
 .../devicetree/bindings/power/max8903-charger.txt  | 30
++++++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644
Documentation/devicetree/bindings/power/max8903-charger.txt

You again skipped all of the maintainers. Your patch won't be applied.
You need to send the patch to the appropriate person so he/she could
apply it.

From this patch:
Rob Herring <robh+dt@xxxxxxxxxx> (maintainer:OPEN FIRMWARE AND FLATTENED
DEVICE TREE BINDINGS,commit_signer:9/19=47%)
Mark Rutland <mark.rutland@xxxxxxx> (maintainer:OPEN FIRMWARE AND
FLATTENED DEVICE TREE BINDINGS)

From power suppyy tree:
Sebastian Reichel <sre@xxxxxxxxxx> (maintainer:POWER SUPPLY
CLASS/SUBSYSTEM and DRIVERS)
Dmitry Eremin-Solenikov <dbaryshkov@xxxxxxxxx> (maintainer:POWER SUPPLY
CLASS/SUBSYSTEM and DRIVERS)
David Woodhouse <dwmw2@xxxxxxxxxxxxx> (maintainer:POWER SUPPLY
CLASS/SUBSYSTEM and DRIVERS)


diff --git
a/Documentation/devicetree/bindings/power/max8903-charger.txt
b/Documentation/devicetree/bindings/power/max8903-charger.txt
new file mode 100644
index 0000000..e0b5366
--- /dev/null
+++ b/Documentation/devicetree/bindings/power/max8903-charger.txt
@@ -0,0 +1,30 @@
+Maxim Semiconductor MAX8903 Battery Charger bindings
+
+Required properties:
+- compatible: "maxim,max8903-charger" for MAX8903 Battery Charger
+- dc_valid: Specifies that the DC portion of the MAX8903 has been
connected up
+        and that dok-gpios should be specified

You don't need the dc_valid nor the usb_valid.  If the dok-gpios are
present - use DC power. If the uok-gpios - USB power.

I did think that as well, but I didn't want to break backwards
compatibility.

What do you mean by backwards compatibility? The driver did not support
DeviceTree before so it is not about DT compatibility. If you think
about platform data then it is different: how you match DT into driver's
structure is up to you.
Ah okay, I misunderstood you here. I thought you meant remove usb_valid and dc_valid from the platform data. However determining usb and dc validity from the gpios set in the DT does make more sense. I will do that!



If both or none: print error.
Its valid to have both connected.

Ah, okay. Just an example.



BTW, It would be nice if send also the user of this bindings - DTS/DTSI.
I'm not sure I understand what you mean here?
The DTSI file used is almost identical to the copy I included in the bindings document. Other then some extra pinmux stuff.

Thanks,
Chris

I mean I would like to see the DTS file which is using this device and
these bindings. It is not necessary but it would be nice.

Best regards,
Krzysztof



Best regards,
Krzysztof

+    - dok-gpios: Valid DC power has been detected
+- usb_valid: Specifies that the USB portion of the MAX8903 has been
connected up
+        and that uok-gpios should be specified
+    - uok-gpios: Valid USB power has been detected
+
+Optional properties:
+- cen-gpios: Charge enable pin
+- chg-gpios: Charger status pin
+- flt-gpios: Fault pin
+- dcm-gpios: Current limit mode setting (DC or USB)
+- usus-gpios: USB suspend pin
+
+
+Example:
+
+    max8903-charger {
+        compatible = "maxim,max8903-charger";
+        dok-gpios = <&gpio2 3 GPIO_ACTIVE_LOW>;
+        flt-gpios = <&gpio2 2 GPIO_ACTIVE_LOW>;
+        chg-gpios = <&gpio3 15 GPIO_ACTIVE_LOW>;
+        cen-gpios = <&gpio2 5 GPIO_ACTIVE_LOW>;
+        dc_valid;
+        status = "okay";
+    };






--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux