From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mout-p-201.mailbox.org (mout-p-201.mailbox.org [80.241.56.171]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 68F873D9DAC; Tue, 28 Apr 2026 09:44:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=80.241.56.171 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777369469; cv=none; b=Oc2YiFvRMr7U+prDs1ePbHSD+DXeK9M/weEnZF6n3hrwz6dM+3SLszOphnBj16bKgGAw7baPC0XKr2mch02RKfWvIGtnhVCN7Q8RlNzhCRPEoGgMEC3Fz+kBIHgSmgumYpINmDTUS7afRDFhR6Juwhb2Q5V6AgQi9opcXjCOV6Y= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1777369469; c=relaxed/simple; bh=ED9C1BRhmBOdQPvQOsI0zMY7qqhOcOC9QqQB6yXPFr8=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=vBpdhPV5Q13tNEkNd8u1XlfPnfBY7DxsmLfElymSGyEBEJel74AJka8Ra3bFleBjV6IQiwKz7xqcFAoHnJtryNFmDWuEHhEyTPqU9y+RX2KcfteiHd4HV/3/Io2KcJy0VXytMmbFhZKQzlWSmZoPACocu4gUE+qKvQUKVz/Dfsg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=timmermann.space; spf=pass smtp.mailfrom=timmermann.space; arc=none smtp.client-ip=80.241.56.171 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=timmermann.space Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=timmermann.space Received: from smtp202.mailbox.org (smtp202.mailbox.org [10.196.197.202]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mout-p-201.mailbox.org (Postfix) with ESMTPS id 4g4b9r5hwLz9v6d; Tue, 28 Apr 2026 11:44:16 +0200 (CEST) Date: Tue, 28 Apr 2026 11:44:05 +0200 From: Lukas Timmermann To: Krzysztof Kozlowski Cc: Neil Armstrong , Jessica Zhang , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Thierry Reding , Sam Ravnborg , Douglas Anderson , Sylwester Nawrocki , Chanwoo Choi , Alim Akhtar , Michael Turquette , Stephen Boyd , dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-samsung-soc@vger.kernel.org, linux-clk@vger.kernel.org, linux-arm-kernel@lists.infradead.org, Alexandre Marquet Subject: Re: [PATCH 5/5] ARM: dts: exynos: Add display support for exynos5250-manta Message-ID: References: <20260423-manta-display-v1-0-196f80c5673a@timmermann.space> <20260423-manta-display-v1-5-196f80c5673a@timmermann.space> <20260425-eccentric-amiable-fossa-fa1cae@quoll> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260425-eccentric-amiable-fossa-fa1cae@quoll> On Sat, Apr 25, 2026 at 11:58:47AM +0200, Krzysztof Kozlowski wrote: > On Thu, Apr 23, 2026 at 11:14:41PM +0200, Lukas Timmermann wrote: > > Adds the necessary entries for panel and backlight as > > well as the fimd controller. > > > > Signed-off-by: Alexandre Marquet > > Signed-off-by: Lukas Timmermann > > --- > > arch/arm/boot/dts/samsung/exynos5250-manta.dts | 99 ++++++++++++++++++++++++++ > > 1 file changed, 99 insertions(+) > > > > diff --git a/arch/arm/boot/dts/samsung/exynos5250-manta.dts b/arch/arm/boot/dts/samsung/exynos5250-manta.dts > > index 76d3657eb22f..eb97a28ff4e3 100644 > > --- a/arch/arm/boot/dts/samsung/exynos5250-manta.dts > > +++ b/arch/arm/boot/dts/samsung/exynos5250-manta.dts > > @@ -25,6 +25,29 @@ aliases { > > mmc1 = &mmc_1; /* WiFi */ > > }; > > > > + backlight: backlight { > > + compatible = "pwm-backlight"; > > + pwms = <&pwm 0 1000000 0>; > > + > > + brightness-levels = <2 255>; /* TODO */ > > + num-interpolated-steps = <254>; /* TODO */ > > + default-brightness-level = <102>; /* TODO */ > > + post-pwm-on-delay-ms = <97>; /* TODO */ > > + > > + power-supply = <&backlight_reg>; > > + enable-gpios = <&gpg0 5 GPIO_ACTIVE_HIGH>; > > + > > + pinctrl-0 = <&led_bl_reset &pwm0_out>; > > + pinctrl-names = "default"; > > + }; > > + > > + backlight_reg: regulator-backlight { > > + compatible = "regulator-fixed"; > > + regulator-name = "APS_EN_18V"; > > + pinctrl-0 = <&aps_en_18v>; > > + pinctrl-names = "default"; > > No enable GPIO? No voltage? > > > + }; > > + > > /* Voltage source unknown */ > > bmp180_vdda_reg: regulator-bmp180-vdda { > > compatible = "regulator-fixed"; > > @@ -105,6 +128,28 @@ multi-led { > > leds = <&status_red>, <&status_green>, <&status_blue>, <&status_white>; > > }; > > > > + panel { > > + compatible = "samsung,ltl101dl02-002"; > > + > > + backlight = <&backlight>; > > + power-supply = <&panel_reg>; > > + enable-gpios = <&gph1 7 GPIO_ACTIVE_HIGH>; > > + pinctrl-0 = <&lcd_en>; > > + pinctrl-names = "default"; > > + no-hpd; > > + > > + port { > > + panel: endpoint { > > + remote-endpoint = <&dp_out>; > > + }; > > + }; > > + }; > > + > > + panel_reg: regulator-panel { > > + compatible = "regulator-fixed"; > > + regulator-name = "LCD_EN"; > > That's incomplete. You should have at least voltage. > > > + }; > > + > > pwrseq: mmc1-pwrseq { > > compatible = "mmc-pwrseq-simple"; > > > > @@ -146,10 +191,36 @@ &cpu1 { > > cpu-supply = <&buck2_reg>; > > }; > > > > +&dp { > > + status = "okay"; > > + > > + samsung,color-space = <0>; > > + samsung,color-depth = <1>; > > + samsung,link-rate = <0x0a>; > > + samsung,lane-count = <4>; > > + samsung,dynamic-range = <0>; > > + samsung,ycbcr-coeff = <0>; > > + > > + ports { > > + port { > > + dp_out: endpoint { > > + remote-endpoint = <&panel>; > > + }; > > + }; > > + }; > > +}; > > + > > &ehci { > > status = "disabled"; > > }; > > > > +&fimd { > > + status = "okay"; > > + > > + assigned-clocks = <&clock CLK_MOUT_FIMD1>, <&clock CLK_MOUT_VPLL>; > > + assigned-clock-parents = <&clock CLK_MOUT_VPLL>; > > +}; > > + > > &i2c_1 { > > status = "okay"; > > > > @@ -396,6 +467,12 @@ ldo25_reg: LDO25 { > > }; > > }; > > > > +&mali { > > + status = "okay"; > > + > > + mali-supply = <&buck4_reg>; > > +}; > > + > > &mixer { > > status = "okay"; > > }; > > @@ -489,6 +566,14 @@ wlan_irq: wlan-irq-pins { > > }; > > > > &pinctrl_1 { > > + aps_en_18v: aps-en-18v-pins { > > + samsung,pins = "gph1-6"; > > + samsung,pin-function = ; /* TODO */ > > TODO why? IOW, what are you missing here? Was this tested? > > TODO note can stay, but I would prefer to see short comment explaining > why. > > > Best regards, > Krzysztof > > Yes, this patch was tested for function. I added reminders to triple check these values before sending the patch but obviously forgot to remove them afterwards. They won't be in v2 of this patch. Also thanks for commenting on gpio and voltage, I try to search in the downstream kernel to find values or add comments when there is no information. Best regards Lukas