Re: [PATCH -next] soundwire: intel: Make sdw_intel_init static

From: Nathan Chancellor
Date: Thu Apr 23 2020 - 14:03:29 EST


On Mon, Apr 20, 2020 at 12:42:12PM +0530, Vinod Koul wrote:
> On 10-04-20, 19:57, YueHaibing wrote:
> > Fix sparse warning:
> >
> > drivers/soundwire/intel_init.c:193:6: warning:
> > symbol 'sdw_intel_init' was not declared. Should it be static?
>
> Applied, thanks
>
> --
> ~Vinod

Why was this applied? It replaces one warning with another (that is
actually visible during a normal kernel build):

$ make -j$(nproc) -s allyesconfig drivers/soundwire/intel_init.o
drivers/soundwire/intel_init.c:193:14: warning: âsdw_intel_initâ defined
but not used [-Wunused-function]
static void *sdw_intel_init(acpi_handle *parent_handle,
^~~~~~~~~~~~~~

Cheers,
Nathan