<div dir="ltr"><div dir="ltr"><br><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Sep 5, 2019 at 3:08 PM Christian Mauderer <<a href="mailto:christian.mauderer@embedded-brains.de">christian.mauderer@embedded-brains.de</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Sorry for the late reply. This mail slipped my attention.<br>
<br>
On 03/09/2019 08:22, Chris Johns wrote:<br>
> On 3/9/19 3:30 pm, Christian Mauderer wrote:<br>
>> On 03/09/2019 01:46, Chris Johns wrote:<br>
>>> On 2/9/19 5:42 pm, Vijay Kumar Banerjee wrote:<br>
>>>> On Mon, Sep 2, 2019 at 4:34 AM Chris Johns <<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a><br>
>>>> <mailto:<a href="mailto:chrisj@rtems.org" target="_blank">chrisj@rtems.org</a>>> wrote:<br>
>>>>     > +     puts("\nRTEMS I2C TEST\n");<br>
>>>>     > +     exit_code = bbb_register_i2c_0();<br>
>>>>     > +     assert(exit_code == 0);<br>
>>>><br>
>>>>     Is this needed for the display to work?<br>
>>>><br>
>>>> Yes. We need to register the rtems i2c device in order to work with the TDA driver<br>
>>>> as libbsd uses rtems i2c driver. The bbb_register_* is making it bbb specific, what<br>
>>>> do you suggest to make it more generic? <br>
>>><br>
>>> Should the I2C be registered during the BSP initalisation? This would remove the<br>
>>> need for this type of call being spread across all applications on the BBB. The<br>
>>> BBB has a lot of resources and the I2C is part of the SoC and so always present.<br>
>>><br>
>>> I cannot think of a way to have a sysinit entry that installs the driver when<br>
>>> the I2C bus used.<br>
>>><br>
>>> Chris<br>
>><br>
>> Hello Chris,<br>
>><br>
>> I would also prefer that the I2C is initialized during BSP init. But<br>
>> note that this opens the same discussion we had with Joel when I<br>
>> suggested the FDT support as a project: It links in the Support for<br>
>> every BSP. I don't really think it hurts for a small driver like I2C on<br>
>> a big target like BBB but we should make sure that everyone is OK with that.<br>
> <br>
> I suggest the I2C driver init is part of the BBB BSP and not part of every BSP.<br>
> I do not see this as the same as the FDT code then again I have not looked at<br>
> the detail.<br>
<br>
Clearly I used the wrong terms. Sorry for the confusion. I meant that it<br>
will link in the I2C support for every application regardless whether<br>
I2C is used or not. Joel mentioned that he would prefer it if not all<br>
drivers are linked in automatically during the FDT support discussion. See:<br>
<br>
<a href="https://lists.rtems.org/pipermail/devel/2019-August/027265.html" rel="noreferrer" target="_blank">https://lists.rtems.org/pipermail/devel/2019-August/027265.html</a><br>
<br>
Joel wrote there: "I think this is a good idea if we can still avoid<br>
bloating apps with all drivers."<br>
<br>
Like I said: I wouldn't mind a small driver like I2C but we should have<br>
all agree to it.<br>
<br>
> <br>
>> What might could be a problem: Is there something in the initialization<br>
>> that might is application specific? The still bad solution for pin<br>
>> config for example? (@Vijay: Bad because it's hard coded in I2C driver<br>
>> and not due to the double init.)<br>
> <br>
> Maybe I am not understanding the issue. I thought there is I2C code is present<br>
> in libbsd for the BBB frame buffer so when the frame buffer is initialised some<br>
> I2C activity happens. If the BBB I2C is not initialised the driver will fail or<br>
> the display will not be initialised and this is why the example has the code to<br>
> initialise the bus.<br>
<br>
Right.<br>
<br>
> <br>
> Examples should not contain BSP specific initialisations or we have to manage<br>
> specific builds of the examples and that is problematic.<br>
<br>
Fully agreed. Just not sure how that could be solved (without adding the<br>
I2C driver).<br>
<br></blockquote><div>Hi,</div><div><br></div><div>Since adding I2C in BBB initialization seems like the right solution, should we ask</div><div>in devel and user lists in a separate thread, to know if anyone is opposed to this change?</div><div><br></div><div>Where should I be looking for to add i2c in BBB initialization?</div><div><br></div><div>Regards,</div><div>Vijay <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> <br>
> Chris<br>
> <br>
<br>
Best regards<br>
<br>
Christian<br>
-- <br>
--------------------------------------------<br>
embedded brains GmbH<br>
Herr Christian Mauderer<br>
Dornierstr. 4<br>
D-82178 Puchheim<br>
Germany<br>
email: <a href="mailto:christian.mauderer@embedded-brains.de" target="_blank">christian.mauderer@embedded-brains.de</a><br>
Phone: +49-89-18 94 741 - 18<br>
Fax:   +49-89-18 94 741 - 08<br>
PGP: Public key available on request.<br>
<br>
Diese Nachricht ist keine geschäftliche Mitteilung im Sinne des EHUG.<br>
</blockquote></div></div>