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: <c3f84f77-dc2c-447e-99cf-dedb20a2f93a@linaro.org>
Date: Mon, 24 Mar 2025 07:59:48 +0100
From: Neil Armstrong <neil.armstrong@...aro.org>
To: j.ne@...teo.net, Rob Herring <robh@...nel.org>,
 Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley
 <conor+dt@...nel.org>,
 Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
 Jerome Brunet <jbrunet@...libre.com>, Kevin Hilman <khilman@...libre.com>
Cc: devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
 linux-arm-kernel@...ts.infradead.org, linux-amlogic@...ts.infradead.org
Subject: Re: [PATCH 3/3] ARM: dts: amlogic: Add TCU Fernsehfee 3.0

On 23/03/2025 13:37, J. Neuschäfer via B4 Relay wrote:
> From: "J. Neuschäfer" <j.ne@...teo.net>
> 
> Fernsehfee[1] ("TV fairy") 3.0 is a set-top box with HDMI input and
> output ports. It originally ran Android 4.4 and a Linux 3.10 kernel.
> 
> The following features are tested and known to work:
> 
> - Ethernet
> - Power LED (switching between green and red)
> - Power button
> - eMMC
> - SD Card
> - USB
> - Wifi
> 
> The following features are untested or not working:
> 
> - HDMI input and output
> - Infrared remote control input and output
> 
> [1]: https://fernsehfee.de/ (German), https://telefairy.com/ (English)
> 
> Signed-off-by: J. Neuschäfer <j.ne@...teo.net>
> ---
>   arch/arm/boot/dts/amlogic/Makefile               |   1 +
>   arch/arm/boot/dts/amlogic/meson8-fernsehfee3.dts | 219 +++++++++++++++++++++++
>   arch/arm/boot/dts/amlogic/meson8.dtsi            |  18 ++
>   3 files changed, 238 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/amlogic/Makefile b/arch/arm/boot/dts/amlogic/Makefile
> index 504c533b1173298ec7f45099888d88b2fb74b978..3c8a1e88b386cd7fb9b5f41f47d7ff78fe7eeacc 100644
> --- a/arch/arm/boot/dts/amlogic/Makefile
> +++ b/arch/arm/boot/dts/amlogic/Makefile
> @@ -1,6 +1,7 @@
>   # SPDX-License-Identifier: GPL-2.0
>   dtb-$(CONFIG_MACH_MESON8) += \
>   	meson8-minix-neo-x8.dtb \
> +	meson8-fernsehfee3.dtb \
>   	meson8b-ec100.dtb \
>   	meson8b-mxq.dtb \
>   	meson8b-odroidc1.dtb \
> diff --git a/arch/arm/boot/dts/amlogic/meson8-fernsehfee3.dts b/arch/arm/boot/dts/amlogic/meson8-fernsehfee3.dts
> new file mode 100644
> index 0000000000000000000000000000000000000000..47507316ac4a56f2bcf84ad0446a06ea9fe890cd
> --- /dev/null
> +++ b/arch/arm/boot/dts/amlogic/meson8-fernsehfee3.dts
> @@ -0,0 +1,219 @@
> +// SPDX-License-Identifier: GPL-2.0-only OR MIT
> +// Copyright (C) 2025 J. Neuschäfer <j.ne@...teo.net>
> +
> +/dts-v1/;
> +
> +#include <dt-bindings/gpio/gpio.h>
> +#include <dt-bindings/input/linux-event-codes.h>
> +#include <dt-bindings/leds/common.h>
> +
> +#include "meson8.dtsi"
> +
> +/ {
> +	model = "Fernsehfee 3.0";
> +	compatible = "tcu,fernsehfee3", "amlogic,meson8";
> +
> +	aliases {
> +		serial0 = &uart_AO;
> +		gpiochip0 = &gpio;
> +		gpiochip1 = &gpio_ao;
> +		i2c0 = &i2c_AO;
> +		i2c1 = &i2c_B;
> +		mmc0 = &sdhc;
> +		mmc1 = &sdio;
> +	};
> +
> +	chosen {
> +		stdout-path = "serial0:115200n8";
> +	};
> +
> +	memory@0 {
> +		device_type = "memory";
> +		reg = <0x0 0x40000000>;  /* 1 GiB */
> +	};
> +
> +	gpio-keys {
> +		compatible = "gpio-keys-polled";
> +		poll-interval = <100>;
> +
> +		power-button {
> +			label = "Power button";
> +			linux,code = <KEY_POWER>;
> +			gpios = <&gpio_ao GPIOAO_3 GPIO_ACTIVE_LOW>;
> +		};
> +	};
> +
> +	leds {
> +		compatible = "gpio-leds";
> +
> +		led-0 {
> +			/*
> +			 * The power LED can be turned red, otherwise it is green.
> +			 */
> +			gpios = <&gpio_ao GPIO_TEST_N GPIO_ACTIVE_LOW>;
> +			function = LED_FUNCTION_POWER;
> +			color = <LED_COLOR_ID_RED>;
> +		};
> +	};
> +
> +	vcc_5v: regulator-5v {
> +		/* 5V rail, always on as long as the system is running */
> +		compatible = "regulator-fixed";
> +		regulator-name = "5V";
> +		regulator-min-microvolt = <5000000>;
> +		regulator-max-microvolt = <5000000>;
> +		regulator-always-on;
> +	};
> +
> +	vcc_3v3: regulator-3v3 {
> +		/* Chipown AP2420 step-down converter */
> +		compatible = "regulator-fixed";
> +		regulator-name = "3.3V";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vcc_5v>;
> +	};
> +
> +	vcc_1v8: regulator-1v8 {
> +		compatible = "regulator-fixed";
> +		regulator-name = "1.8V";
> +		regulator-min-microvolt = <1800000>;
> +		regulator-max-microvolt = <1800000>;
> +		vin-supply = <&vcc_3v3>;
> +	};
> +
> +	wifi_3v3: regulator-wifi {
> +		compatible = "regulator-fixed";
> +		regulator-name = "3.3V-WIFI";
> +		regulator-min-microvolt = <3300000>;
> +		regulator-max-microvolt = <3300000>;
> +		vin-supply = <&vcc_3v3>;
> +		gpio = <&gpio GPIOX_11 GPIO_ACTIVE_HIGH>;
> +		enable-active-high;
> +	};
> +};
> +
> +&ethmac {
> +	status = "okay";
> +	pinctrl-0 = <&eth_pins>;
> +	pinctrl-names = "default";
> +	phy-handle = <&eth_phy0>;
> +	phy-mode = "rmii";
> +
> +	mdio {
> +		compatible = "snps,dwmac-mdio";
> +		#address-cells = <1>;
> +		#size-cells = <0>;
> +
> +		eth_phy0: ethernet-phy@0 {
> +			/* IC Plus IP101A (0x02430c54) */
> +			reg = <0>;
> +
> +			reset-assert-us = <10000>;
> +			reset-deassert-us = <10000>;
> +			reset-gpios = <&gpio GPIOH_4 GPIO_ACTIVE_LOW>;
> +		};
> +	};
> +};
> +
> +&i2c_AO {
> +	status = "okay";
> +	pinctrl-0 = <&i2c_ao_pins>;
> +	pinctrl-names = "default";
> +
> +	pmic@32 {
> +		compatible = "ricoh,rn5t618";
> +		reg = <0x32>;
> +		system-power-controller;
> +	};
> +
> +	eeprom@50 {
> +		/* Fairchild FM24C08A */
> +		compatible = "atmel,24c08";
> +		reg = <0x50>;
> +		pagesize = <16>;
> +		wp-gpios = <&gpio GPIOH_3 GPIO_ACTIVE_HIGH>;
> +		num-addresses = <4>;
> +	};
> +};
> +
> +&i2c_B {
> +	status = "okay";
> +	pinctrl-0 = <&i2c_b_pins>;
> +	pinctrl-names = "default";
> +
> +	/* TODO: SiI9293 HDMI receiver @ 0x39 */
> +};
> +
> +&sdhc {
> +	status = "okay";
> +	pinctrl-0 = <&sdxc_c_pins>;
> +	pinctrl-names = "default";
> +
> +	/* eMMC */
> +	bus-width = <8>;
> +	max-frequency = <100000000>;
> +
> +	disable-wp;
> +	cap-mmc-highspeed;
> +	mmc-hs200-1_8v;
> +	no-sdio;
> +
> +	vmmc-supply = <&vcc_3v3>;
> +	vqmmc-supply = <&vcc_1v8>;
> +};
> +
> +&sdio {
> +	status = "okay";
> +	pinctrl-0 = <&sd_b_pins>;
> +
> +	/* SD card */
> +	sd_card_slot: slot@1 {

Small NIT: no need to for a label here

> +		compatible = "mmc-slot";
> +		reg = <1>;
> +		status = "okay";
> +
> +		bus-width = <4>;
> +		cap-mmc-highspeed;
> +		cap-sd-highspeed;
> +		disable-wp;
> +
> +		cd-gpios = <&gpio CARD_6 GPIO_ACTIVE_LOW>;
> +
> +		vmmc-supply = <&vcc_3v3>;
> +	};
> +};
> +
> +&uart_AO {
> +	status = "okay";
> +	pinctrl-0 = <&uart_ao_a_pins>;
> +	pinctrl-names = "default";
> +};
> +
> +&usb0 {
> +	status = "okay";
> +};
> +
> +&usb0_phy {
> +	status = "okay";
> +};
> +
> +&usb1 {
> +	status = "okay";
> +	dr_mode = "host";
> +	/*
> +	 * This bus features a Realtek RTL8188 2.4GHz WiFi module, with a
> +	 * 3.3V supply voltage that must be enabled before use.
> +	 */
> +	vbus-supply = <&wifi_3v3>;
> +};
> +
> +&usb1_phy {
> +	status = "okay";
> +};
> +
> +&ir_receiver {
> +	status = "okay";
> +	pinctrl-0 = <&ir_recv_pins>;
> +	pinctrl-names = "default";
> +};
> diff --git a/arch/arm/boot/dts/amlogic/meson8.dtsi b/arch/arm/boot/dts/amlogic/meson8.dtsi
> index 9ff142d9fe3f4576fdd3230a966c8a6250870de7..300eccbfc0071ce10290be1c496132ac6b6a4dbc 100644
> --- a/arch/arm/boot/dts/amlogic/meson8.dtsi
> +++ b/arch/arm/boot/dts/amlogic/meson8.dtsi
> @@ -477,6 +477,14 @@ gpio: bank@80 {
>   			gpio-ranges = <&pinctrl_cbus 0 0 120>;
>   		};
>   
> +		i2c_b_pins: i2c-b {
> +			mux {
> +				groups = "i2c_sda_b", "i2c_sck_b";
> +				function = "i2c_b";
> +				bias-disable;
> +			};
> +		};
> +
>   		sd_a_pins: sd-a {
>   			mux {
>   				groups = "sd_d0_a", "sd_d1_a", "sd_d2_a",
> @@ -522,6 +530,16 @@ mux {
>   			};
>   		};
>   
> +		sdxc_c_pins: sdxc-c {
> +			mux {
> +				groups = "sdxc_d0_c", "sdxc_d13_c",
> +					"sdxc_clk_c", "sdxc_cmd_c",
> +					"sdxc_d47_c";
> +				function = "sdxc_c";
> +				bias_pull_up;
> +			};
> +		};
> +
>   		spdif_out_pins: spdif-out {
>   			mux {
>   				groups = "spdif_out";
> 

Apart that:
Reviewed-by: Neil Armstrong <neil.armstrong@...aro.org>

Thanks !
Neil

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