Re: [PATCH v7 03/24] wfx: add Makefile/Kconfig

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller > --- > drivers/net/wireless/silabs/wfx/Kconfig | 12 +++ > drivers/net/wireless/silabs/wfx/Makefile | 26 > 2 files changed, 38 insertions(+) > create mode 100644 drivers/net/

Re: [PATCH v7 03/24] wfx: add Makefile/Kconfig

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > --- /dev/null > +++ b/drivers/net/wireless/silabs/wfx/Makefile > @@ -0,0 +1,26 @@ > +# SPDX-License-Identifier: GPL-2.0 All other files use GPL-2.0-only: bh.c:// SPDX-License-Identifier: GPL-2.0-only

Re: [PATCH v7 05/24] wfx: add main.c/main.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +/* The device needs data about the antenna configuration. This information in > + * provided by PDS (Platform Data Set, this is the wording used in WF200 > + * documentation) files. For hardware integr

Re: [PATCH v7 20/24] wfx: add scan.c/scan.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +/* It is not really necessary to run scan request asynchronously. However, > + * there is a bug in "iw scan" when ieee80211_scan_completed() is called > before > + * wfx_hw_scan() return > + */ > +voi

Re: [PATCH v7 05/24] wfx: add main.c/main.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 11:22:08 CEST Kalle Valo wrote: > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > > > Signed-off-by: Jérôme Pouiller > > [...] > > > +/* The device needs data about the antenna configuration. This information > > in > > + * provided by PDS (Platform Data Set,

Re: [PATCH v7 09/24] wfx: add hwio.c/hwio.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller > --- > drivers/net/wireless/silabs/wfx/hwio.c | 340 + > drivers/net/wireless/silabs/wfx/hwio.h | 79 ++ [...] > +static int indirect_read(struct wfx_dev *wdev, int reg, u32 addr,

Re: [PATCH v7 13/24] wfx: add hif_tx*.c/hif_tx*.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > --- /dev/null > +++ b/drivers/net/wireless/silabs/wfx/hif_tx_mib.h > @@ -0,0 +1,49 @@ > +/* SPDX-License-Identifier: GPL-2.0-only */ > +/* > + * Implementation of the host-to-chip MIBs of the hardware A

Re: [PATCH v7 15/24] wfx: add hif_rx.c/hif_rx.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +static int hif_startup_indication(struct wfx_dev *wdev, > + const struct hif_msg *hif, const void *buf) > +{ > + const struct hif_ind_startup *body = buf; > + > +

Re: [PATCH v7 12/24] wfx: add hif_api_*.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > --- /dev/null > +++ b/drivers/net/wireless/silabs/wfx/hif_api_cmd.h > @@ -0,0 +1,555 @@ > +/* SPDX-License-Identifier: Apache-2.0 */ I don't how I missed this earlier: hif_api_cmd.h:/* SPDX-License-Id

Re: [PATCH v7 12/24] wfx: add hif_api_*.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +#define HIF_STATUS_SUCCESS (cpu_to_le32(0x)) > +#define HIF_STATUS_FAIL(cpu_to_le32(0x0001)) > +#define HIF_STATUS_INVALID_PARAMETER

