| Message ID | 20211210091834.28958-1-yann.gautier@foss.st.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | mmc: mmci: add st,stm32-sdmmc2 compatible | expand |
On Fri, Dec 10, 2021 at 10:19 AM Yann Gautier <yann.gautier@foss.st.com> wrote: > Although this compatible is not used in kernel, as we use the common > MMCI driver, it is used by bootloaders. The U-Boot driver was merged > before the kernel driver and uses this compatible. > To avoid issues when aligning device tree files between kernel and > boot loader, the ST dedicated compatible is added to bindings file. > > Signed-off-by: Yann Gautier <yann.gautier@foss.st.com> Why not Acked-by: Linus Walleij <linus.walleij@linaro.org> > + - description: Entry for STMicroelectronics variant of PL18x. > + This dedicated compatible is used by bootloaders. > + items: > + - const: st,stm32-sdmmc2 > + - const: arm,pl18x > + - const: arm,primecell You *could* mark it deprecated if it's not supposed to be used. But no strong opinion. Yours, Linus Walleij
On Fri, 10 Dec 2021 10:18:34 +0100, Yann Gautier wrote: > Although this compatible is not used in kernel, as we use the common > MMCI driver, it is used by bootloaders. The U-Boot driver was merged > before the kernel driver and uses this compatible. > To avoid issues when aligning device tree files between kernel and > boot loader, the ST dedicated compatible is added to bindings file. > > Signed-off-by: Yann Gautier <yann.gautier@foss.st.com> > --- > Documentation/devicetree/bindings/mmc/arm,pl18x.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > Acked-by: Rob Herring <robh@kernel.org>
On Fri, 10 Dec 2021 at 10:19, Yann Gautier <yann.gautier@foss.st.com> wrote: > > Although this compatible is not used in kernel, as we use the common > MMCI driver, it is used by bootloaders. The U-Boot driver was merged > before the kernel driver and uses this compatible. > To avoid issues when aligning device tree files between kernel and > boot loader, the ST dedicated compatible is added to bindings file. > > Signed-off-by: Yann Gautier <yann.gautier@foss.st.com> Applied for next, thanks! Kind regards Uffe > --- > Documentation/devicetree/bindings/mmc/arm,pl18x.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > index 47595cb483be..eed54bee7665 100644 > --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml > @@ -53,6 +53,12 @@ properties: > items: > - const: arm,pl18x > - const: arm,primecell > + - description: Entry for STMicroelectronics variant of PL18x. > + This dedicated compatible is used by bootloaders. > + items: > + - const: st,stm32-sdmmc2 > + - const: arm,pl18x > + - const: arm,primecell > > clocks: > description: One or two clocks, the "apb_pclk" and the "MCLK" > -- > 2.17.1 >
diff --git a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml index 47595cb483be..eed54bee7665 100644 --- a/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml +++ b/Documentation/devicetree/bindings/mmc/arm,pl18x.yaml @@ -53,6 +53,12 @@ properties: items: - const: arm,pl18x - const: arm,primecell + - description: Entry for STMicroelectronics variant of PL18x. + This dedicated compatible is used by bootloaders. + items: + - const: st,stm32-sdmmc2 + - const: arm,pl18x + - const: arm,primecell clocks: description: One or two clocks, the "apb_pclk" and the "MCLK"
Although this compatible is not used in kernel, as we use the common MMCI driver, it is used by bootloaders. The U-Boot driver was merged before the kernel driver and uses this compatible. To avoid issues when aligning device tree files between kernel and boot loader, the ST dedicated compatible is added to bindings file. Signed-off-by: Yann Gautier <yann.gautier@foss.st.com> --- Documentation/devicetree/bindings/mmc/arm,pl18x.yaml | 6 ++++++ 1 file changed, 6 insertions(+)