All of lore.kernel.org
 help / color / mirror / Atom feed
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Arnd Bergmann <arnd@arndb.de>,
	Philipp Hortmann <philipp.g.hortmann@gmail.com>,
	Arnd Bergmann <arnd@kernel.org>, Kalle Valo <kvalo@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Nicolas Ferre <nicolas.ferre@microchip.com>,
	Alexandre Belloni <alexandre.belloni@bootlin.com>,
	Claudiu Beznea <claudiu.beznea@tuxon.dev>,
	Jakub Kicinski <kuba@kernel.org>, Pavel Machek <pavel@ucw.cz>,
	"David S . Miller" <davem@davemloft.net>,
	linux-kernel@vger.kernel.org, linux-wireless@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org,
	linux-staging@lists.linux.dev
Subject: Re: [PATCH] [RFC] wireless: move obsolete drivers to staging
Date: Fri, 13 Oct 2023 11:24:40 -0500	[thread overview]
Message-ID: <56fe16bf-3fc9-49f9-9646-c236b885afa5@lwfinger.net> (raw)
In-Reply-To: <c8f43e1b-aed1-4b45-ba7c-d896ff66dfa7@app.fastmail.com>

On 10/13/23 10:36, Arnd Bergmann wrote:
> At the moment, I'd suggest focusing on the drivers that still use wext (git grep 
> -w iw_handler_def drivers), if we can show that rtl8192e, rtl8712 or ks7010 have 
> been broken for a while, removing those would help with removing wext altogether.

I do not know about the the others, but rtl8712 is still in use according to the 
flow of questions I see on a GitHub repo that I maintain.

There has been some recent effort to convert rtl8192e to use cfg/nl/mac80211. 
That one is problematic as it has the same PCI_ID as RTL8192SE, but the drivers 
differ. Note the special code in the probe routine of 
drivers/net/wireless/realtek/rtlwifi/rtl8192se to detect if the wrong one has 
triggered the routine. That code should also be in the probe routine for 
RTL8192E with the test negated. I have both devices, but it has been a long time 
since I actually used either of them. I also have an rtl8712 device, thus I 
should be available for testing.

Larry


WARNING: multiple messages have this Message-ID (diff)
From: Larry Finger <Larry.Finger@lwfinger.net>
To: Arnd Bergmann <arnd@arndb.de>,
	Philipp Hortmann <philipp.g.hortmann@gmail.com>,
	Arnd Bergmann <arnd@kernel.org>, Kalle Valo <kvalo@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Alexandre Belloni <alexandre.belloni@bootlin.com>,
	linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev,
	linux-wireless@vger.kernel.org,
	Claudiu Beznea <claudiu.beznea@tuxon.dev>,
	Pavel Machek <pavel@ucw.cz>, Jakub Kicinski <kuba@kernel.org>,
	"David S . Miller" <davem@davemloft.net>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH] [RFC] wireless: move obsolete drivers to staging
Date: Fri, 13 Oct 2023 11:24:40 -0500	[thread overview]
Message-ID: <56fe16bf-3fc9-49f9-9646-c236b885afa5@lwfinger.net> (raw)
In-Reply-To: <c8f43e1b-aed1-4b45-ba7c-d896ff66dfa7@app.fastmail.com>

On 10/13/23 10:36, Arnd Bergmann wrote:
> At the moment, I'd suggest focusing on the drivers that still use wext (git grep 
> -w iw_handler_def drivers), if we can show that rtl8192e, rtl8712 or ks7010 have 
> been broken for a while, removing those would help with removing wext altogether.

I do not know about the the others, but rtl8712 is still in use according to the 
flow of questions I see on a GitHub repo that I maintain.

There has been some recent effort to convert rtl8192e to use cfg/nl/mac80211. 
That one is problematic as it has the same PCI_ID as RTL8192SE, but the drivers 
differ. Note the special code in the probe routine of 
drivers/net/wireless/realtek/rtlwifi/rtl8192se to detect if the wrong one has 
triggered the routine. That code should also be in the probe routine for 
RTL8192E with the test negated. I have both devices, but it has been a long time 
since I actually used either of them. I also have an rtl8712 device, thus I 
should be available for testing.

