On Nov 20, 2010, at 4:35 AM, Wolfram Sang wrote: > I know it's too late, but... > > On Fri, Nov 19, 2010 at 09:53:33PM +0000, Chris Ball wrote: >> On Fri, Nov 19, 2010 at 09:40:02PM +0000, Chris Ball wrote: >>> I don't see why we should re-read ctrl here, since we've already written >>> it back to the device at this point, and we don't use it anywhere below >>> this line. >> >> Ah, I see why now; please ignore this. >> >> Here's a rebased version of the patch, with some more comments: >> >> From: Philip Rakity <[email protected]> >> Date: Fri, 19 Nov 2010 16:48:39 -0500 >> Subject: [PATCH] mmc: sdhci: 8-bit bus width changes >> >> We now: >> * check for a v3 controller before setting 8-bit bus width >> * offer a callback for platform code to switch to 8-bit mode, which >> allows non-v3 controllers to support it > > What does the platform_-prefix of the callback indicate?
used for controllers that can support 8 bits and NOT v3 > >> * introduce a quirk to specify that the board designers have indeed >> brought out all the pins for 8-bit to the slot. > > This is not a quirk, this is platform_data, no? No opinion -- if it makes sense to move to a platform define I am happy to do that but I would like to see some standard platform defines eg This one and the one for CARD_PRESENT that can be used consistently across sdhci.c > >> We were previously relying only on whether the controller supported >> 8-bit, which doesn't tell us anything about the pin configuration in >> the board design. >> >> Signed-off-by: Philip Rakity <[email protected]> >> Tested-by: Giuseppe Cavallaro <[email protected]> >> Signed-off-by: Chris Ball <[email protected]> > > -- > Pengutronix e.K. | Wolfram Sang | > Industrial Linux Solutions | http://www.pengutronix.de/ | -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
