Re: [PATCH] usb: ohci-omap: avoid unused-variable warning

From: Alan Stern
Date: Wed Jan 18 2023 - 11:13:10 EST


On Wed, Jan 18, 2023 at 09:27:34AM +0100, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@xxxxxxxx>
>
> The dead code removal has led to 'need_transceiver' not being
> used at all when OTG support is disabled:
>
> drivers/usb/host/ohci-omap.c: In function 'ohci_omap_reset':
> drivers/usb/host/ohci-omap.c:99:33: error: unused variable 'need_transceiver' [-Werror=unused-variable]
> 99 | int need_transceiver = (config->otg != 0);
>
> Change the #ifdef check into an IS_ENABLED() check to make the
> code more readable and let the compiler see where it is used.
>
> Fixes: 8825acd7cc8a ("ARM: omap1: remove dead code")
> Signed-off-by: Arnd Bergmann <arnd@xxxxxxxx>
> ---

Acked-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>

> The patch that caused the issue is in the boardfile-removal branch
> of the soc tree. I would just add the patch there.