Thread (3 messages) flat view 3 messages, 3 authors, 2021-03-16

Re: [PATCH 1/1] USB: serial: pl2303: TA & TB alternate divider

From: Johan Hovold <johan@kernel.org>
Date: 2021-03-16 09:19:46

On Tue, Mar 16, 2021 at 08:19:57AM +0100, Greg Kroah-Hartman wrote:
On Mon, Mar 15, 2021 at 09:42:05PM -0400, michaelk@IEEE.org wrote:
quoted
From: Michael Katzmann <redacted>

Use an alternate clock divider algorithm and bit ordering for the TA and
TB versions of the pl2303. It was discovered that these variants do not
produce the correct baud rates with the existing scheme.

see https://marc.info/?t=161392209800002&r=1&w=2
Can you point to a lore.kernel.org thread instead?  We (kernel developer
community) have control over lore, but not over marc.info or any other
archive, so we never know if it will be around or not over time :)
I was gonna say that I could fix that up when applying, but turns out
there are some formatting issues with the patch.

Michael, it looks like you've used tabs instead of spaces for
indentation in some places. Try running scripts/checkpatch.pl on the
patch before submitting (which would have caught this).

Care to send a v2?

And the lore link would be:

	https://lore.kernel.org/r/3aee5708-7961-f464-8c5f-6685d96920d6@IEEE.org (local)

Johan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help