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]
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)

Powered by blists - more mailing lists