* [PATCH] regulator: fixed, gpio: dt: regulator-name is required property @ 2017-03-06 21:37 Harald Geyer [not found] ` <1488836249-1193-1-git-send-email-harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> 0 siblings, 1 reply; 4+ messages in thread From: Harald Geyer @ 2017-03-06 21:37 UTC (permalink / raw) To: Liam Girdwood, Mark Brown Cc: Rob Herring, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA, Harald Geyer These two drivers fail to probe if no name is provided. For details see: https://www.spinics.net/lists/kernel/msg2457515.html Signed-off-by: Harald Geyer <harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> --- Documentation/devicetree/bindings/regulator/fixed-regulator.txt | 1 + Documentation/devicetree/bindings/regulator/gpio-regulator.txt | 2 ++ 2 files changed, 3 insertions(+) diff --git a/Documentation/devicetree/bindings/regulator/fixed-regulator.txt b/Documentation/devicetree/bindings/regulator/fixed-regulator.txt index 4fae41d..0c2a6c8 100644 --- a/Documentation/devicetree/bindings/regulator/fixed-regulator.txt +++ b/Documentation/devicetree/bindings/regulator/fixed-regulator.txt @@ -2,6 +2,7 @@ Fixed Voltage regulators Required properties: - compatible: Must be "regulator-fixed"; +- regulator-name: Defined in regulator.txt as optional, but required here. Optional properties: - gpio: gpio to use for enable control diff --git a/Documentation/devicetree/bindings/regulator/gpio-regulator.txt b/Documentation/devicetree/bindings/regulator/gpio-regulator.txt index dd1ed78..1f49615 100644 --- a/Documentation/devicetree/bindings/regulator/gpio-regulator.txt +++ b/Documentation/devicetree/bindings/regulator/gpio-regulator.txt @@ -2,6 +2,8 @@ GPIO controlled regulators Required properties: - compatible : Must be "regulator-gpio". +- regulator-name : Defined in regulator.txt as optional, but required + here. - states : Selection of available voltages and GPIO configs. if there are no states, then use a fixed regulator -- 2.1.4 -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply related [flat|nested] 4+ messages in thread
[parent not found: <1488836249-1193-1-git-send-email-harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org>]
* Re: [PATCH] regulator: fixed, gpio: dt: regulator-name is required property [not found] ` <1488836249-1193-1-git-send-email-harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> @ 2017-03-15 17:10 ` Rob Herring 2017-06-23 13:55 ` Harald Geyer 0 siblings, 1 reply; 4+ messages in thread From: Rob Herring @ 2017-03-15 17:10 UTC (permalink / raw) To: Harald Geyer Cc: Liam Girdwood, Mark Brown, Mark Rutland, devicetree-u79uwXL29TY76Z2rM5mHXA On Mon, Mar 06, 2017 at 09:37:29PM +0000, Harald Geyer wrote: > These two drivers fail to probe if no name is provided. For details see: > https://www.spinics.net/lists/kernel/msg2457515.html > > Signed-off-by: Harald Geyer <harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> > --- > Documentation/devicetree/bindings/regulator/fixed-regulator.txt | 1 + > Documentation/devicetree/bindings/regulator/gpio-regulator.txt | 2 ++ > 2 files changed, 3 insertions(+) Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] regulator: fixed, gpio: dt: regulator-name is required property 2017-03-15 17:10 ` Rob Herring @ 2017-06-23 13:55 ` Harald Geyer [not found] ` <E1dOP3F-0000NF-Pe-rvrRdOlaWMmli2aaYNgTVBW+3FKk/ZpWT2auq/jSWdo@public.gmane.org> 0 siblings, 1 reply; 4+ messages in thread From: Harald Geyer @ 2017-06-23 13:55 UTC (permalink / raw) To: Mark Brown; +Cc: Liam Girdwood, devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring Hi Mark, what happened to this patch? Since it was written to your specification and got an ack from a devicetree maintainer more then 3 months ago, I don't see what is holding it back. I can't find it in any tree. TIA, Harald Rob Herring writes: > On Mon, Mar 06, 2017 at 09:37:29PM +0000, Harald Geyer wrote: > > These two drivers fail to probe if no name is provided. For details see: > > https://www.spinics.net/lists/kernel/msg2457515.html > > > > Signed-off-by: Harald Geyer <harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> > > --- > > Documentation/devicetree/bindings/regulator/fixed-regulator.txt | 1 + > > Documentation/devicetree/bindings/regulator/gpio-regulator.txt | 2 ++ > > 2 files changed, 3 insertions(+) > > Acked-by: Rob Herring <robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org> -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html ^ permalink raw reply [flat|nested] 4+ messages in thread
[parent not found: <E1dOP3F-0000NF-Pe-rvrRdOlaWMmli2aaYNgTVBW+3FKk/ZpWT2auq/jSWdo@public.gmane.org>]
* Re: [PATCH] regulator: fixed, gpio: dt: regulator-name is required property [not found] ` <E1dOP3F-0000NF-Pe-rvrRdOlaWMmli2aaYNgTVBW+3FKk/ZpWT2auq/jSWdo@public.gmane.org> @ 2017-06-23 15:03 ` Mark Brown 0 siblings, 0 replies; 4+ messages in thread From: Mark Brown @ 2017-06-23 15:03 UTC (permalink / raw) To: Harald Geyer Cc: Liam Girdwood, devicetree-u79uwXL29TY76Z2rM5mHXA, Rob Herring [-- Attachment #1: Type: text/plain, Size: 1242 bytes --] On Fri, Jun 23, 2017 at 03:55:09PM +0200, Harald Geyer wrote: > what happened to this patch? Since it was written to your specification > and got an ack from a devicetree maintainer more then 3 months ago, > I don't see what is holding it back. I can't find it in any tree. Please don't send content free pings and please allow a reasonable time for review. People get busy, go on holiday, attend conferences and so on so unless there is some reason for urgency (like critical bug fixes) please allow at least a couple of weeks for review. If there have been review comments then people may be waiting for those to be addressed. Sending content free pings adds to the mail volume (if they are seen at all) which is often the problem and since they can't be reviewed directly if something has gone wrong you'll have to resend the patches anyway, though there are some other maintainers who like them - if in doubt look at how patches for the subsystem are normally handled. Please don't top post, reply in line with needed context. This allows readers to readily follow the flow of conversation and understand what you are talking about and also helps ensure that everything in the discussion is being addressed. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 488 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2017-06-23 15:03 UTC | newest] Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2017-03-06 21:37 [PATCH] regulator: fixed, gpio: dt: regulator-name is required property Harald Geyer [not found] ` <1488836249-1193-1-git-send-email-harald-95f8Dae0BrPYtjvyW6yDsg@public.gmane.org> 2017-03-15 17:10 ` Rob Herring 2017-06-23 13:55 ` Harald Geyer [not found] ` <E1dOP3F-0000NF-Pe-rvrRdOlaWMmli2aaYNgTVBW+3FKk/ZpWT2auq/jSWdo@public.gmane.org> 2017-06-23 15:03 ` Mark Brown
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.