Re: [RESEND PATCH v2 0/3] net: fm: Add support for loading firmware from filesystem

From: Tom Rini <trini@konsulko.com>
To: Ramon Fried <rfried.dev@gmail.com>, Marek Vasut <marex@denx.de>
Cc: Sean Anderson <sean.anderson@seco.com>,
	U-Boot Mailing List <u-boot@lists.denx.de>,
	Joe Hershberger <joe.hershberger@ni.com>,
	Priyanka Jain <priyanka.jain@nxp.com>,
	York Sun <york.sun@nxp.com>,
	Michal Simek <michal.simek@xilinx.com>,
	Patrick Delaunay <patrick.delaunay@foss.st.com>,
	Simon Glass <sjg@chromium.org>
Subject: Re: [RESEND PATCH v2 0/3] net: fm: Add support for loading firmware from filesystem
Date: Mon, 28 Nov 2022 10:03:37 -0500	[thread overview]
Message-ID: <20221128150337.GR3787616@bill-the-cat> (raw)
In-Reply-To: <CAGi-RUKbWPa+NEWm7XhOK2KCbTt3bHABU_UFQHUBDMD6++2ChA@mail.gmail.com>

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

On Sun, Nov 27, 2022 at 12:47:37AM +0200, Ramon Fried wrote:
> On Thu, Nov 17, 2022 at 11:29 PM Sean Anderson <sean.anderson@seco.com> wrote:
> >
> > On 8/14/22 16:48, Ramon Fried wrote:
> > > On Sat, Aug 13, 2022 at 9:15 AM Sean Anderson <sean.anderson@seco.com> wrote:
> > >>
> > >> On 4/22/22 1:30 PM, Sean Anderson wrote:
> > >> > This adds support for loading Fman firmware from a filesystem using the
> > >> > firmware loader subsystem. It was originally part of [1], but has been
> > >> > split off because it is conceptually separate.
> > >> >
> > >> > [1] https://lore.kernel.org/u-boot/20220324182306.2037094-1-sean.anderson@seco.com/
> > >> >
> > >> > Changes in v2:
> > >> > - Split series into two
> > >> >
> > >> > Sean Anderson (3):
> > >> >    misc: fs_loader: Add function to get the chosen loader
> > >> >    net: fm: Add firmware name parameter
> > >> >    net: fm: Support loading firmware from a filesystem
> > >> >
> > >> >   arch/arm/mach-k3/common.c         |  2 +-
> > >> >   arch/arm/mach-omap2/boot-common.c |  2 +-
> > >> >   drivers/fpga/socfpga_arria10.c    | 24 ++-----------------
> > >> >   drivers/misc/fs_loader.c          | 27 +++++++++++++++++++++
> > >> >   drivers/net/fm/fm.c               | 40 +++++++++++++++++++++++++++----
> > >> >   drivers/net/fm/fm.h               |  2 +-
> > >> >   drivers/net/fm/init.c             |  4 ++--
> > >> >   drivers/qe/Kconfig                |  4 ++++
> > >> >   include/fs_loader.h               | 12 ++++++++++
> > >> >   9 files changed, 86 insertions(+), 31 deletions(-)
> > >> >
> > >>
> > >> ping
> > > Pong, sorry, was lost in the mailbox.
> >
> > ping again
> >
> > Looks like this got reviewed but never applied. Does it still apply
> > cleanly? If not I can rebase and resend.
> Tom ?

I don't see a problem with this. I see in patchwork it's assigned to
Marek and "Under Review", which I don't recall doing, so, are you
looking at these currently Marek?

-- 
Tom

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