lkml.org 
[lkml]   [2002]   [Feb]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH] Kernel support for 16C950's CPR register
From
On Wed, Feb 20, 2002 at 04:56:03PM +0100, fabrizio.gennari@philips.com wrote:
> The 16C950 serial port from Oxford Semiconductor has a special register,
> called CPR (Clock Predivisor Register). This increases in steps of 1/8,
> thus allowing a finer control on the generated baud rate than the ordinary
> divisor registers DLL and DLM, that only increase in steps of 1.
>
> There are 3 patches for CPR:
> 1. one written by me
> 2. one written by Jean Tourrilhes
> 3. one written by Ian Abbott
>
> Patches 1 and 3 are available at
> http://sourceforge.net/tracker/?atid=300310&group_id=310&func=browse .
> Patch 2 is available at
> http://www.geocrawler.com/archives/3/8352/2001/10/0/6948700/ .

My patch is also available at :
http://www.hpl.hp.com/personal/Jean_Tourrilhes/bt/

> 1. My patch is crap :) Avoid
> 2. Jean Tourrilhes' patch does not use CPR if the requested speed is
> higher than 115200 b/s. This is fine if the clock frequency is an integral
> multiple of 16*115200=1843200, but otherwise it does not exploit CPR's
> potential of fine-tuning speed.
> 3. Ian Abbott's patch is by far the best

Well... I disagree. My patches has advantage over Ian's patch :
o autoset baud_base/CPR (Ian's patch doesn't)
o allow to set custom MSR (Ian's patch doesn't)
o much simpler. My code is readable, has minimal
impact on the general path (non 16C950) and doesn't have ugly gotos
all over the place.
To claim that Ian's patch is the best, you would need to at
least fix 1 and 2. Number 3 is debatable.
Regards,

Jean
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 13:24    [W:0.042 / U:0.320 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site