linux-kernel - Re: [OLPC] sdhci: add quirk for the Marvell CaFe's vdd/powerup issue

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <20080627182945.615d5adf@mjolnir.drzeus.cx>
Date:	Fri, 27 Jun 2008 18:29:45 +0200
From:	Pierre Ossman <drzeus@...eus.cx>
To:	Andres Salomon <dilinger@...ued.net>
Cc:	akpm@...ux-foundation.org, drzeus-sdhci@...eus.cx,
	sdhci-devel@...ts.drzeus.cx, linux-kernel@...r.kernel.org
Subject: Re: [OLPC] sdhci: add quirk for the Marvell CaFe's vdd/powerup
 issue

On Sat, 21 Jun 2008 21:15:16 -0400
Andres Salomon <dilinger@...ued.net> wrote:

> This has been sitting around unloved for way too long..
> 
> The Marvell CaFe chip's SD implementation chokes during card insertion
> if one attempts to set the voltage and power up in the same
> SDHCI_POWER_CONTROL register write.  This adds a quirk that does
> that particular dance in two steps.
> 
> It also adds an entry to pci_ids.h for the CaFe chip's SD device.
> 
> Signed-off-by: Andres Salomon <dilinger@...ian.org>
> ---

Looks ok, but please try to diff against my -next branch. The sdhci
driver has been split up into a core and PCI part.

> @@ -774,6 +784,14 @@ static void sdhci_set_power(struct sdhci_host *host, unsigned short power)
>  		BUG();
>  	}
>  
> +	/*
> +	 * At least the CaFe chip gets confused if we set the voltage
> +	 * and set turn on power at the same time, so set the voltage first.
> +	 */

This comment should probably say "Marvell controller" or something like
that. Few know of the CaFE name.

-- 
     -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.

Download attachment "signature.asc" of type "application/pgp-signature" (198 bytes)