<br><tt><font size=2>Adrian Reber &lt;adrian@lisas.de&gt; wrote on 24.03.2009
13:31:31:<br>
&gt; On Tue, Mar 24, 2009 at 05:30:41PM +1100, Mark Nelson wrote:<br>
&gt; &gt; The buffer passed to the ibm,get-system-parameter RTAS call must
be<br>
&gt; &gt; in the RMA. To ensure we pass an address in the RMA use rtas_data_buf<br>
&gt; &gt; for the actual RTAS call and then copy the result to value. We
can't<br>
&gt; &gt; just make it static because this can be compiled in as a module.<br>
&gt; &gt; <br>
&gt; &gt; Also add the WDRTAS_SP_SPI_LEN so we don't litter '4' throughout
the<br>
&gt; &gt; function.<br>
&gt; &gt; <br>
&gt; &gt; Signed-off-by: Mark Nelson &lt;markn@au1.ibm.com&gt;<br>
&gt; &gt; ---<br>
&gt; &gt; <br>
&gt; &gt; Adrian, does this patch cause any problems for your pxcabs?<br>
&gt; <br>
&gt; No, it even helps. I have no tried the watchdog until now, but without
the<br>
&gt; patch I get: <br>
&gt; <br>
&gt; &nbsp; &nbsp;wdrtas: could not get sp_spi watchdog timeout (0). Continuing<br>
&gt; <br>
&gt; and with the patch it reads the correct value. So only with your patch<br>
&gt; it works like it is supposed to. Thanks!<br>
&gt; <br>
&gt; Tested-by: Adrian Reber &lt;adrian@lisas.de&gt;</font></tt>
<br>
<br><tt><font size=2>looks good to me.</font></tt>
<br>
<br><tt><font size=2>Acked-by: Utz Bacher &lt;utz.bacher@de.ibm.com&gt;</font></tt>
<br>
<br><tt><font size=2>Utz</font></tt>
<br>
<br><tt><font size=2>:wq</font></tt>