On Thu, Nov 05, 2026 at 08:10:26PM +0100, Marek Vasut wrote:
> On 11/5/26 7:51 PM, Massimo Pegorer wrote:
> > Hi,
> > 
> > Il giorno gio 5 nov 2026 alle ore 15:42 Tom Rini <[email protected]>
> > ha scritto:
> > > 
> > > On Wed, Nov 04, 2026 at 06:33:21PM +0100, Marek Vasut wrote:
> > > > On 11/4/26 3:29 PM, Tom Rini wrote:
> > > > > On Mon, Oct 26, 2026 at 11:11:32PM +0100, Marek Vasut wrote:
> > > > > 
> > > > > > Add conditional around REMOTEPROC menu entries, turn REMOTEPROC into
> > > > > > configurable boolean, and update all defconfigs and Kconfigs which
> > > > > > previously only selected REMOTEPROC driver to also select REMOTEPROC
> > > > > > itself.
> > > > > > 
> > > > > > Signed-off-by: Marek Vasut <[email protected]>
> > > > > > ---
> > > > > > Cc: Massimo Pegorer <[email protected]>
> > > > > > Cc: MD Danish Anwar <[email protected]>
> > > > > > Cc: Tom Rini <[email protected]>
> > > > > > Cc: [email protected]
> > > > > 
> > > > > Unfortunately this misses some handful of at least TI K3 platforms. I
> > > > > think using qconfig.py to confirm which configs have REMOTEPROC set
> > > > > before / after, and that they match would make sorting that part of 
> > > > > the
> > > > > problem out easier. Do you have time to do that, or should I pick it 
> > > > > up
> > > > > and do a v2? Thanks.
> > > > I can respin this, but do you know which platforms got missed ?
> > > 
> > > I lost the list, sorry. It was a number of the r5 builds. Perhaps some
> > > fragment got missed? But qconfig.py -f REMOTEPROC, with a db built with
> > > and without the patch will quickly show which oens it is.
> > 
> > This is the result of comparing the two qoconfig.py outputs:
> 
> u-boot$ git log --oneline HEAD~4..
> 1e29311663e (HEAD -> master) Revert "remoteproc: Hide REMOTEPROC options in
> REMOTEPROC conditional"
> 97162e533ca remoteproc: Hide REMOTEPROC options in REMOTEPROC conditional
> df3b8e16e6d remoteproc: Make REMOTEPROC_MAX_FW_SIZE depend on REMOTEPROC
> d1d45c926a5 (origin/main) Merge branch 'main' of
> https://git.u-boot-project.org/u-boot/custodians/u-boot-riscv

Wait, your new patch didn't just replace the old patch?

-- 
Tom

Attachment: signature.asc
Description: PGP signature

Reply via email to