On Thu, Jul 19, 2018 at 03:07:55PM +0200, Marek Vasut wrote: > 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 <k...@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.
Ah, alright. So yeah, as people want to convert areas, that's fine. But no, just like how we also stay consistent within a file on other things, or fix those inconsistencies then add more stuff, no, we shouldn't have this be inconsistent. Fixing it in either order is fine with me. Thanks! -- Tom
signature.asc
Description: PGP signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
