mbox series

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

Message ID 20220422173032.2259019-1-sean.anderson@seco.com
Headers show
Series net: fm: Add support for loading firmware from filesystem | expand

Message

Sean Anderson April 22, 2022, 5:30 p.m. UTC
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(-)

Comments

Sean Anderson Aug. 13, 2022, 6:15 a.m. UTC | #1
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
Ramon Fried Aug. 14, 2022, 8:48 p.m. UTC | #2
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.
Sean Anderson Nov. 17, 2022, 9:29 p.m. UTC | #3
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.

--Sean
Ramon Fried Nov. 26, 2022, 10:47 p.m. UTC | #4
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 ?
>
> --Sean
Tom Rini Nov. 28, 2022, 3:03 p.m. UTC | #5
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?