<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 10, 2016 at 12:19 AM, Joel Stanley <span dir="ltr"><<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, Oct 4, 2016 at 7:41 AM, Maxim Sloyko <<a href="mailto:maxims@google.com">maxims@google.com</a>> wrote:<br>
>> > +<br>
>> > +static const struct udevice_id ast_i2c_ids[] = {<br>
>> > +       {.compatible = "aspeed,ast2400-i2c-<wbr>controller",},<br>
>> > +       {.compatible = "aspeed,ast2400-i2c-bus",},<br>
>><br>
>> Your subject says g5, but your compatible strings say g4.<br>
>><br>
>> Your probe function doesn't test for which string you're using. Why do<br>
>> you have both -controller and -bus?<br>
><br>
><br>
> I think this has something to do with how the device tree is processed, but<br>
> it seems that because i2c buses are specified under a single controller<br>
> node, if I don't specify compatibility with -controller, it does not match<br>
> -bus nodes.<br>
<br>
</span>I don't follow?<br>
<br>
The hardware has a few registers that are common across many busses.<br>
The device tree describes them in the controller node, with the buses<br>
being children of this node.<br>
<br>
Cheers,<br>
<br>
Joel<br></blockquote><div><br></div><div>I don't use them in this driver. At least not directly.</div><div><br></div><div>What are you proposing? </div></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div><b>M</b>axim <b>S</b>loyko</div></div>
</div></div>