From: Johan Hovold <johan@kernel.org>
To: Marco Felsch <m.felsch@pengutronix.de>
Cc: Rob Herring <robh@kernel.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Jiri Slaby <jirislaby@kernel.org>,
linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-usb@vger.kernel.org
Subject: Re: [PATCH 0/3] USB-Serial serdev support
Date: Mon, 9 Sep 2024 14:03:23 +0200 [thread overview]
Message-ID: <Zt7kCxawoszunWq3@hovoldconsulting.com> (raw)
In-Reply-To: <20240807-v6-10-topic-usb-serial-serdev-v1-0-ed2cc5da591f@pengutronix.de>
On Wed, Aug 07, 2024 at 04:08:47PM +0200, Marco Felsch wrote:
> this patchset is based on Johan's patches [1] but dropped the need of
> the special 'serial' of-node [2].
That's great that you found and referenced my proof-of-concept patches,
but it doesn't seem like you tried to understand why this hasn't been
merged yet.
First, as the commit message you refer to below explain, we need some
way to describe multiport controllers. Just dropping the 'serial' node
does not make that issue go away.
Second, and more importantly, you do not address the main obstacle for
enabling serdev for USB serial which is that the serdev cannot handle
hotplugging.
> [1] https://git.kernel.org/pub/scm/linux/kernel/git/johan/usb-serial.git/log/?h=usb-serial-of
> [2] https://git.kernel.org/pub/scm/linux/kernel/git/johan/usb-serial.git/commit/?h=usb-serial-of&id=b19239022c92567a6a9ed40e8522e84972b0997f
Johan
next prev parent reply other threads:[~2024-09-09 12:03 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-08-07 14:08 [PATCH 0/3] USB-Serial serdev support Marco Felsch
2024-08-07 14:08 ` [PATCH 1/3] serdev: ttyport: make use of tty_kopen_exclusive Marco Felsch
2024-08-08 7:51 ` Jiri Slaby
2024-08-19 10:19 ` Marco Felsch
2024-08-19 10:42 ` Jiri Slaby
2024-08-19 12:23 ` Marco Felsch
2024-08-21 7:25 ` Marco Felsch
2024-08-07 14:08 ` [PATCH 2/3] USB: serial: cosmetic cleanup <space><tab> mix Marco Felsch
2024-08-07 14:08 ` [PATCH 3/3] USB: serial: enable serdev support Marco Felsch
2024-09-09 12:03 ` Johan Hovold [this message]
2024-09-17 4:49 ` [PATCH 0/3] USB-Serial " Marco Felsch
2025-03-11 8:12 ` Johan Hovold
2025-03-13 19:40 ` Marco Felsch
2025-08-21 16:40 ` Marco Felsch
2025-10-23 12:32 ` Johan Hovold
2025-10-23 13:48 ` Marco Felsch
2025-10-24 8:21 ` Johan Hovold
2025-10-24 9:27 ` Marco Felsch
2025-10-24 10:32 ` Johan Hovold
2025-10-24 12:40 ` Marco Felsch
2025-10-24 13:26 ` Johan Hovold
2025-10-24 16:22 ` Marco Felsch
2024-10-01 7:24 ` Marco Felsch
2024-10-01 7:29 ` Greg Kroah-Hartman
2024-10-01 7:47 ` Marco Felsch
2024-10-28 22:57 ` Marco Felsch
2025-03-03 11:25 ` Marco Felsch
2025-03-11 8:20 ` Johan Hovold
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=Zt7kCxawoszunWq3@hovoldconsulting.com \
--to=johan@kernel.org \
--cc=gregkh@linuxfoundation.org \
--cc=jirislaby@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux-usb@vger.kernel.org \
--cc=m.felsch@pengutronix.de \
--cc=robh@kernel.org \
/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