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

[Tom Rini <trini@konsulko.com>]

[-- Attachment #1: Type: text/plain, Size: 1479 bytes --]

On Tue, Apr 15, 2025 at 03:46:32PM -0700, Sukrut Bellary wrote:
> 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

Yes, that's fine, thanks.

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]