[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CABE8wwsHZLiihEReRQwkk+SufkmBw2W4n-WibQgS-rEOnKNGMA@mail.gmail.com>
Date: Mon, 9 Apr 2012 09:53:47 -0700
From: "Williams, Dan J" <dan.j.williams@...el.com>
To: Greg KH <gregkh@...uxfoundation.org>
Cc: Alan Cox <alan@...rguk.ukuu.org.uk>, sudhakar <sudhakar@...com>,
linux-serial@...r.kernel.org, alan@...ux.intel.com,
nhan.h.mai@...el.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 1/2] 8250_pci: Replace commits 448ac154 and e86ff4a
On Mon, Apr 9, 2012 at 9:47 AM, Greg KH <gregkh@...uxfoundation.org> wrote:
> On Tue, Apr 03, 2012 at 09:58:32PM +0100, Alan Cox wrote:
>> On Tue, 3 Apr 2012 12:43:20 -0700
>> sudhakar <sudhakar@...com> wrote:
>>
>> >
>> > From: Sudhakar Mamillapalli <sudhakar@...com>
>> >
>> >
>> > The following commits don't completely fix the KT serial missing
>> > interrupt problems. So they have been replaced instead by
>> > enabling serial backup timer. The iir-once + msi tried approach tried
>> > to enforce one read of the iir per THRE event, but its defeated by any
>> > other event occurring at the wrong time, particularly MSR events.
>> >
>> > 448ac15 serial/8250_pci: setup-quirk workaround for the kt serial controller
>> > e86ff4a serial/8250_pci: init-quirk msi support for kt serial controller
>>
>> These are already in tree. If you are reversing them and adding a
>> different fix then please post the series as
>>
>> - two revert patches exactly reverting the two commits
>> - a follow on patch or two adding the replacement
>>
>> That makes it much easier to review and much more obvious. It also helps
>> make testing simpler.
>
> I agree, Sudhakar, can you please redo these 2 patches?
>
Hi Greg,
This was in the series:
[PATCH 0/6] rework quirks for the "kt" serial port [1]
...but I have just noticed it collides with current git, and I also
need to address Stephen's comments for Tegra. Expect a v2 shortly.
--
Dan
[1]: http://marc.info/?l=linux-serial&m=133373702606784&w=2
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists