Re: [PATCH 10/12] mfd: dt-bindings: samsung,s5m8767: document buck and LDO supplies

From: Lee Jones
Date: Thu Sep 30 2021 - 09:21:40 EST

Next message: Christophe Leroy: "Re: [PATCH v2 5/7] sched: move CPU field back into thread_info if THREAD_INFO_IN_TASK=y"
Previous message: kernel test robot: "[gustavoars:for-next/cast-function 1/1] kernel/bpf/verifier.c:12470:16: warning: cast from 'unsigned int (*)(const void *, const struct bpf_insn *)' to 'u64 (*)(u64, u64, u64, u64, u64)' (aka 'unsigned long long (*)(unsigned long long, unsigned long long, unsigned long long, unsigned long long,..."
In reply to: Krzysztof Kozlowski: "Re: [PATCH 10/12] mfd: dt-bindings: samsung,s5m8767: document buck and LDO supplies"
Next in thread: Krzysztof Kozlowski: "[PATCH 12/12] ARM: dts: exynos: use spaces instead of tabs around '='"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

On Thu, 30 Sep 2021, Krzysztof Kozlowski wrote:

> On 30/09/2021 14:47, Lee Jones wrote:
> > On Tue, 28 Sep 2021, Krzysztof Kozlowski wrote:
> >
> >> Document the properties with regulator supplies for bucks and LDOs. At
> >> least one board uses it (Exynos5250 Arndale).
> >>
> >> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>
> >> ---
> >> .../bindings/mfd/samsung,s5m8767.yaml | 38 +++++++++++++++++++
> >> 1 file changed, 38 insertions(+)
> >
> > These all need Rob's Ack.
>
> Correct.
>
> >
> > But can you start with reworking the subject line please.
> >
> > It should be:
> >
> > "dt-bindings: mfd: <component>: <Subject starting with an uppercase char>"
>
> Sure, just have in mind that Mark wants them the other way around for
> regulator. :)

a) I'm suggesting you use the standard formatting
b) This is not a regulator patch :)

--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog