t; ---
>> drivers/staging/rtlwifi/wifi.h | 4
>
> Presumably the equivalent uses in
> drivers/net/wireless/realtek/rtlwifi/wifi.h
> should be removed as well.
But if it's needed, just do it in separate patch as the patches go via
different trees.
--
Kalle Valo
_
Ajay Singh writes:
> Hi Kalle,
>
> On Thu, 23 Aug 2018 16:37:45 +0530
> Kalle Valo wrote:
>
>> Ajay Singh writes:
>>
>> >> >>> We need help to review and identify if there are any pending
>> >> >>> items for wilc1000 drive
olks might take some time.
>>
>
> Thanks for the update.
> Btw I could see our driver was added to the pending branch. Just
> curious, could you please share details about how the pending branch is
> used ?
I use pending branch mainly for testing bigger patches with kbu
Ulf Hansson writes:
>> If I follow what has been done in other drivers I would write something
>> like:
>>
>> static int wfx_sdio_suspend(struct device *dev)
>> {
>> struct sdio_func *func = dev_to_sdio_func(dev);
>> struct wfx_sdio_priv *bus = sdio_get_drvdata(func);
>>
>
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/
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
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
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
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,
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
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;
> +
> +
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
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
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
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
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
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
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
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 Laborator
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
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
Jérôme Pouiller writes:
> 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.
>>
>
Ulf Hansson writes:
>> > > +static const struct sdio_device_id wfx_sdio_ids[] = {
>> > > + { SDIO_DEVICE(SDIO_VENDOR_ID_SILABS,
>> > > SDIO_DEVICE_ID_SILABS_WF200) },
>> > > + { },
>> > > +};
>> > > +MODULE_DEVICE_TABLE(sdio, wfx_sdio_ids);
>> > > +
>> > > +struct sdio_driver wfx_sdi
Pali Rohár writes:
> 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 re
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
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
Jérôme Pouiller writes:
> On Tuesday 5 October 2021 16:15:22 CEST Kalle Valo wrote:
>> Jerome Pouiller writes:
>>
>> > From: Jérôme Pouiller
> [...]
>> > v8:
>> > - Change the way the DT is handled. The user can now specify the name of
>> &
Jérôme Pouiller writes:
> 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
>>
>> [...]
>>
>>
Pali Rohár writes:
> 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
Jérôme Pouiller writes:
> 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:
>>
Jérôme Pouiller writes:
> On Tuesday 5 October 2021 16:20:19 CEST Kalle Valo wrote:
>> Jerome Pouiller writes:
>>
>> > From: Jérôme Pouiller
> [...]
>> >
>> > v8:
>> > - Change the way the DT is handled. The user can now specify the na
Jérôme Pouiller writes:
> On Wednesday 6 October 2021 17:02:07 CEST Ulf Hansson wrote:
>> On Tue, 5 Oct 2021 at 10:14, Jérôme Pouiller
>> wrote:
>> > On Friday 1 October 2021 17:23:16 CEST Ulf Hansson wrote:
>> > > On Thu, 30 Sept 2021 at 19:06, Pali Rohár wrote:
>> > > > On Thursday 30 Septem
Jérôme Pouiller writes:
> Hi Kalle,
>
> On Friday 1 October 2021 14:18:04 CEST Kalle Valo wrote:
>> Jérôme Pouiller writes:
>>
>> > On Friday 1 October 2021 11:22:08 CEST Kalle Valo wrote:
>> >> Jerome Pouiller writes:
>> >>
>&g
Jérôme Pouiller writes:
>> >> >> I'm not really fond of having this kind of ASCII based parser in the
>> >> >> kernel. Do you have an example compressed file somewhere?
>> >> >
>> >> > An example of uncompressed configuration file can be found here[1]. Once
>> >> > compressed with [2], you get:
>
Kalle Valo writes:
> Jérôme Pouiller writes:
>
>>> >> >> I'm not really fond of having this kind of ASCII based parser in the
>>> >> >> kernel. Do you have an example compressed file somewhere?
>>> >> >
>>> >>
Jérôme Pouiller writes:
> On Thursday 7 October 2021 12:49:47 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
>
> Signed-off-by: Jérôme Pouiller
> ---
> .../bindings/net/wireless/silabs,wfx.yaml | 125 ---
> drivers/staging/wfx/Kconfig | 8 -
> drivers/staging/wfx/Makefile | 25 -
> drivers/staging/wfx/bh.c
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 qui
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/sil
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Apache-2.0 is not allowed in the kernel.
>
> Signed-off-by: Jérôme Pouiller
> ---
> drivers/staging/wfx/hif_api_cmd.h | 2 +-
> drivers/staging/wfx/hif_api_general.h | 2 +-
> drivers/staging/wfx/hif_api_mib.h | 2 +-
> 3 files change
Jérôme Pouiller writes:
> Hi Kalle,
>
> On Tuesday 11 January 2022 18:14:05 CET Jerome Pouiller wrote:
>> From: Jérôme Pouiller
>>
>> Signed-off-by: Jérôme Pouiller
>> ---
>> drivers/net/wireless/silabs/wfx/main.c | 485 +
>> drivers/net/wireless/silabs/wfx/main.h | 4
Jérôme Pouiller writes:
> On Thursday 10 February 2022 15:20:56 CET Kalle Valo wrote:
>>
>> Jérôme Pouiller writes:
>>
>> > Kalle, is this function what you expected? If it is right for you, I am
>> > going to send it to the staging tree.
>>
Jérôme Pouiller writes:
> On Thursday 10 February 2022 15:51:03 CET Kalle Valo wrote:
>> Jérôme Pouiller writes:
>> > On Thursday 10 February 2022 15:20:56 CET Kalle Valo wrote:
>> >> Jérôme Pouiller writes:
>> >>
>> >> > Kalle, is this
Jérôme Pouiller writes:
>> > There is also the patch 01/24 about the SDIO IDs.
>> >
>> > I think the v10 could contain only 3 patches:
>> >
>> > 1. mmc: sdio: add SDIO IDs for Silabs WF200 chip
>> > 2. dt-bindings: introduce silabs,wfx.yaml
>> > 3. [all the patches 3 to 24 squashed]
>
Greg Kroah-Hartman writes:
> On Wed, Oct 07, 2020 at 12:19:36PM +0200, Jerome Pouiller wrote:
>> From: Jérôme Pouiller
>>
>> I think the wfx driver is now mature enough to be accepted in the
>> drivers/net/wireless directory.
>>
>> There is still one item on the TODO list. It is an idea to imp
Kalle Valo writes:
> Greg Kroah-Hartman writes:
>
>> On Wed, Oct 07, 2020 at 12:19:36PM +0200, Jerome Pouiller wrote:
>>> From: Jérôme Pouiller
>>>
>>> I think the wfx driver is now mature enough to be accepted in the
>>> drivers/net/wireless di
Jérôme Pouiller writes:
> On Thursday 8 October 2020 09:30:06 CEST Kalle Valo wrote:
> [...]
>> Yes, the driver needs to be reviewed in linux-wireless list. I recommend
>> submitting the whole driver in a patchset with one file per patch, which
>> seems to be the easi
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Smatch complains:
>
>main.c:228 wfx_send_pdata_pds() warn: potential NULL parameter dereference
> 'tmp_buf'
>227 tmp_buf = kmemdup(pds->data, pds->size, GFP_KERNEL);
>228 ret = wfx_send_pds(wdev, tmp_buf, pds->siz
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Smatch complains:
>
>drivers/staging/wfx/hif_rx.c:177 hif_scan_complete_indication() warn:
> potential NULL parameter dereference 'wvif'
>drivers/staging/wfx/data_tx.c:576 wfx_flush() warn: potential NULL
> parameter dereference 'wvif
Jérôme Pouiller writes:
> On Tuesday 13 October 2020 18:49:35 CEST Rob Herring wrote:
>> On Mon, Oct 12, 2020 at 12:46:26PM +0200, Jerome Pouiller wrote:
>> > From: Jérôme Pouiller
> [...]
>> > + Note that in add of the properties below, the WFx driver also supports
>> > + `mac-address` and `l
Ulf Hansson writes:
> On Thu, 15 Oct 2020 at 16:03, Jérôme Pouiller
> wrote:
>>
>> On Wednesday 14 October 2020 14:43:34 CEST Pali Rohár wrote:
>> > On Wednesday 14 October 2020 13:52:15 Jérôme Pouiller wrote:
>> > > On Tuesday 13 October 2020 22:11:56 CEST Pali Rohár wrote:
>> > > > On Monday 1
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Signed-off-by: Jérôme Pouiller
[...]
> +wfx-$(CONFIG_SPI) += bus_spi.o
> +wfx-$(subst m,y,$(CONFIG_MMC)) += bus_sdio.o
Why this subst? And why only for MMC?
--
https://patchwork.kernel.org/project/linux-wireless/list/
https://wireless.wi
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Add Silabs SDIO ID to sdio_ids.h.
>
> 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.
>
> Signed-off-by: Jérôme Pouiller
> ---
> inclu
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Signed-off-by: Jérôme Pouiller
> ---
> drivers/net/wireless/silabs/wfx/Kconfig | 8
> drivers/net/wireless/silabs/wfx/Makefile | 25
> 2 files changed, 33 insertions(+)
> create mode 100644 drivers/net/wir
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> Signed-off-by: Jérôme Pouiller
[...]
> +static const struct ieee80211_supported_band wfx_band_2ghz = {
> + .channels = wfx_2ghz_chantable,
> + .n_channels = ARRAY_SIZE(wfx_2ghz_chantable),
> + .bitrates = wfx_rates,
> + .n_bi
Jerome Pouiller writes:
> +/*
> + * Internal helpers.
> + *
> + * About CONFIG_VMAP_STACK:
> + * When CONFIG_VMAP_STACK is enabled, it is not possible to run DMA on stack
> + * allocated data. Functions below that work with registers (aka functions
> + * ending with "32") automatically reallocate
Jerome Pouiller writes:
> --- /dev/null
> +++ b/drivers/net/wireless/silabs/wfx/hif_api_general.h
> @@ -0,0 +1,267 @@
> +/* SPDX-License-Identifier: Apache-2.0 */
> +/*
> + * WFx hardware interface definitions
> + *
> + * Copyright (c) 2018-2020, Silicon Laboratories Inc.
> + */
> +
> +#ifndef WF
Jerome Pouiller writes:
> From: Jérôme Pouiller
>
> I think the wfx driver is now mature enough to be accepted in the
> drivers/net/wireless directory.
What's the status with firmware images? Can anyone take the latest
kernel and linux-firmware and use this driver normally?
--
https://patchwo
Jerome Pouiller writes:
> +static bool ieee80211_is_action_back(struct ieee80211_hdr *hdr)
> +{
> + struct ieee80211_mgmt *mgmt = (struct ieee80211_mgmt *)hdr;
> +
> + if (!ieee80211_is_action(mgmt->frame_control))
> + return false;
> + if (mgmt->u.action.category != WLAN_
Jerome Pouiller writes:
> +void wfx_tx(struct ieee80211_hw *hw, struct ieee80211_tx_control *control,
> + struct sk_buff *skb)
> +{
> + struct wfx_dev *wdev = hw->priv;
> + struct wfx_vif *wvif;
> + struct ieee80211_sta *sta = control ? control->sta : NULL;
> + struct ieee
Jerome Pouiller writes:
> +/* NOTE: wfx_send_pds() destroy buf */
> +int wfx_send_pds(struct wfx_dev *wdev, u8 *buf, size_t len)
> +{
> + int ret;
> + int start, brace_level, i;
> +
> + start = 0;
> + brace_level = 0;
> + if (buf[0] != '{') {
> + dev_err(wdev->dev, "valid PDS
Jérôme Pouiller writes:
> On Tuesday 22 December 2020 16:02:38 CET Kalle Valo wrote:
>> Jerome Pouiller writes:
>>
>> > From: Jérôme Pouiller
>> >
>> > Signed-off-by: Jérôme Pouiller
>>
>> [...]
>>
>> > +wfx-$(CONFIG
Jérôme Pouiller writes:
> On Tuesday 22 December 2020 16:27:01 CET Greg Kroah-Hartman wrote:
>>
>> On Tue, Dec 22, 2020 at 05:10:11PM +0200, Kalle Valo wrote:
>> > Jerome Pouiller writes:
>> >
>> > > +/*
>> > > + * Intern
t; @@ -18,8 +18,6 @@ struct wilc_wfi_radiotap_cb_hdr {
> u16 tx_flags;
> } __packed;
>
> -static struct net_device *wilc_wfi_mon; /* global monitor netdev */
> -
> static u8 srcadd[6];
> static u8 bssid[6];
I hope you are working on moving srcadd an
(Sorry for taking so long with the reply, I wanted first to check what
the rtlwifi in staging contains.)
Larry Finger writes:
> On 08/24/2017 07:14 AM, Kalle Valo wrote:
>> Dan Carpenter writes:
>>
>>> Smatch is distrustful of the "capab" value and marks it a
ne (like 12 months or something) for the
drivers/staging/rtlwifi and after that you refuse to take any patches
for it. Hopefully this makes it clear for everyone that this fork is
just temporary. I think Larry is trying to do this, which is great.
2) We move the whole rtlwifi driver to staging. A very bad option but
still better than forking the drivers.
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Larry Finger writes:
> On 10/11/2017 08:13 AM, Greg Kroah-Hartman wrote:
>
>> On Wed, Oct 11, 2017 at 12:06:00PM +0300, Kalle Valo wrote:
>> I think it's horrid too. But, if no one is able to do the real work
>> here, we hurt users who just need to use their h
ike 10-12 patches per set. I think Dave also has a similar
rule for net patches.
I wrote a FAQ entry about this:
https://wireless.wiki.kernel.org/en/developers/documentation/submittingpatches#too_many_patches
--
Kalle Valo
___
devel mailing list
de...@lin
is nor really helpful :)
> Please don't repeat bad experience of Broadcom.
I agree with Oleksij here, but I want to still point out that there are
cases when using magic numbers are ok, for example look at
ar5008_initvals.h from ath9k. So it depends on case by case.
--
Kalle Valo
_
i has support for hardware which have not reached
customers yet.
> On the way, I'll attend netdev workshop in Korea, so we can meet there
> if you attend too.
I'm also attending Netdev 2.2, looking forward to meeting you there.
--
Kalle Valo
__
ome people split the driver to one file per
patch for easier review but the final commit needs to be one big patch
with all files included. If you are adding big files to rtlwifi I think
you could try to follow the same model.
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
e in net-next now.
Yup, I see it in net-next:
d17504b16ea2 wireless/lib80211: Convert from ahash to shash
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
read from DT
> -replace SIOCDEVPRIVATE commands with generic API functions
> -use wext-core handling instead of private SIOCSIWPRIV implementation
>From wireless point of view: if I see wext mentioned anywhere in the
driver I stop right t
Kalle Valo writes:
> Ajay Singh writes:
>
>> Hi Greg,
>>
>> We all are working on submitting and reviewing patches for wilc1000 in
>> staging driver for quite some time.
>>
>> We would like to have feedback on the next steps to bring wilc1000
Ajay Singh writes:
> On Thu, 16 Aug 2018 13:53:50 +0300
> Kalle Valo wrote:
>
>> Kalle Valo writes:
>>
>> > Ajay Singh writes:
>> >
>> >> Hi Greg,
>> >>
>> >> We all are working on submitting and reviewing patche
Arend van Spriel writes:
> On 8/17/2018 9:49 AM, Kalle Valo wrote:
>> Ajay Singh writes:
>>
>>> On Thu, 16 Aug 2018 13:53:50 +0300
>>> Kalle Valo wrote:
>>>
>>>> Kalle Valo writes:
>>>>
>>>>> From wireless point
her trees which might cause conflicts etc.
> And the fixes will be submitted to staging tree in parallel. right?
I don't think the fixes matter in the initial review. So yeah, Greg can
apply fixes as he sees fit for now. At some point we need to do a cut
off period and freeze the driver fo
| 2 --
> drivers/net/wireless/broadcom/brcm80211/Kconfig | 1 -
> drivers/net/wireless/quantenna/qtnfmac/Kconfig | 2 +-
> 12 files changed, 12 insertions(+), 19 deletions(-)
For wireless:
Acked-by: Kalle Valo
Leaving the hunks for linux-wireless list to see:
> diff --git a/d
r
>> attributes by mistake.
>>
>> This commit cleans up all usages of BIT macro with the above field
>> in cfg80211 by changing it to BIT_ULL instead.
>>
>> Signed-off-by: Omer Efrat
>
> Acked-by: Greg Kroah-Hartman
Via which tree is this
Greg KH writes:
> On Mon, Jun 18, 2018 at 10:29:43AM +0300, Kalle Valo wrote:
>> Greg KH writes:
>>
>> > On Sun, Jun 17, 2018 at 01:07:36PM +0300, Omer Efrat wrote:
>> >> The BIT macro uses unsigned long which some architectures handle as 32 bit
>> >
writes:
> From: Ajay Singh
>
> Remove labels and not relevant property from DT binding documentation
> examples as suggested in [1].
>
> 1. https://patchwork.ozlabs.org/patch/1252837
Just a nitpick but patchwork links are not that reliable in the long
run. Instead using a lore link is better as
writes:
> From: Ajay Singh
>
> This patch series is to review and move wilc1000 driver out of staging.
> Most of the review comments received in [1] & [2] are addressed in the
> latest code. Please review and provide your inputs.
>
> [1].
> https://lore.kernel.org/linux-wireless/1537957525-1146
George Spelvin writes:
> On Fri, Apr 03, 2020 at 12:10:29PM +0300, Dan Carpenter wrote:
>> On Thu, Apr 02, 2020 at 03:30:34PM +, George Spelvin wrote:
>> > On Thu, Apr 02, 2020 at 11:27:45AM +0300, Dan Carpenter wrote:
>> > > I don't know how this patch made it through two versions without an
r and get out of staging :) So next step
> is I guess study the ath6kl driver, learn how CFG80211 is done and
> implement that interface in ks7010? Oh, and test that it works.
Please keep linux-wireless list in loop so that people on that list can
help.
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
starting points to get familiar with
the stack.
And I guess you already saw the documentation:
https://wireless.wiki.kernel.org/en/developers/documentation/mac80211
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[V4.1+]
I think the "realtek:" prefix is superfluous, "rtlwifi:" should be
enough. I'll also add a fixes line before I commit this:
Fixes: 54328e64047a ("rtlwifi: rtl8821ae: Fix system lockups on boot")
--
Kalle Valo
s on boot")
> Signed-off-by: Larry Finger
> Cc: Stable [V4.1+]
Thanks, applied to wireless-drivers.git.
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
> Signed-off-by: Larry Finger
> ---
>
> Kalle,
>
> Is this change clear, and strong enough?
Looks good to me, thanks!
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
ndling of module parameters
b68d0ae7e586 rtlwifi: rtl8192cu: Add missing parameter setup
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Start modification for new drivers")
> Signed-off-by: Larry Finger
> Cc: Stable
Thanks, applied to wireless-drivers-next.git.
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
anges needed for the driver.
>
> Signed-off-by: Larry Finger
> Cc: Stable [V4.0+]
Thanks, 2 patches applied to wireless-drivers-next.git:
c18d8f509571 rtlwifi: rtl8723be: Add antenna select module parameter
baa170229095 rtlwifi: btcoexist: Implement antenna selection
Kalle Valo
_
58206e rtlwifi: rtl8821ae: Fix Smatch warnings
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
tree you are planning to submit
these for. For example, should I take ath6kl and mwifiex patches or
someone else?
--
Kalle Valo
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Julia Lawall writes:
> On Tue, 17 May 2016, Kalle Valo wrote:
>
>> Julia Lawall writes:
>>
>> > firmare -> firmware
>> >
>> > ---
>> >
>> > drivers/media/dvb-frontends/mn88473.c |2 +-
>> > drivers/net/w
ling
> code will be modified so that this long delay can be avoided; however,
> this change is being pushed now to fix the problem in kernel 4.6.0.
>
> Fixes: 49f86ec21c01 ("rtlwifi: Change long delays to sleeps")
> Reported-by: James Feeney
> Signed-off-by: Larry Finge
Larry Finger wrote:
> Commit 49f86ec21c01 ("rtlwifi: Change long delays to sleeps") was correct
> for most cases; however, driver rtl8192ce calls the affected routines while
> in atomic context. The kernel bug output is as follows:
>
> BUG: scheduling while atomic: wpa_supplicant/627/0x0002
>
he fix is to allow both bands even when the EEPROM
> is incorrectly encoded.
>
> Fixes: d10101a60372 ("rtlwifi: rtl8821ae: Fix problem with regulatory
> information")
> Signed-off-by: Larry Finger
> Cc: littlesmart...@gmail.com
>
security and chaning virtual interface
>
> Changes in V2: details in each patch files.
> - 0004-staging-wilc1000-add-sdio-resume-suspend.patch
It's good practise to also add "staging: wilc1000: " prefix to the cover
letter.
--
Kalle Valo
oth bands even when the EEPROM
> is incorrectly encoded.
>
> Fixes: d10101a60372 ("rtlwifi: rtl8821ae: Fix problem with regulatory
> information")
> Signed-off-by: Larry Finger
> Cc: littlesmart...@gmail.com
> Cc: g...@co
;> any TODO tasks left to do beside checkpatch.pl clean-up?
For questions like this you should CC linux-wireless.
> I happened to look through this code recently. It's terrible.
And uses wireless extensions, yuck. There's a lot of work to get it into
reasonable shape, fixing chec
1 - 100 of 164 matches
Mail list logo
