Re: [PATCH v2 2/2] ARM: dts: am335x: add am335x-sancloud-bbe board support

[Koen Kooi]

> Op 12 jul. 2018, om 05:08 heeft Robert Nelson <robertcnelson@xxxxxxxxx> het volgende geschreven:
> 
> On Wed, Jul 11, 2018 at 8:31 AM, Tony Lindgren <tony@xxxxxxxxxxx> wrote:
>> * Koen Kooi <koen@xxxxxxxxxxxxxxxxxxxxx> [180711 07:36]:
>>> +&cpu0_opp_table {
>>> +     /*
>>> +      * All PG 2.0 silicon may not support 1GHz but some of the early
>>> +      * BeagleBone Blacks have PG 2.0 silicon which is guaranteed
>>> +      * to support 1GHz OPP so enable it for PG 2.0 on this board.
>>> +      */
>>> +     oppnitro-1000000000 {
>>> +             opp-supported-hw = <0x06 0x0100>;
>>> +     };
>>> +};
>> 
>> Is the above valid for new boards or should it be just
>> dropped? Or is the comment just out of date?
> 
> For SanCloud, this can be dropped, they got normal 1Ghz parts from TI.
> 
> and they aren't using the Octavo SIP which didn't have the 1Ghz efuse set..
> 
> So just the normal am335x errata, nothing special required in itâs dts...

Thanks for the info! Iâll send a v3 with that bit removed shortly after I figure out what trips up kbuild.

regards,

Koen