Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Tue, Sep 09, 2014 at 11:29:11AM +0300, Matti Vaittinen wrote: > On Mon, Sep 08, 2014 at 07:26:57AM -0400, ext Jason Cooper wrote: > > Matti, > > > > On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: > > > --- > > > > Here. Otherwise, it'll get included in the commit message, which I > > don't think was your intention. > > Ok, thanks. > > > > > > .../devicetree/bindings/i2c/trivial-devices.txt| 1 - > > > .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ > > > drivers/rtc/rtc-ds1307.c | 66 > > > -- > > > 3 files changed, 80 insertions(+), 5 deletions(-) > > > create mode 100644 > > > Documentation/devicetree/bindings/rtc/dallas,ds1339.txt > > > > I would split this into two patches, one for the binding documentation, > > and one for the C file changes. You can then use > > ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing > > jumps out from my memory). > > Splitting sounds reasonable. I will do this and also invert the default > diode setting to be connected as was suggested by Pavel in mail responce > to v1 of patch. And Alessandro is the maintainer for rtc according to > the MAINTAINERS file. > > > > > Most likely, the rtc maintainer will take the series with the DT > > maintainers Acks. You made all the binding changes as discussed by Mark > > and I, so fwiw, > > > > Acked-by: Jason Cooper > > > > Just keep in mind that I'm not a DT maintainer, I've just moved a lot of > > patches into that area :-P When you respin the series to split it, just > > throw my Ack on the binding docs. > > Thanks, although the doc will change when I invert the default diode > setting =) Yep, I'm fine with the inversion. Feel free to keep the Ack. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Mon, Sep 08, 2014 at 07:26:57AM -0400, ext Jason Cooper wrote: > Matti, > > On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: > > --- > > Here. Otherwise, it'll get included in the commit message, which I > don't think was your intention. Ok, thanks. > > > .../devicetree/bindings/i2c/trivial-devices.txt| 1 - > > .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ > > drivers/rtc/rtc-ds1307.c | 66 > > -- > > 3 files changed, 80 insertions(+), 5 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/rtc/dallas,ds1339.txt > > I would split this into two patches, one for the binding documentation, > and one for the C file changes. You can then use > ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing > jumps out from my memory). Splitting sounds reasonable. I will do this and also invert the default diode setting to be connected as was suggested by Pavel in mail responce to v1 of patch. And Alessandro is the maintainer for rtc according to the MAINTAINERS file. > > Most likely, the rtc maintainer will take the series with the DT > maintainers Acks. You made all the binding changes as discussed by Mark > and I, so fwiw, > > Acked-by: Jason Cooper > > Just keep in mind that I'm not a DT maintainer, I've just moved a lot of > patches into that area :-P When you respin the series to split it, just > throw my Ack on the binding docs. Thanks, although the doc will change when I invert the default diode setting =) Br. Matti -- = Matti Vaittinen Senile SW Specialist FINLAND ~~ When things go utterly wrong vim users can always type :help! ~~ -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Mon, Sep 08, 2014 at 07:26:57AM -0400, ext Jason Cooper wrote: Matti, On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: --- Here. Otherwise, it'll get included in the commit message, which I don't think was your intention. Ok, thanks. .../devicetree/bindings/i2c/trivial-devices.txt| 1 - .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ drivers/rtc/rtc-ds1307.c | 66 -- 3 files changed, 80 insertions(+), 5 deletions(-) create mode 100644 Documentation/devicetree/bindings/rtc/dallas,ds1339.txt I would split this into two patches, one for the binding documentation, and one for the C file changes. You can then use ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing jumps out from my memory). Splitting sounds reasonable. I will do this and also invert the default diode setting to be connected as was suggested by Pavel in mail responce to v1 of patch. And Alessandro is the maintainer for rtc according to the MAINTAINERS file. Most likely, the rtc maintainer will take the series with the DT maintainers Acks. You made all the binding changes as discussed by Mark and I, so fwiw, Acked-by: Jason Cooper ja...@lakedaemon.net Just keep in mind that I'm not a DT maintainer, I've just moved a lot of patches into that area :-P When you respin the series to split it, just throw my Ack on the binding docs. Thanks, although the doc will change when I invert the default diode setting =) Br. Matti -- = Matti Vaittinen Senile SW Specialist FINLAND ~~ When things go utterly wrong vim users can always type :help! ~~ -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Tue, Sep 09, 2014 at 11:29:11AM +0300, Matti Vaittinen wrote: On Mon, Sep 08, 2014 at 07:26:57AM -0400, ext Jason Cooper wrote: Matti, On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: --- Here. Otherwise, it'll get included in the commit message, which I don't think was your intention. Ok, thanks. .../devicetree/bindings/i2c/trivial-devices.txt| 1 - .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ drivers/rtc/rtc-ds1307.c | 66 -- 3 files changed, 80 insertions(+), 5 deletions(-) create mode 100644 Documentation/devicetree/bindings/rtc/dallas,ds1339.txt I would split this into two patches, one for the binding documentation, and one for the C file changes. You can then use ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing jumps out from my memory). Splitting sounds reasonable. I will do this and also invert the default diode setting to be connected as was suggested by Pavel in mail responce to v1 of patch. And Alessandro is the maintainer for rtc according to the MAINTAINERS file. Most likely, the rtc maintainer will take the series with the DT maintainers Acks. You made all the binding changes as discussed by Mark and I, so fwiw, Acked-by: Jason Cooper ja...@lakedaemon.net Just keep in mind that I'm not a DT maintainer, I've just moved a lot of patches into that area :-P When you respin the series to split it, just throw my Ack on the binding docs. Thanks, although the doc will change when I invert the default diode setting =) Yep, I'm fine with the inversion. Feel free to keep the Ack. thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Mon, 8 Sep 2014 07:26:57 -0400 Jason Cooper wrote: > Most likely, the rtc maintainer will take the series with the DT > maintainers Acks. You made all the binding changes as discussed by Mark > and I, so fwiw, > > Acked-by: Jason Cooper Acked-by: Alessandro Zummo -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
Matti, On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: >From here: > Hi dt bindings maintainers (and others interested in device-tree bindings), > >I would like to get this included in your tree. Do you think there is >still something I could improve/change in order to get this accepted? >Or do you think I should address this to someone else? > > > > > > > > > To here should go below the '---' > Some DS13XX devices have "trickle chargers". Introduce a device tree binding > for specifying the trickle charger configuration for ds1339. > > Signed-off-by: Matti Vaittinen > --- Here. Otherwise, it'll get included in the commit message, which I don't think was your intention. > .../devicetree/bindings/i2c/trivial-devices.txt| 1 - > .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ > drivers/rtc/rtc-ds1307.c | 66 > -- > 3 files changed, 80 insertions(+), 5 deletions(-) > create mode 100644 Documentation/devicetree/bindings/rtc/dallas,ds1339.txt I would split this into two patches, one for the binding documentation, and one for the C file changes. You can then use ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing jumps out from my memory). Most likely, the rtc maintainer will take the series with the DT maintainers Acks. You made all the binding changes as discussed by Mark and I, so fwiw, Acked-by: Jason Cooper Just keep in mind that I'm not a DT maintainer, I've just moved a lot of patches into that area :-P When you respin the series to split it, just throw my Ack on the binding docs. thx, Jason. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
Matti, On Mon, Sep 08, 2014 at 10:32:24AM +0300, Matti Vaittinen wrote: From here: Hi dt bindings maintainers (and others interested in device-tree bindings), I would like to get this included in your tree. Do you think there is still something I could improve/change in order to get this accepted? Or do you think I should address this to someone else? To here should go below the '---' Some DS13XX devices have trickle chargers. Introduce a device tree binding for specifying the trickle charger configuration for ds1339. Signed-off-by: Matti Vaittinen matti.vaitti...@nsn.com --- Here. Otherwise, it'll get included in the commit message, which I don't think was your intention. .../devicetree/bindings/i2c/trivial-devices.txt| 1 - .../devicetree/bindings/rtc/dallas,ds1339.txt | 18 ++ drivers/rtc/rtc-ds1307.c | 66 -- 3 files changed, 80 insertions(+), 5 deletions(-) create mode 100644 Documentation/devicetree/bindings/rtc/dallas,ds1339.txt I would split this into two patches, one for the binding documentation, and one for the C file changes. You can then use ./scripts/get_maintainer.pl to see who the rtc maintainer is (nothing jumps out from my memory). Most likely, the rtc maintainer will take the series with the DT maintainers Acks. You made all the binding changes as discussed by Mark and I, so fwiw, Acked-by: Jason Cooper ja...@lakedaemon.net Just keep in mind that I'm not a DT maintainer, I've just moved a lot of patches into that area :-P When you respin the series to split it, just throw my Ack on the binding docs. thx, Jason. -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Re: [RESEND PATCH v3] rtc: ds1307: add trickle charger device tree binding
On Mon, 8 Sep 2014 07:26:57 -0400 Jason Cooper ja...@lakedaemon.net wrote: Most likely, the rtc maintainer will take the series with the DT maintainers Acks. You made all the binding changes as discussed by Mark and I, so fwiw, Acked-by: Jason Cooper ja...@lakedaemon.net Acked-by: Alessandro Zummo a.zu...@towertech.it -- Best regards, Alessandro Zummo, Tower Technologies - Torino, Italy http://www.towertech.it -- To unsubscribe from this list: send the line unsubscribe linux-kernel in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/