2017-11-29 23:22:46

by David Daney

[permalink] [raw]
Subject: Re: [PATCH] dt-bindings: Remove leading 0x from bindings notation

On 11/29/2017 12:55 PM, Mathieu Malaterre wrote:
> Improve the binding example by removing all the leading 0x to fix the
> following dtc warnings:
>
> Warning (unit_address_format): Node /XXX unit name should not have leading "0x"

How does it fix the warnings? You are not changing the .dts files that
are compiled.

This may also cause the binding documentation to differ from the reality
of what the actual device trees contain.

>
> Converted using the following command:
>
> find Documentation/devicetree/bindings -name "*.txt" -exec sed -i -e 's/([^ ])\@0x([0-9a-f])/$1\@$2/g' {} +
>
> This is a follow up to commit 48c926cd3414
>
> Signed-off-by: Mathieu Malaterre <[email protected]>
> ---
> I've also checked using the original perl command that I did not introduce:
>
> Warning (unit_address_format): Node /XXX unit name should not have leading 0s
>
> Documentation/devicetree/bindings/arm/ccn.txt | 2 +-
> Documentation/devicetree/bindings/arm/omap/crossbar.txt | 2 +-
> .../devicetree/bindings/arm/tegra/nvidia,tegra20-mc.txt | 2 +-
> Documentation/devicetree/bindings/clock/axi-clkgen.txt | 2 +-
> .../devicetree/bindings/clock/brcm,bcm2835-aux-clock.txt | 2 +-
> Documentation/devicetree/bindings/clock/exynos4-clock.txt | 2 +-
> Documentation/devicetree/bindings/clock/exynos5250-clock.txt | 2 +-
> Documentation/devicetree/bindings/clock/exynos5410-clock.txt | 2 +-
> Documentation/devicetree/bindings/clock/exynos5420-clock.txt | 2 +-
> Documentation/devicetree/bindings/clock/exynos5440-clock.txt | 2 +-
> .../devicetree/bindings/clock/ti-keystone-pllctrl.txt | 2 +-
> Documentation/devicetree/bindings/clock/zx296702-clk.txt | 4 ++--
> Documentation/devicetree/bindings/crypto/fsl-sec4.txt | 4 ++--
> .../devicetree/bindings/devfreq/event/rockchip-dfi.txt | 2 +-
> Documentation/devicetree/bindings/display/atmel,lcdc.txt | 4 ++--
> Documentation/devicetree/bindings/dma/qcom_hidma_mgmt.txt | 4 ++--
> Documentation/devicetree/bindings/dma/zxdma.txt | 2 +-
> Documentation/devicetree/bindings/gpio/gpio-altera.txt | 2 +-
> Documentation/devicetree/bindings/i2c/i2c-jz4780.txt | 2 +-
> Documentation/devicetree/bindings/iio/pressure/hp03.txt | 2 +-
> .../devicetree/bindings/input/touchscreen/bu21013.txt | 2 +-
> .../devicetree/bindings/interrupt-controller/arm,gic.txt | 4 ++--
> .../bindings/interrupt-controller/img,meta-intc.txt | 2 +-
> .../bindings/interrupt-controller/img,pdc-intc.txt | 2 +-
> .../bindings/interrupt-controller/st,spear3xx-shirq.txt | 2 +-
> Documentation/devicetree/bindings/mailbox/altera-mailbox.txt | 6 +++---
> .../devicetree/bindings/mailbox/brcm,iproc-pdc-mbox.txt | 2 +-
> Documentation/devicetree/bindings/media/exynos5-gsc.txt | 2 +-
> Documentation/devicetree/bindings/media/mediatek-vcodec.txt | 2 +-
> Documentation/devicetree/bindings/media/rcar_vin.txt | 2 +-
> Documentation/devicetree/bindings/media/samsung-fimc.txt | 2 +-
> Documentation/devicetree/bindings/media/sh_mobile_ceu.txt | 2 +-
> Documentation/devicetree/bindings/media/video-interfaces.txt | 10 +++++-----
> .../devicetree/bindings/memory-controllers/ti/emif.txt | 2 +-
> .../devicetree/bindings/mfd/ti-keystone-devctrl.txt | 2 +-
> Documentation/devicetree/bindings/misc/brcm,kona-smc.txt | 2 +-
> Documentation/devicetree/bindings/mmc/brcm,kona-sdhci.txt | 2 +-
> Documentation/devicetree/bindings/mmc/brcm,sdhci-iproc.txt | 2 +-
> Documentation/devicetree/bindings/mmc/ti-omap-hsmmc.txt | 4 ++--
> Documentation/devicetree/bindings/mtd/gpmc-nor.txt | 6 +++---
> Documentation/devicetree/bindings/mtd/mtk-nand.txt | 2 +-
> Documentation/devicetree/bindings/net/altera_tse.txt | 4 ++--
> Documentation/devicetree/bindings/net/mdio.txt | 2 +-
> Documentation/devicetree/bindings/net/socfpga-dwmac.txt | 2 +-
> Documentation/devicetree/bindings/nios2/nios2.txt | 2 +-
> Documentation/devicetree/bindings/pci/altera-pcie.txt | 2 +-
> Documentation/devicetree/bindings/pci/fsl,imx6q-pcie.txt | 2 +-
> Documentation/devicetree/bindings/pci/hisilicon-pcie.txt | 2 +-
> Documentation/devicetree/bindings/phy/sun4i-usb-phy.txt | 2 +-
> .../devicetree/bindings/pinctrl/brcm,cygnus-pinmux.txt | 2 +-
> Documentation/devicetree/bindings/pinctrl/pinctrl-atlas7.txt | 4 ++--
> Documentation/devicetree/bindings/pinctrl/pinctrl-sirf.txt | 2 +-
> .../devicetree/bindings/pinctrl/rockchip,pinctrl.txt | 4 ++--
> Documentation/devicetree/bindings/regulator/regulator.txt | 2 +-
> Documentation/devicetree/bindings/serial/efm32-uart.txt | 2 +-
> .../devicetree/bindings/serio/allwinner,sun4i-ps2.txt | 2 +-
> .../devicetree/bindings/soc/ti/keystone-navigator-qmss.txt | 2 +-
> Documentation/devicetree/bindings/sound/adi,axi-i2s.txt | 2 +-
> Documentation/devicetree/bindings/sound/adi,axi-spdif-tx.txt | 2 +-
> Documentation/devicetree/bindings/sound/ak4613.txt | 2 +-
> Documentation/devicetree/bindings/sound/ak4642.txt | 2 +-
> Documentation/devicetree/bindings/sound/max98371.txt | 2 +-
> Documentation/devicetree/bindings/sound/max9867.txt | 2 +-
> Documentation/devicetree/bindings/sound/renesas,fsi.txt | 2 +-
> Documentation/devicetree/bindings/sound/rockchip-spdif.txt | 2 +-
> Documentation/devicetree/bindings/sound/st,sti-asoc-card.txt | 8 ++++----
> Documentation/devicetree/bindings/spi/efm32-spi.txt | 2 +-
> Documentation/devicetree/bindings/thermal/thermal.txt | 12 ++++++------
> Documentation/devicetree/bindings/ufs/ufs-qcom.txt | 4 ++--
> Documentation/devicetree/bindings/ufs/ufshcd-pltfrm.txt | 2 +-
> Documentation/devicetree/bindings/usb/ehci-st.txt | 2 +-
> Documentation/devicetree/bindings/usb/ohci-st.txt | 2 +-
> .../devicetree/bindings/watchdog/ingenic,jz4740-wdt.txt | 2 +-
> 73 files changed, 99 insertions(+), 99 deletions(-)
>

From 1585435636920881123@xxx Wed Nov 29 21:00:47 +0000 2017
X-GM-THRID: 1585435636920881123
X-Gmail-Labels: Inbox,Category Forums,HistoricalUnread