<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 6, 2015 at 6:11 PM, Michal Simek <span dir="ltr"><<a href="mailto:michal.simek@xilinx.com" target="_blank">michal.simek@xilinx.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 05/06/2015 10:40 AM, Arnd Bergmann wrote:<br>
> On Wednesday 06 May 2015 09:45:15 Michal Simek wrote:<br>
>> On 05/06/2015 09:31 AM, Arnd Bergmann wrote:<br>
>>> On Wednesday 06 May 2015 10:41:07 Suman Tripathi wrote:<br>
>>>>>> @@ -533,6 +567,16 @@<br>
>>>>>>                         interrupts = <0x0 0x4f 0x4>;<br>
>>>>>>                 };<br>
>>>>>><br>
>>>>>> +               sdhc0: sdhc@1c000000 {<br>
>>>>>> +                       device_type = "sdhc";<br>
>>>>><br>
>>>>> device_type generally should not be used (there are a few exceptions).<br>
>>>><br>
>>>> Okay !!<br>
>>>><br>
>>><br>
>>> While we're at it, please change sdhc@1c000000 to mmc@1c000000.<br>
>>> Even though Linux does not care, we try to use the standard device<br>
>>> names for consistency.<br>
>><br>
>> Do we have a list of these names somewhere?<br>
>> Normally I do use ePARP - generic names recommendation but mmc or sdhci<br>
>> are not listed there.<br>
>> Both combination mmc@ or sdhci@ are used in the kernel.<br>
>><br>
>> On zynq and zynqmp we do use shdci@.<br>
>><br>
><br>
> Ah, I thought ePAPR listed mmc already. Using "sdhci" is a little too<br>
> specific here, since a lot of mmc hosts are not sdhci compliant, and<br>
> "sdhc" is completely wrong, because that identifies a specific card<br>
> type, but a host that supports SDHC cards will generally also work<br>
> with SD (less than 4GB) or SDXC (more than 48GB) cards.<br>
<br>
</div></div>Yes "sdhc" is completely wrong.<br></blockquote><div><br></div><div>But spec name in search engine's gives  SDHC 3.0 as general. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Based on our datasheet(also version used on Zynq and ZynqMP) this IP is<br>
compliant with SD HC 3.00, SDIO 3.0, SD MC 3.01 SD MCS 1.01, MMC 4.51.<br>
Not sure about the version which they use.<br>
Also not sure which spec the IP should have to be able to say that we<br>
can use sdhci name. Do you have exact SPEC name?<br></blockquote><div><br></div><div>I also think sdhci because the binding is sdhci written by Arasan. Anyway I will change to sdhci.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
Michal<br>
<br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr"><div>Thanks,</div><div>with regards,</div>Suman Tripathi</div></div>
</div></div>