Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-23 Thread Luis Chamberlain
On Wed, May 24, 2023 at 03:35:41PM +1000, David Airlie wrote: > On Wed, May 24, 2023 at 3:26 PM Luis Chamberlain wrote: > > > > Hey Dave, just curious if there was going to be another follow up patch > > for this or if it was already posted. I don't see it clearly so just > > wanted to double chec

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-23 Thread David Airlie
On Wed, May 24, 2023 at 3:26 PM Luis Chamberlain wrote: > > On Wed, May 03, 2023 at 01:19:31PM +1000, Dave Airlie wrote: > > > > > > > > > > >> > the GROUP until after the FIRMWARE, so this can't work, as it already > > > >> > will have included all the ones below, hence why I bracketed top and >

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-23 Thread Luis Chamberlain
On Wed, May 03, 2023 at 01:19:31PM +1000, Dave Airlie wrote: > > > > > > > >> > the GROUP until after the FIRMWARE, so this can't work, as it already > > >> > will have included all the ones below, hence why I bracketed top and > > >> > bottom with a group. > > >> > > >> well... that is something t

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-02 Thread Dave Airlie
> > > > >> > the GROUP until after the FIRMWARE, so this can't work, as it already > >> > will have included all the ones below, hence why I bracketed top and > >> > bottom with a group. > >> > >> well... that is something that can be adapted easily by using a 2 pass > >> approach, filtering out th

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-02 Thread Luis Chamberlain
On Tue, May 02, 2023 at 11:11:58AM -0700, Lucas De Marchi wrote: > Based on the above and my previous reply, I think we should have > something more explicit about the order rather than relying on the > toolchain behavior. You can open code ELF sections and provide SORT() but you can also use help

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-05-02 Thread Lucas De Marchi
On Mon, Apr 24, 2023 at 03:56:53PM -0700, Luis Chamberlain wrote: On Mon, Apr 24, 2023 at 10:01:13AM -0700, Lucas De Marchi wrote: On Mon, Apr 24, 2023 at 03:44:18PM +1000, Dave Airlie wrote: > On Fri, 21 Apr 2023 at 05:09, Lucas De Marchi wrote: > > > > On Wed, Apr 19, 2023 at 02:36:52PM +100

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-04-24 Thread Luis Chamberlain
On Mon, Apr 24, 2023 at 10:01:13AM -0700, Lucas De Marchi wrote: > On Mon, Apr 24, 2023 at 03:44:18PM +1000, Dave Airlie wrote: > > On Fri, 21 Apr 2023 at 05:09, Lucas De Marchi > > wrote: > > > > > > On Wed, Apr 19, 2023 at 02:36:52PM +1000, Dave Airlie wrote: > > > >From: Dave Airlie > > > >

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-04-24 Thread Lucas De Marchi
On Mon, Apr 24, 2023 at 03:44:18PM +1000, Dave Airlie wrote: On Fri, 21 Apr 2023 at 05:09, Lucas De Marchi wrote: On Wed, Apr 19, 2023 at 02:36:52PM +1000, Dave Airlie wrote: >From: Dave Airlie > >This adds a tag that will go into the module info, only one firmware from >the group given needs

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-04-23 Thread Dave Airlie
On Fri, 21 Apr 2023 at 05:09, Lucas De Marchi wrote: > > On Wed, Apr 19, 2023 at 02:36:52PM +1000, Dave Airlie wrote: > >From: Dave Airlie > > > >This adds a tag that will go into the module info, only one firmware from > >the group given needs to be available for this driver to work. This allows

Re: [PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-04-20 Thread Lucas De Marchi
On Wed, Apr 19, 2023 at 02:36:52PM +1000, Dave Airlie wrote: From: Dave Airlie This adds a tag that will go into the module info, only one firmware from the group given needs to be available for this driver to work. This allows dracut to avoid adding in firmware that aren't needed. This just b

[PATCH] modules/firmware: add a new option to denote a firmware group to choose one.

2023-04-18 Thread Dave Airlie
From: Dave Airlie This adds a tag that will go into the module info, only one firmware from the group given needs to be available for this driver to work. This allows dracut to avoid adding in firmware that aren't needed. This just brackets a module list in the modinfo, the modules in the list w