Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-14 Thread Kalle Valo
matthias@kernel.org wrote: > From: Matthias Brugger > > Apart from a firmware binary the chip needs a config file used by the > FW. Add the config files to modinfo so that they can be read by > userspace. > > Signed-off-by: Matthias Brugger I agree with Hans, this does not look a good sol

Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-03 Thread Hans de Goede
Hi, On 7/3/20 4:01 PM, Matthias Brugger wrote: On 02/07/2020 20:00, Hans de Goede wrote: Hi, On 7/1/20 5:46 PM, Matthias Brugger wrote: Hi Hans, On 01/07/2020 17:38, Hans de Goede wrote: Hi, On 7/1/20 5:31 PM, matthias@kernel.org wrote: From: Matthias Brugger Apart from a firmware

Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-03 Thread Matthias Brugger
On 02/07/2020 20:00, Hans de Goede wrote: > Hi, > > On 7/1/20 5:46 PM, Matthias Brugger wrote: >> Hi Hans, >> >> On 01/07/2020 17:38, Hans de Goede wrote: >>> Hi, >>> >>> On 7/1/20 5:31 PM, matthias@kernel.org wrote: From: Matthias Brugger Apart from a firmware binary the ch

Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-02 Thread Hans de Goede
Hi, On 7/1/20 5:46 PM, Matthias Brugger wrote: Hi Hans, On 01/07/2020 17:38, Hans de Goede wrote: Hi, On 7/1/20 5:31 PM, matthias@kernel.org wrote: From: Matthias Brugger Apart from a firmware binary the chip needs a config file used by the FW. Add the config files to modinfo so that t

Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-01 Thread Matthias Brugger
Hi Hans, On 01/07/2020 17:38, Hans de Goede wrote: > Hi, > > On 7/1/20 5:31 PM, matthias@kernel.org wrote: >> From: Matthias Brugger >> >> Apart from a firmware binary the chip needs a config file used by the >> FW. Add the config files to modinfo so that they can be read by >> userspace. >

Re: [PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-01 Thread Hans de Goede
Hi, On 7/1/20 5:31 PM, matthias@kernel.org wrote: From: Matthias Brugger Apart from a firmware binary the chip needs a config file used by the FW. Add the config files to modinfo so that they can be read by userspace. The configfile firmware filename is dynamically generated, just adding

[PATCH] brcmfmac: expose firmware config files through modinfo

2020-07-01 Thread matthias . bgg
From: Matthias Brugger Apart from a firmware binary the chip needs a config file used by the FW. Add the config files to modinfo so that they can be read by userspace. Signed-off-by: Matthias Brugger --- .../wireless/broadcom/brcm80211/brcmfmac/sdio.c | 16 1 file changed,