On Tue, 13 Nov 2018 16:46:33 +0530 Jagan Teki <[email protected]> wrote:
Hi, I couldn't find a schematic for this board, but some things in here look inconsistent: > Amarula A64-Relic board by default bound with Techstar TS8550B > MIPI-DSI panel, add support for it. > > DSI panel connected via board DSI port with, > - DC1SW as AVDD supply > - DCDC2 as DVDD supply Are you sure of that? That's typically the CPU power supply. Also I can't find it below. Should that read DLDO2 instead? > - DCDC1 as VCC-DSI supply Can't find this below, either. Is it DLDO1? > - PD24 gpio for reset pin > - PD23 gpio for backlight enable pin > > Signed-off-by: Jagan Teki <[email protected]> > --- > .../allwinner/sun50i-a64-amarula-relic.dts | 46 > +++++++++++++++++++ 1 file changed, 46 insertions(+) > > diff --git > a/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts > b/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts index > 6cb2b7f0c817..ecc0d8094815 100644 --- > a/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts +++ > b/arch/arm64/boot/dts/allwinner/sun50i-a64-amarula-relic.dts @@ -9,6 > +9,7 @@ #include "sun50i-a64.dtsi" > #include <dt-bindings/gpio/gpio.h> > +#include <dt-bindings/pwm/pwm.h> > > / { > model = "Amarula A64-Relic"; > @@ -18,6 +19,14 @@ > serial0 = &uart0; > }; > > + backlight: backlight { > + compatible = "pwm-backlight"; > + pwms = <&pwm 0 50000 PWM_POLARITY_INVERTED>; > + brightness-levels = <1 2 4 8 16 32 64 128 512>; > + default-brightness-level = <2>; > + enable-gpios = <&pio 3 23 GPIO_ACTIVE_HIGH>; /* > LCD-BL-EN: PD23 */ > + }; > + > chosen { > stdout-path = "serial0:115200n8"; > }; > @@ -30,6 +39,28 @@ > }; > }; > > +&de { > + status = "okay"; > +}; > + > +&dphy { > + status = "okay"; > +}; > + > +&dsi { > + vcc-dsi-supply = <®_dldo1>; Ah, there we have the SoC DSI power supply I was missing for the BPi-M64 patch. But is it DLDO1 or DCDC1, like you wrote above? > + status = "okay"; > + > + panel@0 { > + compatible = "techstar,ts8550b"; > + reg = <0>; > + avdd-supply = <®_dc1sw>; > + dvdd-supply = <®_dldo2>; > + reset-gpios = <&pio 3 24 GPIO_ACTIVE_HIGH>; /* > LCD-RST: PD24 */ > + backlight = <&backlight>; > + }; > +}; > + > &ehci0 { > status = "okay"; > }; > @@ -72,6 +103,12 @@ > status = "okay"; > }; > > +&pwm { > + pinctrl-names = "default"; > + pinctrl-0 = <&pwm_pin>; > + status = "okay"; > +}; > + > &r_rsb { > status = "okay"; > > @@ -107,6 +144,15 @@ > regulator-name = "vcc-pll-avcc"; > }; > > +®_dc1sw { > + /* > + * This regulator also indirectly drives the PD pingroup > GPIOs, > + * which also controls the power LED. > + */ Is that true for this board as well or is this just a copy&paste leftover from the BananaPi-M64? If not, you should loose the regulator-always-on property. > + regulator-always-on; > + regulator-name = "vcc-phy"; Shouldn't this be called "vcc-dsi" or so? Cheers, Andre. > +}; > + > ®_dcdc1 { > regulator-always-on; > regulator-min-microvolt = <3300000>; _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
