<html><head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="Generator" content="Microsoft Exchange Server">
<!-- converted from text -->
<style><!-- .EmailQuote { margin-left: 1pt; padding-left: 4pt; border-left: #800000 2px solid; } --></style></head>
<body>
<style>
<!--
.EmailQuote
{margin-left:1pt;
padding-left:4pt;
border-left:#800000 2px solid}
-->
</style><body><span style="font-family:Arial">Reviewed-by. Philip Rakity <prakity@nvidia.com><br><br><div id="htc_header" style="">----- Reply message -----<br>From: "Daniel Drake" <dsd@laptop.org><br>To: "Philip Rakity" <prakity@nvidia.com><br>Cc: "cjb@laptop.org" <cjb@laptop.org>, "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>, "jrudholm@gmail.com" <jrudholm@gmail.com>, "devicetree-discuss@lists.ozlabs.org" <devicetree-discuss@lists.ozlabs.org><br>Subject: [PATCH 1/2] sdhci: add quirk for lack of 1.8v support<br>Date: Sun, Nov 25, 2012 18:50<br><br></div></span><br><br>
<DIV>
<HR>
</DIV>
<DIV>This email message is for the sole use of the intended recipient(s) and may
contain confidential information. Any unauthorized review, use, disclosure
or distribution is prohibited. If you are not the intended recipient,
please contact the sender by reply email and destroy all copies of the original
message. </DIV>
<DIV>
<HR>
</DIV>
<P></P>
</body>
<font size="2"><div class="PlainText">On Sun, Nov 25, 2012 at 12:36 PM, Philip Rakity <prakity@nvidia.com> wrote:<br>
> The same effect can be achieved by setting the quirk<br>
> SDHCI_QUIRK_MISSING_CAPS<br>
> reading caps[0] and removing from caps[1] the UHS values in the controller specific code for your board.<br>
><br>
> What is the reason you cannot do this ?<br>
<br>
The same reasons we already touched upon in the other thread.<br>
<a href="http://marc.info/?l=linux-mmc&m=135240077826454&w=2">http://marc.info/?l=linux-mmc&m=135240077826454&w=2</a><br>
<a href="http://marc.info/?l=linux-mmc&m=135240181826846&w=2">http://marc.info/?l=linux-mmc&m=135240181826846&w=2</a><br>
<br>
To reiterate (I know this has been broken out over a couple of weeks):<br>
<br>
There is no controller-specific or board-specific code. We are working<br>
with the device tree. It must be done generically.<br>
<br>
SDHCI_QUIRK_MISSING_CAPS seems to be aimed at cases where the<br>
controller does not report caps, or it is lying about the caps it<br>
does/doesn't support. In this case our controller is being truthful,<br>
so this does not feel appropriate.<br>
<br>
Also, the only realistic way that I can see to combine<br>
SDHCI_QUIRK_MISSING_CAPS with the device tree would be to put the<br>
caps/caps1 override data in the device tree. This just seems ugly, and<br>
feels like it is taking the wrong approach of again suggesting that<br>
the controller is providing bad data, when in this case actually it is<br>
correctly saying that 1.8v is supported (by the controller - it<br>
obviously cannot speak for external components).<br>
<br>
We don't have a regulator to work with here.<br>
<br>
Thanks<br>
Daniel<br>
</div></font>
</body>
</html>