Re: [PATCH RESEND v3 2/2] ARM: dts: sun4i: Add support for Topwise A721 tablet
Hi, On Tue, Feb 16, 2021 at 05:59:54PM +0100, Pascal Roeleven wrote: > The Topwise A721/LY-F1 tablet is a tablet sold around 2012 under > different brands. The mainboard mentions A721 clearly, so this tablet > is best known under this name. > > Signed-off-by: Pascal Roeleven > --- > arch/arm/boot/dts/Makefile | 3 +- > arch/arm/boot/dts/sun4i-a10-topwise-a721.dts | 242 +++ > 2 files changed, 244 insertions(+), 1 deletion(-) > create mode 100644 arch/arm/boot/dts/sun4i-a10-topwise-a721.dts > > diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile > index 3d1ea0b251..ba25b4235a 100644 > --- a/arch/arm/boot/dts/Makefile > +++ b/arch/arm/boot/dts/Makefile > @@ -1103,7 +1103,8 @@ dtb-$(CONFIG_MACH_SUN4I) += \ > sun4i-a10-olinuxino-lime.dtb \ > sun4i-a10-pcduino.dtb \ > sun4i-a10-pcduino2.dtb \ > - sun4i-a10-pov-protab2-ips9.dtb > + sun4i-a10-pov-protab2-ips9.dtb \ > + sun4i-a10-topwise-a721.dtb > dtb-$(CONFIG_MACH_SUN5I) += \ > sun5i-a10s-auxtek-t003.dtb \ > sun5i-a10s-auxtek-t004.dtb \ > diff --git a/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts > b/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts > new file mode 100644 > index 00..936171d30b > --- /dev/null > +++ b/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts > @@ -0,0 +1,242 @@ > +// SPDX-License-Identifier: GPL-2.0+ > +/* > + * Copyright 2020 Pascal Roeleven > + */ > + > +/dts-v1/; > +#include "sun4i-a10.dtsi" > +#include "sunxi-common-regulators.dtsi" > + > +#include > +#include > +#include > +#include > + > +/ { > + model = "Topwise A721"; > + compatible = "topwise,a721", "allwinner,sun4i-a10"; > + > + aliases { > + serial0 = > + }; > + > + backlight: backlight { > + compatible = "pwm-backlight"; > + pwms = < 0 10 PWM_POLARITY_INVERTED>; > + power-supply = <_vbat>; > + enable-gpios = < 7 7 GPIO_ACTIVE_HIGH>; /* PH7 */ > + brightness-levels = <0 30 40 50 60 70 80 90 100>; > + default-brightness-level = <8>; > + }; > + > + chosen { > + stdout-path = "serial0:115200n8"; > + }; > + > + panel: panel { > + compatible = "starry,kr070pe2t"; > + backlight = <>; > + power-supply = <_lcd_power>; > + > + port { > + panel_input: endpoint { > + remote-endpoint = <_out_panel>; > + }; > + }; > + }; > + > + reg_lcd_power: reg-lcd-power { > + compatible = "regulator-fixed"; > + regulator-name = "reg-lcd-power"; > + gpio = < 7 8 GPIO_ACTIVE_HIGH>; /* PH8 */ > + enable-active-high; > + }; > + > + reg_vbat: reg-vbat { > + compatible = "regulator-fixed"; > + regulator-name = "vbat"; > + regulator-min-microvolt = <370>; > + regulator-max-microvolt = <370>; > + }; > + > +}; > + > + { > + status = "okay"; > +}; > + > + { > + cpu-supply = <_dcdc2>; > +}; > + > + { > + status = "okay"; > +}; > + > + { > + status = "okay"; > +}; > + > + { > + status = "okay"; > +}; > + > + { > + status = "okay"; > + > + axp209: pmic@34 { > + reg = <0x34>; > + interrupts = <0>; > + }; > +}; > + > +#include "axp209.dtsi" > + > +_power_supply { > + status = "okay"; > +}; > + > +_power_supply { > + status = "okay"; > +}; > + > + { > + status = "okay"; > + > + mma7660: accelerometer@4c { Chances are you don't need that label? > + compatible = "fsl,mma7660"; > + reg = <0x4c>; > + }; > +}; > + > + { > + status = "okay"; > + > + ft5406ee8: touchscreen@38 { Ditto > + compatible = "edt,edt-ft5406"; > + reg = <0x38>; > + interrupt-parent = <>; > + interrupts = <7 21 IRQ_TYPE_EDGE_FALLING>; > + touchscreen-size-x = <800>; > + touchscreen-size-y = <480>; > + vcc-supply = <_vcc3v3>; > + }; > +}; > + > + { > + vref-supply = <_ldo2>; > + status = "okay"; > + > + button-vol-down { > + label = "Volume Down"; > + linux,code = ; > + channel = <0>; > + voltage = <761904>; > + }; > + > + button-vol-up { > + label = "Volume Up"; > + linux,code = ; > + channel = <0>; > + voltage = <571428>; > + }; Those two nodes will raise a warning during the DT validation, please make sure it doesn't have any warning. Maxime signature.asc Description: PGP signature
[PATCH RESEND v3 2/2] ARM: dts: sun4i: Add support for Topwise A721 tablet
The Topwise A721/LY-F1 tablet is a tablet sold around 2012 under different brands. The mainboard mentions A721 clearly, so this tablet is best known under this name. Signed-off-by: Pascal Roeleven --- arch/arm/boot/dts/Makefile | 3 +- arch/arm/boot/dts/sun4i-a10-topwise-a721.dts | 242 +++ 2 files changed, 244 insertions(+), 1 deletion(-) create mode 100644 arch/arm/boot/dts/sun4i-a10-topwise-a721.dts diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index 3d1ea0b251..ba25b4235a 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -1103,7 +1103,8 @@ dtb-$(CONFIG_MACH_SUN4I) += \ sun4i-a10-olinuxino-lime.dtb \ sun4i-a10-pcduino.dtb \ sun4i-a10-pcduino2.dtb \ - sun4i-a10-pov-protab2-ips9.dtb + sun4i-a10-pov-protab2-ips9.dtb \ + sun4i-a10-topwise-a721.dtb dtb-$(CONFIG_MACH_SUN5I) += \ sun5i-a10s-auxtek-t003.dtb \ sun5i-a10s-auxtek-t004.dtb \ diff --git a/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts b/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts new file mode 100644 index 00..936171d30b --- /dev/null +++ b/arch/arm/boot/dts/sun4i-a10-topwise-a721.dts @@ -0,0 +1,242 @@ +// SPDX-License-Identifier: GPL-2.0+ +/* + * Copyright 2020 Pascal Roeleven + */ + +/dts-v1/; +#include "sun4i-a10.dtsi" +#include "sunxi-common-regulators.dtsi" + +#include +#include +#include +#include + +/ { + model = "Topwise A721"; + compatible = "topwise,a721", "allwinner,sun4i-a10"; + + aliases { + serial0 = + }; + + backlight: backlight { + compatible = "pwm-backlight"; + pwms = < 0 10 PWM_POLARITY_INVERTED>; + power-supply = <_vbat>; + enable-gpios = < 7 7 GPIO_ACTIVE_HIGH>; /* PH7 */ + brightness-levels = <0 30 40 50 60 70 80 90 100>; + default-brightness-level = <8>; + }; + + chosen { + stdout-path = "serial0:115200n8"; + }; + + panel: panel { + compatible = "starry,kr070pe2t"; + backlight = <>; + power-supply = <_lcd_power>; + + port { + panel_input: endpoint { + remote-endpoint = <_out_panel>; + }; + }; + }; + + reg_lcd_power: reg-lcd-power { + compatible = "regulator-fixed"; + regulator-name = "reg-lcd-power"; + gpio = < 7 8 GPIO_ACTIVE_HIGH>; /* PH8 */ + enable-active-high; + }; + + reg_vbat: reg-vbat { + compatible = "regulator-fixed"; + regulator-name = "vbat"; + regulator-min-microvolt = <370>; + regulator-max-microvolt = <370>; + }; + +}; + + { + status = "okay"; +}; + + { + cpu-supply = <_dcdc2>; +}; + + { + status = "okay"; +}; + + { + status = "okay"; +}; + + { + status = "okay"; +}; + + { + status = "okay"; + + axp209: pmic@34 { + reg = <0x34>; + interrupts = <0>; + }; +}; + +#include "axp209.dtsi" + +_power_supply { + status = "okay"; +}; + +_power_supply { + status = "okay"; +}; + + { + status = "okay"; + + mma7660: accelerometer@4c { + compatible = "fsl,mma7660"; + reg = <0x4c>; + }; +}; + + { + status = "okay"; + + ft5406ee8: touchscreen@38 { + compatible = "edt,edt-ft5406"; + reg = <0x38>; + interrupt-parent = <>; + interrupts = <7 21 IRQ_TYPE_EDGE_FALLING>; + touchscreen-size-x = <800>; + touchscreen-size-y = <480>; + vcc-supply = <_vcc3v3>; + }; +}; + + { + vref-supply = <_ldo2>; + status = "okay"; + + button-vol-down { + label = "Volume Down"; + linux,code = ; + channel = <0>; + voltage = <761904>; + }; + + button-vol-up { + label = "Volume Up"; + linux,code = ; + channel = <0>; + voltage = <571428>; + }; +}; + + { + vmmc-supply = <_vcc3v3>; + bus-width = <4>; + cd-gpios = < 7 1 GPIO_ACTIVE_LOW>; /* PH01 */ + status = "okay"; +}; + + { + status = "okay"; +}; + + { + status = "okay"; +}; + +_sram { + status = "okay"; +}; + + { + vcc-pb-supply = <_vcc3v3>; + vcc-pf-supply = <_vcc3v3>; + vcc-ph-supply = <_vcc3v3>; +}; + + { + pinctrl-names = "default"; + pinctrl-0 = <_pin>; + status = "okay"; +}; + +_dcdc2 { + regulator-always-on; + regulator-min-microvolt = <100>; + regulator-max-microvolt = <140>; + regulator-name = "vdd-cpu"; +}; + +_dcdc3 { + regulator-always-on; + regulator-min-microvolt =