On 2021/4/30 23:21, Rob Herring wrote: > On Thu, Apr 22, 2021 at 04:58:36PM +0800, Zhen Lei wrote: >> The /schemas/serial.yaml from dt-schema only has the property $nodename, >> whereas the kernel-defined /schemas/serial/serial.yaml contains more >> useful properties, support for more complex application scenarios. >> >> For example: >> 1) The property "current-speed" in fsl-lx2160a.dtsi >> 2) The subnode "bluetooth" in hi3660-hikey960.dts >> >> Signed-off-by: Zhen Lei <thunder.leizhen@xxxxxxxxxx> >> --- >> Documentation/devicetree/bindings/serial/pl011.yaml | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) > > What about all these: Currently, the extended attributes described in /schemas/serial/serial.yaml are not referenced in the dts of other serial ports except pl011. So I didn't change them. At least I run dtbs_check one by one on the ARM64 platform, no warning is reported. Now, it seems that I was wrong, /schemas/serial/serial.yaml is applicable to all serial ports. Even if those attributes are not currently referenced, the reference relationship should be corrected in advance. > > $ git grep 'schemas/serial.yaml' > Documentation/devicetree/bindings/serial/8250.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/brcm,bcm7271-uart.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/ingenic,uart.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/pl011.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/qca,ar9330-uart.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/samsung_uart.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/sifive-serial.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/serial/snps-dw-apb-uart.yaml: - $ref: /schemas/serial.yaml# > Documentation/devicetree/bindings/soc/qcom/qcom,geni-se.yaml: $ref: /schemas/serial.yaml# > > . >
