[PATCH 2/2] ARM: dts: BCM5301X: Add support for Buffalo WXR-1750DHP

[Krzysztof Kozlowski]

On Thu, Jun 26, 2025 at 12:43:15AM +0900, Taishi Shimizu wrote:
> +
> +#include "bcm4708.dtsi"
> +#include "bcm5301x-nand-cs0-bch8.dtsi"
> +
> +/ {
> +	compatible = "buffalo,wxr-1750dhp", "brcm,bcm4708";
> +	model = "Buffalo WXR-1750DHP";
> +
> +	chosen {
> +		bootargs = "console=ttyS0,115200";

Please use stdout path property.

> +	};
> +
> +	memory at 0 {
> +		reg = <0x00000000 0x08000000>,
> +		      <0x88000000 0x08000000>;
> +		device_type = "memory";
> +	};
> +
> +	gpio-keys {
> +		compatible = "gpio-keys";
> +
> +		button-aoss {
> +			label = "AOSS";
> +			linux,code = <KEY_WPS_BUTTON>;
> +			gpios = <&chipcommon 2 GPIO_ACTIVE_LOW>;
> +		};
> +
> +		/* GPIO 8 and 9 are a tri-state switch button with
> +		 * ROUTER / AP / WB.
> +		 */
> +		button-bridge {
> +			label = "WB";
> +			linux,code = <BTN_2>;
> +			linux,input-type = <EV_SW>;
> +			gpios = <&chipcommon 8 GPIO_ACTIVE_HIGH>;
> +		};
> +
> +		/* GPIO 3 is a switch button with AUTO / MANUAL. */
> +		button-manual {
> +			label = "MANUAL";
> +			linux,code = <BTN_0>;
> +			linux,input-type = <EV_SW>;
> +			gpios = <&chipcommon 3 GPIO_ACTIVE_HIGH>;
> +		};
> +
> +		button-restart {
> +			label = "Reset";
> +			linux,code = <KEY_RESTART>;
> +			gpios = <&chipcommon 11 GPIO_ACTIVE_LOW>;
> +		};
> +
> +		button-router {
> +			label = "ROUTER";
> +			linux,code = <BTN_1>;
> +			linux,input-type = <EV_SW>;
> +			gpios = <&chipcommon 9 GPIO_ACTIVE_HIGH>;
> +		};
> +	};
> +
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		led-power0 {
> +			label = "bcm53xx:white:power";

You should use rather color and function properties.

> +			linux,default-trigger = "default-on";
> +			gpios = <&chipcommon 5 GPIO_ACTIVE_HIGH>;
> +		};

Best regards,
Krzysztof