linux-kernel - Re: patch sdhci-add-no_busy_irq-quirk-for-marvell-cafe-host-chip.patch added to 2.6.28-stable tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 13 Mar 2009 17:24:44 -0700
From:	Greg KH <gregkh@...e.de>
To:	Stefan Lippers-Hollmann <s.L-H@....de>
Cc:	linux-kernel@...r.kernel.org, dilinger@...ian.org,
	drzeus@...eus.cx, stable@...nel.org
Subject: Re: patch
	sdhci-add-no_busy_irq-quirk-for-marvell-cafe-host-chip.patch added
	to 2.6.28-stable tree

On Sat, Mar 14, 2009 at 01:20:49AM +0100, Stefan Lippers-Hollmann wrote:
> Hi
> 
> On Freitag, 13. März 2009, gregkh@...e.de wrote:
> > This is a note to let you know that we have just queued up the patch titled
> > 
> >     Subject: sdhci: Add NO_BUSY_IRQ quirk for Marvell CAFE host chip
> > 
> > to the 2.6.28-stable tree.  Its filename is
> > 
> >     sdhci-add-no_busy_irq-quirk-for-marvell-cafe-host-chip.patch
> > 
> > A git repo of this tree can be found at 
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > 
> > From a0874897b1ba106298e4303a25456a473fc40f3d Mon Sep 17 00:00:00 2001
> > From: Andres Salomon <dilinger@...ian.org>
> > Date: Mon, 2 Mar 2009 21:48:20 +0100
> > Subject: sdhci: Add NO_BUSY_IRQ quirk for Marvell CAFE host chip
> > 
> > From: Andres Salomon <dilinger@...ian.org>
> > 
> > commit a0874897b1ba106298e4303a25456a473fc40f3d upstream.
> > 
> > As described here: http://lkml.org/lkml/2009/2/20/265
> > 
> > The CAFE chip is broken due to commit e809517f6fa5803a5a1cd5602.
> > Anton added a quirk here: http://lkml.org/lkml/2009/2/20/279 that fixes
> > CAFE's problem.  This adds the quirk for CAFE.
> 
> Adding this patch alone breaks compiling the current -stable queue for 
> 2.6.28:
> 
>   CC [M]  drivers/mmc/host/sdhci-pci.o
> drivers/mmc/host/sdhci-pci.c:110: error: 'SDHCI_QUIRK_NO_BUSY_IRQ' undeclared here (not in a function)
> make[6]: *** [drivers/mmc/host/sdhci-pci.o] Error 1
> make[5]: *** [drivers/mmc/host] Error 2
> make[4]: *** [drivers/mmc] Error 2
> make[3]: *** [drivers] Error 2
> 
> It seems to depend the following patch, wich introduces this particular 
> quirk type:
> 
> commit f945405cdecd9e0ae3e58ff84cabd19b4522965e
> Author: Ben Dooks <ben-linux@...ff.org>
> Date:   Fri Feb 20 20:33:08 2009 +0300
> 
>     sdhci: Add quirk for controllers with no end-of-busy IRQ
> 
>     The Samsung SDHCI (and FSL eSDHC) controller block seems to fail
>     to generate an INT_DATA_END after the transfer has completed and
>     the bus busy state finished.
> 
>     Changes in e809517f6fa5803a5a1cd56026f0e2190fc13d5c to use the
>     new busy method are the cause of the behaviour change.
> 
>     Signed-off-by: Ben Dooks <ben-linux@...ff.org>
>     Signed-off-by: Anton Vorontsov <avorontsov@...mvista.com>
>     Signed-off-by: Pierre Ossman <drzeus@...eus.cx>

Ah, good catch, thanks, I'll go add that patch as well.

greg k-h
--
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/