On 24/01/2023 10:42, Herve Codina wrote: > Hi Krzysztof, > > On Tue, 17 Jan 2023 12:31:09 +0100 > Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > >> On 13/01/2023 11:37, Herve Codina wrote: >>> Add support for the QMC (QUICC Multichannel Controller) >>> available in some PowerQUICC SoC such as MPC885 or MPC866. >>> >>> Signed-off-by: Herve Codina <herve.codina@xxxxxxxxxxx> >>> --- >>> .../bindings/soc/fsl/cpm_qe/fsl,qmc.yaml | 164 ++++++++++++++++++ >>> 1 file changed, 164 insertions(+) >>> create mode 100644 Documentation/devicetree/bindings/soc/fsl/cpm_qe/fsl,qmc.yaml >>> >>> diff --git a/Documentation/devicetree/bindings/soc/fsl/cpm_qe/fsl,qmc.yaml b/Documentation/devicetree/bindings/soc/fsl/cpm_qe/fsl,qmc.yaml >>> new file mode 100644 >>> index 000000000000..3ec52f1635c8 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/soc/fsl/cpm_qe/fsl,qmc.yaml >>> @@ -0,0 +1,164 @@ >>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) >>> +%YAML 1.2 >>> +--- >>> +$id: http://devicetree.org/schemas/soc/fsl/cpm_qe/fsl,qmc.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: PowerQUICC CPM QUICC Multichannel Controller (QMC) >>> + >>> +maintainers: >>> + - Herve Codina <herve.codina@xxxxxxxxxxx> >>> + >>> +description: | >>> + The QMC (QUICC Multichannel Controller) emulates up to 64 channels within >>> + one serial controller using the same TDM physical interface routed from >>> + TSA. >>> + >>> +properties: >>> + compatible: >>> + items: >>> + - enum: >>> + - fsl,mpc885-scc-qmc >>> + - fsl,mpc866-scc-qmc >>> + - const: fsl,cpm1-scc-qmc >>> + >>> + reg: >>> + items: >>> + - description: SCC (Serial communication controller) register base >>> + - description: SCC parameter ram base >>> + - description: Dual port ram base >>> + >>> + reg-names: >>> + items: >>> + - const: scc_regs >>> + - const: scc_pram >>> + - const: dpram >>> + >>> + interrupts: >>> + maxItems: 1 >>> + description: SCC interrupt line in the CPM interrupt controller >>> + >>> + fsl,tsa: >>> + $ref: /schemas/types.yaml#/definitions/phandle >>> + description: phandle to the TSA >>> + >>> + fsl,tsa-cell-id: >>> + $ref: /schemas/types.yaml#/definitions/uint32 >>> + enum: [1, 2, 3] >>> + description: | >>> + TSA cell ID (dt-bindings/soc/fsl,tsa.h defines these values) >>> + - 1: SCC2 >>> + - 2: SCC3 >>> + - 3: SCC4 >> >> Is this used as argument to tsa? If so, this should be part of fsl,tsa >> property, just like we do for all syscon-like phandles. > > Yes, indeed. > I will move 'fsl,tsa' to 'fsl,tsa-cell' with 'fsl,tsa-cell' a phandle/number > pair (the phandle to TSA node and the TSA cell id to use) Move to fsl,tsa, not from. Best regards, Krzysztof
