<div dir="ltr">Hi Arnd,<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 6, 2015 at 5:27 PM, Suman Tripathi <span dir="ltr"><<a href="mailto:stripathi@apm.com" target="_blank">stripathi@apm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Wed, May 6, 2015 at 2:10 PM, Arnd Bergmann <span dir="ltr"><<a href="mailto:arnd@arndb.de" target="_blank">arnd@arndb.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div>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>
</div></div>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></blockquote><div><br></div></div></div><div>Agree on this . Will change it.  </div></div></div></div></blockquote><div><br></div><div>One more point as we are resuing the arasan driver,  is it compulsory to use the name used in binding info</div><div>for arasan ?? It is sdhci for arasan. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span><font color="#888888"><br>
        Arnd<br>
</font></span></blockquote></div><span class=""><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr"><div>Thanks,</div><div>with regards,</div>Suman Tripathi</div></div>
</span></div></div>
</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>