Re: [PATCH v7 12/24] wfx: add hif_api_*.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > --- /dev/null > +++ b/drivers/net/wireless/silabs/wfx/hif_api_general.h > @@ -0,0 +1,256 @@ > +/* SPDX-License-Identifier: Apache-2.0 */ > +/* > + * WF200 hardware interface definitions > + * > + * Copy

Re: [PATCH v7 12/24] wfx: add hif_api_*.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 13:41:55 CEST Kalle Valo wrote: > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > > > Signed-off-by: Jérôme Pouiller > > [...] > > > --- /dev/null > > +++ b/drivers/net/wireless/silabs/wfx/hif_api_cmd.h > > @@ -0,0 +1,555 @@ > > +/* SPDX-License-Identifier: Ap

Re: [PATCH v7 10/24] wfx: add fwio.c/fwio.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +static int get_firmware(struct wfx_dev *wdev, u32 keyset_chip, > + const struct firmware **fw, int *file_offset) > +{ > + int keyset_file; > + char filename[256]; > + co

Re: [PATCH v7 21/24] wfx: add debug.c/debug.h

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Signed-off-by: Jérôme Pouiller [...] > +static int wfx_ps_timeout_set(void *data, u64 val) > +{ > + struct wfx_dev *wdev = (struct wfx_dev *)data; > + struct wfx_vif *wvif; > + > + wdev->force_ps_timeout = val; > + wvif = NUL

Re: [PATCH v7 05/24] wfx: add main.c/main.h

2021-10-01 Thread Kalle Valo
Jérôme Pouiller writes: > On Friday 1 October 2021 11:22:08 CEST Kalle Valo wrote: >> Jerome Pouiller writes: >> >> > From: Jérôme Pouiller >> > >> > Signed-off-by: Jérôme Pouiller >> >> [...] >> >> > +/* The device needs data about the antenna configuration. This >> > information in >> >

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

2021-10-01 Thread Kalle Valo
Hi Ulf, sorry for the late reply, my Gnus tells me it took me 24 weeks to reply :) Ulf Hansson writes: > On Wed, 7 Apr 2021 at 14:00, Kalle Valo wrote: >> >> Ulf Hansson writes: >> >> >> If I follow what has been done in other drivers I would write something >> >> like: >> >> >> >> static i

Re: [PATCH v7 09/24] wfx: add hwio.c/hwio.h

2021-10-01 Thread Kalle Valo
Kalle Valo writes: >> --- /dev/null >> +++ b/drivers/net/wireless/silabs/wfx/hwio.h >> @@ -0,0 +1,79 @@ >> +/* SPDX-License-Identifier: GPL-2.0-only */ >> +/* >> + * Low-level I/O functions. >> + * >> + * Copyright (c) 2017-2020, Silicon Laboratories, Inc. >> + * Copyright (c) 2010, ST-Ericsson >

Re: [PATCH v7 00/24] wfx: get out from the staging area

2021-10-01 Thread Kalle Valo
Jerome Pouiller writes: > From: Jérôme Pouiller > > Hello, > > I think the wfx driver is now mature enough to be accepted in the > drivers/net/wireless directory. > > The firmware is now a part of the linux-firmware repository since relase > 20210315[1]. It had taken a bit of time because I have

Re: [PATCH v7 12/24] wfx: add hif_api_*.h

2021-10-01 Thread Kalle Valo
Jérôme Pouiller writes: > On Friday 1 October 2021 13:41:55 CEST Kalle Valo wrote: >> Jerome Pouiller writes: >> >> > From: Jérôme Pouiller >> > >> > Signed-off-by: Jérôme Pouiller >> >> [...] >> >> > --- /dev/null >> > +++ b/drivers/net/wireless/silabs/wfx/hif_api_cmd.h >> > @@ -0,0 +1,555

Re: [PATCH] binder: use cred instead of task for selinux checks

2021-10-01 Thread Paul Moore
On Thu, Sep 30, 2021 at 10:45 PM Todd Kjos wrote: > > Save the struct cred associated with a binder process > at initial open to avoid potential race conditions > when converting to a security ID. > > Since binder was integrated with selinux, it has passed > 'struct task_struct' associated with th

Re: [PATCH] binder: use cred instead of task for selinux checks

2021-10-01 Thread Paul Moore
On Fri, Oct 1, 2021 at 10:38 AM Paul Moore wrote: > On Thu, Sep 30, 2021 at 10:45 PM Todd Kjos wrote: > > > > Save the struct cred associated with a binder process > > at initial open to avoid potential race conditions > > when converting to a security ID. > > > > Since binder was integrated with

Re: [PATCH v7 10/24] wfx: add fwio.c/fwio.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 13:58:38 CEST Kalle Valo wrote: > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > > > Signed-off-by: Jérôme Pouiller > > [...] > > > +static int get_firmware(struct wfx_dev *wdev, u32 keyset_chip, > > + const struct firmware **fw, int *file

Re: [PATCH v7 13/24] wfx: add hif_tx*.c/hif_tx*.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 11:55:33 CEST Kalle Valo 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. > > > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > >

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

2021-10-01 Thread Ulf Hansson
On Thu, 30 Sept 2021 at 19:06, Pali Rohár wrote: > > On Thursday 30 September 2021 18:51:09 Jérôme Pouiller wrote: > > Hello Ulf, > > > > On Thursday 30 September 2021 12:07:55 CEST Ulf Hansson wrote: > > > On Mon, 20 Sept 2021 at 18:12, Jerome Pouiller > > > wrote: > > > > > > > > From: Jérôme P

Re: [PATCH v7 05/24] wfx: add main.c/main.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 11:22:08 CEST Kalle Valo 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. > > > Jerome Pouiller writes: > > > From: Jérôme Pouiller > > >

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

2021-10-01 Thread Ulf Hansson
On Thu, 30 Sept 2021 at 18:51, Jérôme Pouiller wrote: > > Hello Ulf, > > On Thursday 30 September 2021 12:07:55 CEST Ulf Hansson wrote: > > On Mon, 20 Sept 2021 at 18:12, Jerome Pouiller > > wrote: > > > > > > From: Jérôme Pouiller > > > > > > Signed-off-by: Jérôme Pouiller > > > --- > > > dri

Re: [PATCH v7 10/24] wfx: add fwio.c/fwio.h

2021-10-01 Thread Pali Rohár
On Friday 01 October 2021 17:09:41 Jérôme Pouiller wrote: > On Friday 1 October 2021 13:58:38 CEST Kalle Valo wrote: > > Jerome Pouiller writes: > > > > > From: Jérôme Pouiller > > > > > > Signed-off-by: Jérôme Pouiller > > > > [...] > > > > > +static int get_firmware(struct wfx_dev *wdev, u3

Re: [PATCH v7 13/24] wfx: add hif_tx*.c/hif_tx*.h

2021-10-01 Thread Pali Rohár
On Friday 01 October 2021 17:17:52 Jérôme Pouiller wrote: > On Friday 1 October 2021 11:55:33 CEST Kalle Valo 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. >

Re: [PATCH v7 10/24] wfx: add fwio.c/fwio.h

2021-10-01 Thread Jérôme Pouiller
On Friday 1 October 2021 18:08:32 CEST Pali Rohár wrote: > On Friday 01 October 2021 17:09:41 Jérôme Pouiller wrote: > > On Friday 1 October 2021 13:58:38 CEST Kalle Valo wrote: > > > Jerome Pouiller writes: > > > > > > > From: Jérôme Pouiller > > > > > > > > Signed-off-by: Jérôme Pouiller > > >

Re: [PATCH v2] binder: use cred instead of task for selinux checks

2021-10-01 Thread Casey Schaufler
On 10/1/2021 10:55 AM, Todd Kjos wrote: > Save the struct cred associated with a binder process > at initial open to avoid potential race conditions > when converting to a security ID. > > Since binder was integrated with selinux, it has passed > 'struct task_struct' associated with the binder_proc

Re: [PATCH v2] binder: use cred instead of task for selinux checks

2021-10-01 Thread Casey Schaufler
On 10/1/2021 12:50 PM, Jann Horn wrote: > On Fri, Oct 1, 2021 at 9:36 PM Jann Horn wrote: >> On Fri, Oct 1, 2021 at 8:46 PM Casey Schaufler >> wrote: >>> On 10/1/2021 10:55 AM, Todd Kjos wrote: Save the struct cred associated with a binder process at initial open to avoid potential rac

Re: [PATCH] binder: use cred instead of task for selinux checks

2021-10-01 Thread Todd Kjos
On Fri, Oct 1, 2021 at 7:38 AM Paul Moore wrote: > > On Thu, Sep 30, 2021 at 10:45 PM Todd Kjos wrote: > > > > Save the struct cred associated with a binder process > > at initial open to avoid potential race conditions > > when converting to a security ID. > > > > Since binder was integrated wit

Re:Investment Invitation

2021-10-01 Thread Luis Fernandez
Good day, I have an investment proposal that I would like to discuss with you. I await your urgent response so that I will explain more regarding this awesome Investment Opportunity. Regards, Luis Fernandez ___ devel mailing list de...@linuxdriverpro

[PATCH v2] binder: use cred instead of task for selinux checks

2021-10-01 Thread Todd Kjos
Save the struct cred associated with a binder process at initial open to avoid potential race conditions when converting to a security ID. Since binder was integrated with selinux, it has passed 'struct task_struct' associated with the binder_proc to represent the source and target of transactions