Re: [PATCH net-next v2 0/3] net: Allow more drivers with COMPILE_TEST

From: David Miller
Date: Wed May 16 2018 - 14:34:27 EST


From: Florian Fainelli <f.fainelli@xxxxxxxxx>
Date: Wed, 16 May 2018 11:52:55 -0700

> This patch series includes more drivers to be build tested with COMPILE_TEST
> enabled. This helps cover some of the issues I just ran into with missing
> a driver *sigh*.
>
> Changes in v2:
>
> - allow FEC to build outside of CONFIG_ARM/ARM64 by defining a layout of
> registers, this is not meant to run, so this is not a real issue if we
> are not matching the correct register layout

Ok, this is a lot better.

But man, some of these drivers...

drivers/net/ethernet/ti/davinci_cpdma.c: In function ʽcpdma_desc_pool_destroyʼ:
drivers/net/ethernet/ti/davinci_cpdma.c:194:7: warning: format ʽ%dʼ expects argument of type ʽintʼ, but argument 2 has type ʽsize_t {aka long unsigned int}ʼ [-Wformat=]
"cpdma_desc_pool size %d != avail %d",
^
gen_pool_size(pool->gen_pool),
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

and on and on and on...

But I'm really happy to see FEC and others at least being build tested
in more scenerios.

So applied, thanks.