lkml.org 
[lkml]   [2013]   [Feb]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [ 061/128] 8250: blacklist Winbond CIR port
From
Date
On Sun, 2013-02-03 at 17:26 +0000, Sean Young wrote:
> On Sun, Feb 03, 2013 at 03:47:45PM +0100, Ben Hutchings wrote:
> > 3.2-stable review patch. If anyone has any objections, please let me know.
> >
> > ------------------
> >
> > From: Sean Young <sean@mess.org>
> >
> > commit 65ecc9c02dbad033a73a32916d17c107c5b25031 upstream.
> >
> > The legacy serial driver will detect the Winbond CIR device as a serial
> > port, since it looks exactly like a serial port unless you know what
> > it is from the PNP ID.
> >
> > Here we track this port as a special PORT_8250_CIR type, preventing the
> > legacy serial driver from probing it.
>
> This commit relies on an earlier commit which ensures that 8250 PNP probe
> is done before legacy 8250 probe, else the Winbond CIR device will be
> claimed by the legacy 8250 driver rendering above commit useless.
>
> commit 835d844d1a28efba81d5aca7385e24c29d3a6db2
> Author: Sean Young <sean@mess.org>
> Date: Fri Sep 7 19:06:23 2012 +0100
>
> 8250_pnp: do pnp probe before legacy probe
>
> We first probe the legacy serial ports and then check pnp. If there
> is a non-standard configuration then this might not work, also this
> change is needed so we can blacklist Winbond CIR based on PNP ID.
>
> For this to work the 8250_pnp driver must be merged into the 8250
> module.

That looks a bit too disruptive for stable. I added the Winbond CIR
patch because commit ebebd49a8eab ('8250/16?50: Add support for Broadcom
TruManage redirected serial port') textually depends on it and it looked
like a worthwhile fix too. Since this doesn't do anything on its own,
I'll drop it and rebase the TruManage patch.

Ben.

--
Ben Hutchings
Everything should be made as simple as possible, but not simpler.
- Albert Einstein
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2013-02-04 01:01    [W:0.638 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site