Re: [PATCH] sdhci: 8 bit widths - allow new QUIRK for 8 bit and v3 sd controller

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Nov 8, 2010 at 4:21 PM, Philip Rakity <prakity@xxxxxxxxxxx> wrote:
>
> On Nov 7, 2010, at 2:04 PM, Wolfram Sang wrote:
>
>> On Fri, Oct 08, 2010 at 12:31:34PM -0700, Philip Rakity wrote:
>>>
>>> 8 bit width is a new feature in the sd 3.0 spec.  Add checks to
>>> ensure HOST_CONTROL register set 8 bit mode only if sd 3.0
>>>
>>> add quirk to indicate that board design supports 8 data lines.
>>> The controller can support 8 data bits but the board design may
>>> only bring out 4 data bits.
>>
>> This isn't a quirk, this is platform_data, no?
>
> Never sure of the difference between a quirk and platform data.
> example:  Broken card detect is a quirk but maybe some slots on the board work and some do not

In my case, one slot uses the 8-bit but another is 4-bit so host quirk
is not acceptable.
I prefer the platform data also.

Thank you,
Kyungmin Park

>
>>
>> Without looking deeply into it, I think we need to use the callback you
>> introduced in the other patch?
>>
>
> It is needed in my experience because of how board design is done.  The controller can support 8 bits
> but the board may not have DAT4-DAT7 connected.
>
>> Regards,
>>
>>   Wolfram
>>
>> PS: Sorry if there already was a discussion I might have missed. The bus width
>> thingie became a bit scattered, so I lost track...
>>
>> --
>> 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 majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux