linux-kernel - Re: [PATCH 16/19] powerpc: dts: Add LANCOM NWAPP2 board devicetree

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <b1e0fa68-6f55-4c21-8d61-f8d58d932afc@kernel.org>
Date: Fri, 10 Jan 2025 16:24:27 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: j.ne@...teo.net, Michael Ellerman <mpe@...erman.id.au>,
 Nicholas Piggin <npiggin@...il.com>,
 Christophe Leroy <christophe.leroy@...roup.eu>,
 Naveen N Rao <naveen@...nel.org>, Madhavan Srinivasan <maddy@...ux.ibm.com>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Linus Walleij
 <linus.walleij@...aro.org>, Bartosz Golaszewski <brgl@...ev.pl>,
 Frank Li <Frank.Li@....com>
Cc: linuxppc-dev@...ts.ozlabs.org, linux-kernel@...r.kernel.org,
 devicetree@...r.kernel.org, linux-gpio@...r.kernel.org
Subject: Re: [PATCH 16/19] powerpc: dts: Add LANCOM NWAPP2 board devicetree

On 02/01/2025 19:31, J. Neuschäfer via B4 Relay wrote:
>  arch/powerpc/boot/dts/Makefile          |   1 +
>  arch/powerpc/boot/dts/lancom-nwapp2.dts | 276 ++++++++++++++++++++++++++++++++
>  2 files changed, 277 insertions(+)
> 
> diff --git a/arch/powerpc/boot/dts/Makefile b/arch/powerpc/boot/dts/Makefile
> index 6aee895d5baaa2c978d4b1c82a6d198d9e166ea2..43e20d8b776b1716a91e6ec8e135c0a45c8e8c77 100644
> --- a/arch/powerpc/boot/dts/Makefile
> +++ b/arch/powerpc/boot/dts/Makefile
> @@ -6,5 +6,6 @@ dtb-$(CONFIG_OF_ALL_DTBS) := $(patsubst $(src)/%.dts,%.dtb, $(wildcard $(src)/*.
>  
>  dtb-$(CONFIG_MPC831x) += \
>  	kmeter1.dtb \
> +	lancom-nwapp2.dtb \
>  	mpc8313erdb.dtb \
>  	mpc8315erdb.dtb
> diff --git a/arch/powerpc/boot/dts/lancom-nwapp2.dts b/arch/powerpc/boot/dts/lancom-nwapp2.dts
> new file mode 100644
> index 0000000000000000000000000000000000000000..d46d94c27ecb91db32c8acb4136cb6e8d0df52f5
> --- /dev/null
> +++ b/arch/powerpc/boot/dts/lancom-nwapp2.dts
> @@ -0,0 +1,276 @@
> +// SPDX-License-Identifier: GPL-2.0-or-later
> +// Copyright 2024 J. Neuschäfer
> +//
> +// LANCOM NWAPP2, marketed as:
> +// - LANCOM L-321agn Wireless (R2)
> +// - LANCOM L-322agn dual Wireless (R2)
> +// - LANCOM L-322E Wireless
> +#include "mpc8314e.dtsi"
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/leds/common.h>
> +#include <dt-bindings/input/input.h>
> +
> +/ {
> +	compatible = "lancom,nwapp2", "fsl,mpc8314e";

Missing bindings. Please run scripts/checkpatch.pl and fix reported
warnings. After that, run also `scripts/checkpatch.pl --strict` and
(probably) fix more warnings. Some warnings can be ignored, especially
from --strict run, but the code here looks like it needs a fix. Feel
free to get in touch if the warning is not clear.


> +	model = "LANCOM NWAPP2";
> +
> +	memory {
> +		reg = <0x00000000 0x08000000>;	// 128MB at 0
> +	};

...

> +
> +	led-power-top {
> +		label = "multicolor:power-top";
> +		compatible = "leds-group-multicolor";
> +		color = <LED_COLOR_ID_MULTI>;
> +		function = LED_FUNCTION_POWER;
> +		leds = <&led_power_top_red>, <&led_power_top_green>;
> +	};
> +
> +	led-wlan-top {
> +		label = "multicolor:wlan-top";
> +		compatible = "leds-group-multicolor";
> +		color = <LED_COLOR_ID_MULTI>;
> +		leds = <&led_wlan_top_red>, <&led_wlan_top_green>;
> +	};
> +};
> +
> +&gpio {
> +	gpio-line-names =
> +		/*  0 -  7 */ "DEBUG#", "RESET_BTN#", "", "EXP_LATCH", "", "", "", "PCIE_RST#",
> +		/*  8 - 15 */ "", "", "BOARD_RST", "", "", "", "", "",
> +		/* 16 - 23 */ "", "", "WDT_TOGGLE", "", "", "", "PHY_RST#", "",
> +		/* 24 - 31 */ "", "", "", "", "", "", "", "";
> +};
> +
> +&localbus {
> +	status = "okay";
> +	ranges = <0x0 0x0 0xfe000000 0x00008000>;
> +
> +	nand@0,0 {
> +		#address-cells = <1>;
> +		#size-cells = <1>;
> +		compatible = "fsl,mpc8315-fcm-nand",

Compatible is always the first property.

> +			     "fsl,elbc-fcm-nand";
> +		reg = <0x0 0x0 0x2000>;

reg follows.

> +
> +		partitions {
> +			compatible = "fixed-partitions";
> +			#address-cells = <1>;
> +			#size-cells = <1>;
> +



Best regards,
Krzysztof