[U-Boot] [PATCH v2] Add Beaglebone Enhanced support

Marek Vasut marex at denx.de
Thu Jul 19 13:07:55 UTC 2018


On 07/19/2018 03:02 PM, Tom Rini wrote:
> On Thu, Jul 19, 2018 at 02:55:37PM +0200, Marek Vasut wrote:
>> On 07/18/2018 10:13 AM, Koen Kooi wrote:
>>> The "Beaglebone Enhanced" by Sancloud is based on the Beaglebone Black,
>>> but with the following differences:
>>>
>>>  * Gigabit capable PHY
>>>  * Extra USB hub, optional i2c control
>>>  * lps3331ap barometer connected over i2c
>>>  * MPU6050 6 axis MEMS accelerometer/gyro connected over i2c
>>>  * 1GiB DDR3 RAM
>>>  * RTL8723 Wifi/Bluetooth connected over USB
>>>
>>> Signed-off-by: Koen Kooi <koen at dominion.thruhere.net>
>>>
>>> ---
>>>
>> [...]
>>
>>> diff --git a/board/ti/am335x/board.h b/board/ti/am335x/board.h
>>> index 652b10b..48df914 100644
>>> --- a/board/ti/am335x/board.h
>>> +++ b/board/ti/am335x/board.h
>>> @@ -43,9 +43,15 @@ static inline int board_is_bbg1(void)
>>>  	return board_is_bone_lt() && !strncmp(board_ti_get_rev(), "BBG1", 4);
>>>  }
>>>  
>>> +static inline int board_is_bben(void)
>>
>> Should be just static int ... the compiler can decide.
> 
> No, it should be inline like all of the others in the file are.  And
> then yes, if someone wants to do a size comparison with or without
> inlining on all of those functions, sure.

Can you explain why it should be static inline ?
Last time this discussion came up in Linux kernel ML (decisions from
which we seem to apply, cfr the SPDX C++ comments and DTC), static
inline was frowned upon with the argument that the compiler can decide.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list