linux-kernel - Re: [PATCH 2/2] ARM: dts: at91: sama5d2: fix compilation warning

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <334fd5d3-2985-0df2-0324-b05c8ef93b42@microchip.com>
Date:   Mon, 4 Jul 2022 06:46:28 +0000
From:   <Claudiu.Beznea@...rochip.com>
To:     <Nicolas.Ferre@...rochip.com>, <alexandre.belloni@...tlin.com>,
        <robh+dt@...nel.org>, <krzysztof.kozlowski+dt@...aro.org>
CC:     <linux-arm-kernel@...ts.infradead.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 2/2] ARM: dts: at91: sama5d2: fix compilation warning

On 15.06.2022 11:06, Claudiu Beznea wrote:
> Fix the following compilation warning:
> arch/arm/boot/dts/sama5d2.dtsi:371.29-382.6: Warning
> (avoid_unnecessary_addr_size): /ahb/apb/ethernet@...08000:
> unnecessary #address-cells/#size-cells without "ranges" or child "reg" property
> also defined at arch/arm/boot/dts/at91-sama5d2_icp.dts:353.8-363.3
> 
> Signed-off-by: Claudiu Beznea <claudiu.beznea@...rochip.com>

Applied to at91-dt, thanks!

> ---
>  arch/arm/boot/dts/at91-sama5d27_som1.dtsi   | 2 ++
>  arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi | 2 ++
>  arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts   | 2 ++
>  arch/arm/boot/dts/at91-sama5d2_xplained.dts | 2 ++
>  arch/arm/boot/dts/sama5d2.dtsi              | 2 --
>  5 files changed, 8 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
> index a4623cc67cc1..c365731b2cce 100644
> --- a/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
> +++ b/arch/arm/boot/dts/at91-sama5d27_som1.dtsi
> @@ -83,6 +83,8 @@ kernel@...000 {
>  			macb0: ethernet@...08000 {
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&pinctrl_macb0_default>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
>  				phy-mode = "rmii";
>  
>  				ethernet-phy@7 {
> diff --git a/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi b/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
> index ba621783acdb..76b2025c67b4 100644
> --- a/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
> +++ b/arch/arm/boot/dts/at91-sama5d27_wlsom1.dtsi
> @@ -194,6 +194,8 @@ regulator-state-mem {
>  &macb0 {
>  	pinctrl-names = "default";
>  	pinctrl-0 = <&pinctrl_macb0_default>;
> +	#address-cells = <1>;
> +	#size-cells = <0>;
>  	phy-mode = "rmii";
>  
>  	ethernet-phy@0 {
> diff --git a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> index 8ed58af01391..88cf125a55ed 100644
> --- a/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> +++ b/arch/arm/boot/dts/at91-sama5d2_ptc_ek.dts
> @@ -139,6 +139,8 @@ spi0: spi@...00000 {
>  			macb0: ethernet@...08000 {
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
>  				phy-mode = "rmii";
>  				status = "okay";
>  
> diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> index cdfe891f9a9e..bcc3c1bacb25 100644
> --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts
> @@ -147,6 +147,8 @@ flash@0 {
>  			macb0: ethernet@...08000 {
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&pinctrl_macb0_default &pinctrl_macb0_phy_irq>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
>  				phy-mode = "rmii";
>  				status = "okay";
>  
> diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi
> index b147db8b973f..5b2f2e922d40 100644
> --- a/arch/arm/boot/dts/sama5d2.dtsi
> +++ b/arch/arm/boot/dts/sama5d2.dtsi
> @@ -384,8 +384,6 @@ macb0: ethernet@...08000 {
>  				interrupts = <5  IRQ_TYPE_LEVEL_HIGH 3		/* Queue 0 */
>  					      66 IRQ_TYPE_LEVEL_HIGH 3          /* Queue 1 */
>  					      67 IRQ_TYPE_LEVEL_HIGH 3>;        /* Queue 2 */
> -				#address-cells = <1>;
> -				#size-cells = <0>;
>  				clocks = <&pmc PMC_TYPE_PERIPHERAL 5>, <&pmc PMC_TYPE_PERIPHERAL 5>;
>  				clock-names = "hclk", "pclk";
>  				status = "disabled";