Larry


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2023-10-13 16:24 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-10-10 15:27 [PATCH] [RFC] wireless: move obsolete drivers to staging Arnd Bergmann
2023-10-10 15:27 ` Arnd Bergmann
2023-10-10 16:08 ` Greg Kroah-Hartman
2023-10-10 16:08   ` Greg Kroah-Hartman
2023-10-11  5:40   ` Kalle Valo
2023-10-11  5:40     ` Kalle Valo
2023-10-11  6:16     ` Greg Kroah-Hartman
2023-10-11  6:16       ` Greg Kroah-Hartman
2023-10-11  7:29       ` Kalle Valo
2023-10-11  7:29         ` Kalle Valo
2023-10-11 15:09         ` Jakub Kicinski
2023-10-11 15:09           ` Jakub Kicinski
2023-10-12 11:47           ` Kalle Valo
2023-10-12 11:47             ` Kalle Valo
2023-10-12 14:36             ` Arnd Bergmann
2023-10-12 14:36               ` Arnd Bergmann
2023-10-12 14:48               ` Kalle Valo
2023-10-12 14:48                 ` Kalle Valo
2023-10-12 15:11                 ` Geert Uytterhoeven
2023-10-12 15:11                   ` Geert Uytterhoeven
2023-10-12 15:11                   ` Geert Uytterhoeven
2023-10-12 15:28               ` Johannes Berg
2023-10-12 15:28                 ` Johannes Berg
2023-10-12 15:28                 ` Johannes Berg
2023-10-12 15:39                 ` Geert Uytterhoeven
2023-10-12 15:39                   ` Geert Uytterhoeven
2023-10-12 15:39                   ` Geert Uytterhoeven
2023-10-12 15:41                   ` Johannes Berg
2023-10-12 15:41                     ` Johannes Berg
2023-10-12 15:41                     ` Johannes Berg
2023-10-12 16:36                     ` Geoff Levand
2023-10-12 16:36                       ` Geoff Levand
2023-10-12 16:36                       ` Geoff Levand
2023-10-13 15:44                       ` Arnd Bergmann
2023-10-13 15:44                         ` Arnd Bergmann
2023-10-13 15:44                         ` Arnd Bergmann
2023-10-13 18:02                         ` Johannes Berg
2023-10-13 18:02                           ` Johannes Berg
2023-10-13 18:02                           ` Johannes Berg
2023-10-18 20:37                           ` Jeff Johnson
2023-10-18 20:37                             ` Jeff Johnson
2023-10-18 20:37                             ` Jeff Johnson
2023-10-12 16:16               ` Philipp Hortmann
2023-10-12 16:16                 ` Philipp Hortmann
2023-10-12 16:18               ` Dan Williams
2023-10-12 16:18                 ` Dan Williams
2023-10-11  7:24     ` Arnd Bergmann
2023-10-11  7:24       ` Arnd Bergmann
2023-10-11  7:41       ` Greg Kroah-Hartman
2023-10-11  7:41         ` Greg Kroah-Hartman
2023-10-11  8:44       ` Kalle Valo
2023-10-11  8:44         ` Kalle Valo
2023-10-11  9:01         ` Arnd Bergmann
2023-10-11  9:01           ` Arnd Bergmann
2023-10-11 18:13 ` Philipp Hortmann
2023-10-11 18:13   ` Philipp Hortmann
2023-10-11 20:22   ` Arnd Bergmann
2023-10-11 20:22     ` Arnd Bergmann
2023-10-12  6:23     ` Dan Carpenter
2023-10-12  6:23       ` Dan Carpenter
2023-10-13  6:42 ` Philipp Hortmann
2023-10-13  6:42   ` Philipp Hortmann
2023-10-13 15:36   ` Arnd Bergmann
2023-10-13 15:36     ` Arnd Bergmann
2023-10-13 16:24     ` Larry Finger [this message]
2023-10-13 16:24       ` Larry Finger
2023-10-15 18:42     ` Philipp Hortmann
2023-10-15 18:42       ` Philipp Hortmann
2023-10-18 16:05       ` Arnd Bergmann
2023-10-18 16:05         ` Arnd Bergmann
2023-10-13 11:43 ` kernel test robot

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=56fe16bf-3fc9-49f9-9646-c236b885afa5@lwfinger.net \
    --to=larry.finger@lwfinger.net \
    --cc=alexandre.belloni@bootlin.com \
    --cc=arnd@arndb.de \
    --cc=arnd@kernel.org \
    --cc=claudiu.beznea@tuxon.dev \
    --cc=davem@davemloft.net \
    --cc=gregkh@linuxfoundation.org \
    --cc=kuba@kernel.org \
    --cc=kvalo@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-staging@lists.linux.dev \
    --cc=linux-wireless@vger.kernel.org \
    --cc=nicolas.ferre@microchip.com \
    --cc=pavel@ucw.cz \
    --cc=philipp.g.hortmann@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.