<html>
  <head>
    <meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">On 08/23/2013 11:40 PM, Scott Wood
      wrote:<br>
    </div>
    <blockquote cite="mid:1377272410.20722.46.camel@snotra.buserror.net"
      type="cite">
      <pre wrap="">On Fri, 2013-08-23 at 14:39 +0800, Zhang Haijun wrote:
</pre>
      <blockquote type="cite">
        <pre wrap="">Hi, Anton and all

Is there any advice on these two patches ?

[PATCH 2/4 V2] mmc: esdhc: workaround for dma err in the last system 
transaction
[PATCH 3/4 V3] mmc: esdhc: Correct host version of T4240-R1.0-R2.0.


[PATCH 1/4 V4] powerpc/85xx: Add support for 85xx cpu type detection
This patch is Act-by Scott.
Patch 4/4 is split to four patches and Act-by Anton.


Thanks all.



</pre>
      </blockquote>
      <pre wrap="">[snip]
</pre>
      <blockquote type="cite">
        <blockquote type="cite">
          <blockquote type="cite">
            <pre wrap="">+      if (!(((SVR_SOC_VER(svr) == SVR_T4240) && (SVR_REV(svr) == 0x10))
||
+               ((SVR_SOC_VER(svr) == SVR_B4860) && (SVR_REV(svr) == 0x10))
||
+               ((SVR_SOC_VER(svr) == SVR_P1010) && (SVR_REV(svr) == 0x10))
||
+               ((SVR_SOC_VER(svr) == SVR_P3041) && (SVR_REV(svr) <= 0x20))
||
+               ((SVR_SOC_VER(svr) == SVR_P2041) && (SVR_REV(svr) <= 0x20))
||
+               ((SVR_SOC_VER(svr) == SVR_P5040) && SVR_REV(svr) == 0x20)))
+               return;
</pre>
          </blockquote>
        </blockquote>
      </blockquote>
      <pre wrap="">
You need to include variants here.  If P5040 is affected, then P5021 is
affected.  If P2041 is affected, then P2040 is affected, etc.

-Scott


</pre>
    </blockquote>
    <font size="-1">Hi, Scott<br>
      <br>
      <font size="-1"><font size="-1">This <font size="-1">worka<font
              size="-1">round is for </font></font></font></font></font><font
      size="-1"><font size="-1"><font size="-1"><font size="-1"><font
              size="-1">CR:ENGR00229586: A-005055<font size="-1">,
                Config<font size="-1">s Affected only<font size="-1">
                    list th<font size="-1">e<font size="-1">se <font
                          size="-1">soc and its version</font>.<br>
                        <font size="-1">I was also wo<font size="-1">nder
                            <font size="-1">why only these boards<font
                                size="-1">?<br>
                                <br>
                                <font size="-1"><font size="-1">But <font
                                      size="-1">I<font size="-1"> can't
                                        add s<font size="-1">oc </font>like
                                        P5021 as <font size="-1">I
                                          think it should be. Maybe
                                          there are some <font
                                            size="-1">difference <font
                                              size="-1">between them.</font></font></font></font></font></font></font><br>
                              </font></font></font></font></font></font></font></font></font></font></font></font></font></font><br>
    <pre class="moz-signature" cols="72">-- 
Thanks & Regards

Haijun</pre>
  </body>
</html>