Re: [PATCH 3/3] arm: dts: am335x: Add nodes for lcdc, panel and backlight

[Sukrut Bellary <sbellary@baylibre.com>]

On Tue, Mar 25, 2025 at 12:31:06PM -0700, Sukrut Bellary wrote:
> On Mon, Mar 24, 2025 at 11:20:09AM -0600, Tom Rini wrote:
> > On Wed, Mar 19, 2025 at 01:25:16PM -0700, Sukrut Bellary wrote:
> > 
> > > For AM335x EVM[1], enable nodes required to enable the support for LCD
> > > and PWM backlight.
> > > This is required for the splash screen support.
> > > 
> > > [1] AM335x EVM - https://www.ti.com/tool/TMDXEVM3358
> > > 
> > > Signed-off-by: Sukrut Bellary <sbellary@baylibre.com>
> > > ---
> > >  arch/arm/dts/am335x-evm.dts | 15 ++++++++++++++-
> > >  1 file changed, 14 insertions(+), 1 deletion(-)
> > 
> > Can you see how hard moving to OF_UPSTREAM here is first please? Thanks.
> >
> 
> Sure, I will check.
> Thanks.

I checked OF_UPSTREAM for AM335x.
For the panel driver, timings are fetched differently in the u-boot and kernel.
For OF_UPSTREAM, we need to change the lcdc panel driver in u-boot to be in
sync with kernel dts or see if we can reuse the simple_panel driver.

We plan to take care of OF_UPSTREAM later.
So, For now, can we split this series into two as
1. Enable Auxillary PWM using eCAP and pin muxing(These are independent
of OF_UPSTREAM)
2. Handle dts changes once we have OF_UPSTREAM for AM33xx.
If this is ok, I will post the patches per pt.1

Thanks.

> > -- 
> > Tom
> 
>