linux-kernel - Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

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]

Message-ID: <459D103E.4080605@drzeus.cx>
Date:	Thu, 04 Jan 2007 15:33:34 +0100
From:	Pierre Ossman <drzeus-list@...eus.cx>
To:	Philip Langdale <philipl@...rt.org>
CC:	linux-kernel@...r.kernel.org, Alex Dubov <oakad@...oo.com>,
	Andrew Morton <akpm@...l.org>
Subject: Re: [PATCH 2.6.19] mmc: Add support for SDHC cards (Take 4)

Philip Langdale wrote:
> In the process of developing this patch, we realised that the R6 response definition
> was incorrect and that it should have been identical to R1 (and so should R7).
> Correcting this mistake revealed problems in a couple of host controller drivers that
> relied on the response definitions to be unique. As we need a story for R7, I've
> also fixed up the R6 handling in the affected drivers but I have no idea if it will
> work in practice as I lack the hardware.
>
>   

Do this fix in a separate patch. And cc the relevant maintainers.

> --- /usr/src/linux/drivers/mmc/imxmmc.c	2007-01-01 07:12:02.000000000 -0800
> +++ linux-2.6.19-sdhc/drivers/mmc/imxmmc.c	2007-01-04 05:50:41.000000000 -0800
> @@ -351,9 +360,6 @@
>  	case MMC_RSP_R3: /* short */
>  		cmdat |= CMD_DAT_CONT_RESPONSE_FORMAT_R3;
>  		break;
> -	case MMC_RSP_R6: /* short CRC */
> -		cmdat |= CMD_DAT_CONT_RESPONSE_FORMAT_R6;
> -		break;
>  	default:
>  		break;
>  	}
>   

I think this chunk suffices. Until proven otherwise, regard R6 and R7 as
information the hw does not need to know about. Same thing in tifm_sd.

Rgds

-- 
     -- Pierre Ossman

  Linux kernel, MMC maintainer        http://www.kernel.org
  PulseAudio, core developer          http://pulseaudio.org
  rdesktop, core developer          http://www.rdesktop.org

-
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/