From: Johan Hovold <johan@kernel.org>
To: Charles Yeh <charlesyeh522@gmail.com>
Cc: gregkh@linuxfoundation.org, linux-usb@vger.kernel.org,
charles-yeh@prolific.com.tw
Subject: Re: [PATCH v3] USB: serial: pl2303: add support for PL256X multi-port devices
Date: Mon, 3 Aug 2026 09:01:17 +0200 [thread overview]
Message-ID: <anA8vXRbP_S2jPUV@hovoldconsulting.com> (raw)
In-Reply-To: <20260721112440.1816-1-charlesyeh522@gmail.com>
On Tue, Jul 21, 2026 at 07:24:40PM +0800, Charles Yeh wrote:
> Prolific PL256X devices are multi-port USB-to-UART controllers,
> including the PL2533, PL2543 and PL2565 variants.
>
> These devices use vendor requests that differ from those used by the
> existing TYPE_HX and TYPE_HXN devices. They also require a separate UART
> reset request and use a port-specific register for configuring flow
> control.
>
> Add a new TYPE_MP device type and select the appropriate vendor requests,
> reset operation and flow-control register for PL256X devices. Store the
> USB interface number so that requests can be directed to the corresponding
> UART port.
>
> Detect the supported PL256X variants using bcdDevice before issuing any
> legacy vendor requests, as PL256X devices do not accept those requests.
>
> PL256X devices support baud rates up to 24 Mbps and do not use divisor
> encoding.
>
> Signed-off-by: Charles Yeh <charlesyeh522@gmail.com>
> ---
> v3:
> * update the patch subject to describe PL256X multi-port support
> * add device variant comments to the bcdDevice table
> * use lowercase hexadecimal constants
> * simplify pl256x_uart_reset() by removing unnecessary arguments
> * improve the UART reset error message
> * fix continuation-line indentation
> * preserve reverse Christmas tree declaration order
> * add a default error case for unsupported interface numbers
> * remove the redundant flow-control configuration from startup
Thanks for the update. Now applied.
Johan
prev parent reply other threads:[~2026-08-03 7:01 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-21 11:24 [PATCH v3] USB: serial: pl2303: add support for PL256X multi-port devices Charles Yeh
2026-08-03 7:01 ` Johan Hovold [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=anA8vXRbP_S2jPUV@hovoldconsulting.com \
--to=johan@kernel.org \
--cc=charles-yeh@prolific.com.tw \
--cc=charlesyeh522@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=linux-usb@vger.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 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.