Re: [PATCH] gpio: aspeed: fix compile testing warning

From: Linus Walleij
Date: Fri Jul 13 2018 - 03:06:03 EST


On Mon, Jul 9, 2018 at 4:56 PM Arnd Bergmann <arnd@xxxxxxxx> wrote:

> Gcc cannot always see that BUG_ON(1) is guaranteed to not
> return, so we get a warning message in some configurations:
>
> drivers/gpio/gpio-aspeed.c: In function 'bank_reg':
> drivers/gpio/gpio-aspeed.c:244:1: error: control reaches end of non-void function [-Werror=return-type]
>
> Using a plain BUG() is easier here and avoids the problem.
>
> Fixes: 44ddf559d579 ("gpio: aspeed: Rework register type accessors")
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>

Patch applied with Benjamin's ACK.

Yours,
Linus Walleij