8 May
2018
8 May
'18
2:45 a.m.
On Thu, Apr 19, 2018 at 12:14:01PM +0900, Masahiro Yamada wrote:
Putting zero length array at the end of struct is a common technique to embed arbitrary length of members. There is no good reason to let regmap_alloc_count() branch by "if (count <= 1)".
As far as I understood the code, regmap->base is an alias of regmap->ranges[0].start, but it is not helpful but make the code just ugly.
Rename regmap_alloc_count() to regmap_alloc() because the _count suffix seems pointless.
Signed-off-by: Masahiro Yamada yamada.masahiro@socionext.com Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot/master, thanks!
--
Tom