Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Jérôme Pouiller
On Wednesday 12 January 2022 18:48:48 CET Pali Rohár wrote: > CAUTION: This email originated from outside of the organization. Do not click > links or open attachments unless you recognize the sender and know the > content is safe. > > > On Wednesday 12 January 2022 17:45:45 Jérôme Pouiller

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Pali Rohár
On Wednesday 12 January 2022 17:45:45 Jérôme Pouiller wrote: > On Wednesday 12 January 2022 12:43:32 CET Pali Rohár wrote: > > > > On Wednesday 12 January 2022 12:18:58 Jérôme Pouiller wrote: > > > On Wednesday 12 January 2022 11:58:59 CET Pali Rohár wrote: > > > > On Tuesday 11 January 2022

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Jérôme Pouiller
On Wednesday 12 January 2022 12:43:32 CET Pali Rohár wrote: > > On Wednesday 12 January 2022 12:18:58 Jérôme Pouiller wrote: > > On Wednesday 12 January 2022 11:58:59 CET Pali Rohár wrote: > > > On Tuesday 11 January 2022 18:14:08 Jerome Pouiller wrote: > > > > +static const struct sdio_device_id

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Ulf Hansson
On Wed, 12 Jan 2022 at 12:43, Pali Rohár wrote: > > On Wednesday 12 January 2022 12:18:58 Jérôme Pouiller wrote: > > On Wednesday 12 January 2022 11:58:59 CET Pali Rohár wrote: > > > On Tuesday 11 January 2022 18:14:08 Jerome Pouiller wrote: > > > > +static const struct sdio_device_id

[driver-core:module_ns] BUILD SUCCESS 132187dccc9b217ffb129254f07e90fd30375a83

2022-01-12 Thread kernel test robot
tree/branch: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/driver-core.git module_ns branch HEAD: 132187dccc9b217ffb129254f07e90fd30375a83 module.h: allow #define strings to work with MODULE_IMPORT_NS elapsed time: 5120m configs tested: 146 configs skipped: 3 The following configs

Re: [PATCH v9 23/24] wfx: remove from the staging area

2022-01-12 Thread Kalle Valo
Jérôme Pouiller writes: > On Wednesday 12 January 2022 08:49:54 CET Kalle Valo wrote: >> Jerome Pouiller writes: >> >> > From: Jérôme Pouiller >> > >> > Signed-off-by: Jérôme Pouiller >> > --- >> > .../bindings/net/wireless/silabs,wfx.yaml | 125 --- >> > drivers/staging/wfx/Kconfig

Re: [PATCH v9 01/24] mmc: sdio: add SDIO IDs for Silabs WF200 chip

2022-01-12 Thread Kalle Valo
Ulf Hansson writes: > On Tue, 11 Jan 2022 at 18:14, Jerome Pouiller > wrote: >> >> From: Jérôme Pouiller >> >> Note that the values used by Silabs are uncommon. A driver cannot fully >> rely on the SDIO PnP. It should also check if the device is declared in >> the DT. >> >> So, to apply the

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Pali Rohár
On Wednesday 12 January 2022 13:06:17 Greg Kroah-Hartman wrote: > On Wed, Jan 12, 2022 at 12:43:32PM +0100, Pali Rohár wrote: > > Btw, is there any project which maintains SDIO ids, like there is > > pci-ids.ucw.cz for PCI or www.linux-usb.org/usb-ids.html for USB? > > Both of those projects have

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Greg Kroah-Hartman
On Wed, Jan 12, 2022 at 12:43:32PM +0100, Pali Rohár wrote: > Btw, is there any project which maintains SDIO ids, like there is > pci-ids.ucw.cz for PCI or www.linux-usb.org/usb-ids.html for USB? Both of those projects have nothing to do with the kernel drivers or values at all, they are only for

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Pali Rohár
On Wednesday 12 January 2022 12:18:58 Jérôme Pouiller wrote: > On Wednesday 12 January 2022 11:58:59 CET Pali Rohár wrote: > > On Tuesday 11 January 2022 18:14:08 Jerome Pouiller wrote: > > > +static const struct sdio_device_id wfx_sdio_ids[] = { > > > + { SDIO_DEVICE(SDIO_VENDOR_ID_SILABS,

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Jérôme Pouiller
On Wednesday 12 January 2022 11:58:59 CET Pali Rohár wrote: > On Tuesday 11 January 2022 18:14:08 Jerome Pouiller wrote: > > +static const struct sdio_device_id wfx_sdio_ids[] = { > > + { SDIO_DEVICE(SDIO_VENDOR_ID_SILABS, SDIO_DEVICE_ID_SILABS_WF200) }, > > + { }, > > +}; > > Hello! Is

Re: [PATCH v9 01/24] mmc: sdio: add SDIO IDs for Silabs WF200 chip

2022-01-12 Thread Ulf Hansson
On Tue, 11 Jan 2022 at 18:14, Jerome Pouiller wrote: > > From: Jérôme Pouiller > > Note that the values used by Silabs are uncommon. A driver cannot fully > rely on the SDIO PnP. It should also check if the device is declared in > the DT. > > So, to apply the quirks necessary for the Silabs

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Pali Rohár
On Tuesday 11 January 2022 18:14:08 Jerome Pouiller wrote: > +static const struct sdio_device_id wfx_sdio_ids[] = { > + { SDIO_DEVICE(SDIO_VENDOR_ID_SILABS, SDIO_DEVICE_ID_SILABS_WF200) }, > + { }, > +}; Hello! Is this table still required? > +MODULE_DEVICE_TABLE(sdio, wfx_sdio_ids); > +

Re: [PATCH v9 08/24] wfx: add bus_sdio.c

2022-01-12 Thread Ulf Hansson
[...] > +static const struct of_device_id wfx_sdio_of_match[] = { > + { .compatible = "silabs,wf200",.data = _wf200 }, > + { .compatible = "silabs,brd4001a", .data = _brd4001a }, > + { .compatible = "silabs,brd8022a", .data = _brd8022a }, > + { .compatible =

Re: [PATCH v9 02/24] dt-bindings: introduce silabs,wfx.yaml

2022-01-12 Thread Jérôme Pouiller
On Tuesday 11 January 2022 23:19:56 CET Rob Herring wrote: > On Tue, Jan 11, 2022 at 06:14:02PM +0100, Jerome Pouiller wrote: > > From: Jérôme Pouiller > > > > Prepare the inclusion of the wfx driver in the kernel. > > > > Signed-off-by: Jérôme Pouiller > > --- > >

Re: [PATCH v9 23/24] wfx: remove from the staging area

2022-01-12 Thread Jérôme Pouiller
On Wednesday 12 January 2022 08:49:54 CET Kalle Valo wrote: > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > > > Signed-off-by: Jérôme Pouiller > > --- > > .../bindings/net/wireless/silabs,wfx.yaml | 125 --- > > drivers/staging/wfx/Kconfig | 8 - > >