Hi Anton<div>    Thanks for helping me to amend these patch.</div><div>    I got many good suggestions and learn a lot in this process:)</div><div>    BTW, you mean you will help to amend that several minor warning with battery tree? </div>
<div>    If there is something wrong, I can correct it.</div><div><br></div><div>Thanks  </div><div><br></div><div>    <br><br><div class="gmail_quote">2012/8/23 Anton Vorontsov <span dir="ltr"><<a href="mailto:anton.vorontsov@linaro.org" target="_blank">anton.vorontsov@linaro.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Aug 22, 2012 at 08:48:15PM -0700, Anton Vorontsov wrote:<br>
[....]<br>
<div class="im">> drivers/mfd/88pm860x-core.c:803:53: warning: incorrect type in assignment (different base types)<br>
> drivers/mfd/88pm860x-core.c:803:53:    expected struct charger_regulator *charger_regulators<br>
> drivers/mfd/88pm860x-core.c:803:53:    got struct regulator_bulk_data static [toplevel] *<br>
><br>
> They are minor, except for the last one. You seemed to use<br>
> 'regulator_bulk_data' struct (just as charger manager documentation<br>
> wrongly tells you, yup), but in real it should have been<br>
> 'struct charger_regulator'. The only reason that it worked is<br>
> because both 'supply' and 'regulator_name' struct members are the<br>
> first in these structs. :-)<br>
<br>
</div>Oh, actually, I see that this is a result of extcon rework, so not<br>
your fault at all.<br>
<span class="HOEnZb"><font color="#888888"><br>
Anton.<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div> </div>
<div>----------------------------------</div>
<div>Best Regards</div>
<div>Jett Zhou</div>
<div> </div><br>
</div>