Message ID | 20240530111205.1764-1-bavishimithil@gmail.com (mailing list archive) |
---|---|
State | Superseded |
Headers | show |
Series | [v6] ASoC: dt-bindings: omap-mcpdm: Convert to DT schema | expand |
On 30/05/2024 13:12, Mighty wrote: > From: Mithil Bavishi <bavishimithil@gmail.com> > > Convert the OMAP4+ McPDM bindings to DT schema. > > Signed-off-by: Mithil Bavishi <bavishimithil@gmail.com> > + > +properties: > + compatible: > + const: ti,omap4-mcpdm > + > + reg: > + items: > + - description: MPU access base address > + - description: L3 interconnect address > + > + reg-names: > + items: > + - const: mpu > + - const: dma > + > + interrupts: > + maxItems: 1 > + > + dmas: > + maxItems: 2 > + > + dma-names: > + items: > + - const: up_link > + - const: dn_link Please mention in the commit msg all changes (and explain why!) done during conversion (comparing to original binding). I am pretty sure I gave this feedback already. Best regards, Krzysztof
> Please mention in the commit msg all changes (and explain why!) done > during conversion (comparing to original binding). I am pretty sure I > gave this feedback already. I read that i'm supposed to add the changelog below the --- line, my bad. I'll mention the changes in the commit message itself and that too from v2 right?
On 01/06/2024 08:16, Mithil wrote: >> Please mention in the commit msg all changes (and explain why!) done >> during conversion (comparing to original binding). I am pretty sure I >> gave this feedback already. > I read that i'm supposed to add the changelog below the --- line, my > bad. I'll mention the changes in the commit message itself and that > too from v2 right? I think you are now mixing different things. changelog and differences comparing to pure 1-to-1 conversion. Best regards, Krzysztof
On Sat, Jun 1, 2024 at 8:44 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > I think you are now mixing different things. changelog and differences > comparing to pure 1-to-1 conversion. Oh my apologies, the commit message being more descriptive? Should it also include the changelogs, or just a general description about mcpdm and it being used in omap4/omap5 platforms?
On 01/06/2024 08:16, Mithil wrote: >> Please mention in the commit msg all changes (and explain why!) done >> during conversion (comparing to original binding). I am pretty sure I >> gave this feedback already. > I read that i'm supposed to add the changelog below the --- line, my > bad. I'll mention the changes in the commit message itself and that > too from v2 right? > https://lore.kernel.org/all/20240528164227.1988357-1-andreas@kemnade.info/ https://lore.kernel.org/all/20240530211654.7946-1-ansuelsmth@gmail.com/ https://lore.kernel.org/all/20240522151616.27397-1-pratik.farkase@wsisweden.com/ Best regards, Krzysztof
On Sun, Jun 2, 2024 at 8:49 PM Krzysztof Kozlowski <krzk@kernel.org> wrote: > > https://lore.kernel.org/all/20240528164227.1988357-1-andreas@kemnade.info/ > > https://lore.kernel.org/all/20240530211654.7946-1-ansuelsmth@gmail.com/ > > https://lore.kernel.org/all/20240522151616.27397-1-pratik.farkase@wsisweden.com/ Understood, thanks a lot! So something along the lines of Convert the OMAP4+ McPDM bindings from txt to yaml (dtschema). Drop ti.hwmods property as it is not needed since the sysc conversion. Add dma, dma-names, reg-names properties to match the DTS so as to not break the already existing ABI. Update example to match the existing node. Also update the changelog from the start noting all changes. This much for v7 correct?
diff --git a/Documentation/devicetree/bindings/sound/omap-mcpdm.txt b/Documentation/devicetree/bindings/sound/omap-mcpdm.txt deleted file mode 100644 index ff98a0cb5..000000000 --- a/Documentation/devicetree/bindings/sound/omap-mcpdm.txt +++ /dev/null @@ -1,30 +0,0 @@ -* Texas Instruments OMAP4+ McPDM - -Required properties: -- compatible: "ti,omap4-mcpdm" -- reg: Register location and size as an array: - <MPU access base address, size>, - <L3 interconnect address, size>; -- interrupts: Interrupt number for McPDM -- ti,hwmods: Name of the hwmod associated to the McPDM -- clocks: phandle for the pdmclk provider, likely <&twl6040> -- clock-names: Must be "pdmclk" - -Example: - -mcpdm: mcpdm@40132000 { - compatible = "ti,omap4-mcpdm"; - reg = <0x40132000 0x7f>, /* MPU private access */ - <0x49032000 0x7f>; /* L3 Interconnect */ - interrupts = <0 112 0x4>; - interrupt-parent = <&gic>; - ti,hwmods = "mcpdm"; -}; - -In board DTS file the pdmclk needs to be added: - -&mcpdm { - clocks = <&twl6040>; - clock-names = "pdmclk"; - status = "okay"; -}; diff --git a/Documentation/devicetree/bindings/sound/ti,omap4-mcpdm.yaml b/Documentation/devicetree/bindings/sound/ti,omap4-mcpdm.yaml new file mode 100644 index 000000000..cdea0a008 --- /dev/null +++ b/Documentation/devicetree/bindings/sound/ti,omap4-mcpdm.yaml @@ -0,0 +1,73 @@ +# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/sound/ti,omap4-mcpdm.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: OMAP McPDM + +maintainers: + - Misael Lopez Cruz <misael.lopez@ti.com> + +description: + OMAP ALSA SoC DAI driver using McPDM port used by TWL6040 + +properties: + compatible: + const: ti,omap4-mcpdm + + reg: + items: + - description: MPU access base address + - description: L3 interconnect address + + reg-names: + items: + - const: mpu + - const: dma + + interrupts: + maxItems: 1 + + dmas: + maxItems: 2 + + dma-names: + items: + - const: up_link + - const: dn_link + + clocks: + maxItems: 1 + + clock-names: + items: + - const: pdmclk + +required: + - compatible + - reg + - reg-names + - interrupts + - dmas + - dma-names + - clocks + - clock-names + +additionalProperties: false + +examples: + - | + #include <dt-bindings/interrupt-controller/arm-gic.h> + mcpdm@0 { + compatible = "ti,omap4-mcpdm"; + reg = <0x0 0x7f>, /* MPU private access */ + <0x49032000 0x7f>; /* L3 Interconnect */ + reg-names = "mpu", "dma"; + interrupts = <GIC_SPI 112 IRQ_TYPE_LEVEL_HIGH>; + interrupt-parent = <&gic>; + dmas = <&sdma 65>, <&sdma 66>; + dma-names = "up_link", "dn_link"; + clocks = <&twl6040>; + clock-names = "pdmclk"; + };