From: David Sterba <dsterba@suse.cz>
To: Arnd Bergmann <arnd@arndb.de>
Cc: Jiri Slaby <jirislaby@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
linux-kernel@vger.kernel.org, Jiri Kosina <jikos@kernel.org>,
David Sterba <dsterba@suse.com>,
Dominik Brodowski <linux@dominikbrodowski.net>
Subject: Re: [PATCH 1/2] tty: ipwireless: move Kconfig entry to tty
Date: Thu, 23 Feb 2023 18:24:03 +0100 [thread overview]
Message-ID: <20230223172403.GW10580@suse.cz> (raw)
In-Reply-To: <86e99adb-eccd-4c78-8128-b3690aa559c9@app.fastmail.com>
On Thu, Feb 23, 2023 at 09:20:53AM +0100, Arnd Bergmann wrote:
> On Wed, Feb 22, 2023, at 09:50, Jiri Slaby (SUSE) wrote:
> > The ipwireless' Kconfig entry is currently sitting in char/pcmcia. But
> > the driver is in tty/. So move the entry to tty/ too.
> >
> > First, it makes sense to have the entry along the driver. Second, whole
> > char/pcmcia is about to pass away.
> >
> > Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
> > Cc: Jiri Kosina <jikos@kernel.org>
> > Cc: David Sterba <dsterba@suse.com>
> > Cc: Arnd Bergmann <arnd@arndb.de>
> > Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
>
> This is clearly correct
>
> Reviewed-by: Arnd Bergmann <arnd@arndb.de>
>
> In Dominik's list from
> https://lore.kernel.org/all/Y07d7rMvd5++85BJ@owl.dominikbrodowski.net/
>
> he had listed this driver as 'some activity in 2021', but all I can
> find are treewide cleanups (mostly from you) after some work by David
> in 2008 and Dominik's own PCMCIA rework in 2010. My guess is that
> most likely this driver can be removed entirely, but you and David
> are the ones who know if there are still users.
The driver works and did not need any development, so the treewide or
minor bugfixes are understandable. I had some discussions with a user in
2011 regarding card detection but otherwise nothing. I can't tell how
many users are there, there are some IPWireless cards in second hand
market and cards for pcmcia can be bought in local shops.
I'd vote for keeping the driver as long as the pcmcia subsystem exists
(option a).
prev parent reply other threads:[~2023-02-23 17:30 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-02-22 8:50 [PATCH 1/2] tty: ipwireless: move Kconfig entry to tty Jiri Slaby (SUSE)
2023-02-22 8:50 ` [PATCH 2/2] char: pcmcia: remove all the drivers Jiri Slaby (SUSE)
2023-02-22 8:51 ` Jiri Slaby
2023-02-23 8:20 ` [PATCH 1/2] tty: ipwireless: move Kconfig entry to tty Arnd Bergmann
2023-02-23 17:24 ` David Sterba [this message]
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=20230223172403.GW10580@suse.cz \
--to=dsterba@suse.cz \
--cc=arnd@arndb.de \
--cc=dsterba@suse.com \
--cc=gregkh@linuxfoundation.org \
--cc=jikos@kernel.org \
--cc=jirislaby@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@dominikbrodowski.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox