Re: [PATCH v4 0/2] Refactor tidss_drv and enable early splash on AM625

[Tom Rini <trini@konsulko.com>]

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

On Tue, Nov 11, 2025 at 02:38:10PM +0530, Swamil Jain wrote:
> 
> On 06-11-2025 19:07, Tom Rini wrote:
> > On Thu, Nov 06, 2025 at 05:56:38PM +0530, Anshul Dalal wrote:
> > > On Thu Nov 6, 2025 at 2:28 PM IST, Swamil Jain wrote:
> > > > This series:
> > > > - Refactors tidss_drv
> > > > - Adds am62x_a53_splashscreen.config to enable splashscreen at u-boot
> > > >    SPL
> > > > 
> > > > Signed-off-by: Swamil Jain <s-jain1@ti.com>
> > > > ---
> > > 
> > > Tested-by: Anshul Dalal <anshuld@ti.com>
> > > 
> > > I also went ahead and did a sandbox build and the driver seems to build
> > > fine contrary to what Tom saw[1].
> > > 
> > > Below are the steps I followed, am I missing something here?
> > >   1. make sandbox_defconfig
> > >   2. Select CONFIG_VIDEO_TIDSS via menuconfig
> > >   3. make -j`nproc`
> > > 
> > > [1]: https://lore.kernel.org/u-boot/20251105164114.GQ6688@bill-the-cat/
> > 
> > v4 drops the header in question :)
> > 
> 
> Hi Tom, as we have the patches tested and sandbox builds also fine, does the
> series look good, could it be possible to pick it?

I've assigned it to the video custodian as yes it looks generally fine,
thanks.

-- 
Tom

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