On Sun, May 16, 2021 at 11:04 AM Geert Uytterhoeven
<geert@xxxxxxxxxxxxxx> wrote:
On Sun, May 16, 2021 at 6:24 AM Finn Thain <fthain@xxxxxxxxxxxxxxxxxxx> wrote:
On Sun, 16 May 2021, Arnd Bergmann wrote:
From: Arnd Bergmann <arnd@xxxxxxxx>
This driver always warns about unused functions because it includes
an file that it doesn't actually need:
I don't think you can omit #include "lib8390.c" here without changing
driver behaviour, because of the macros in effect.
I think this change would need some actual testing unless you can show
that the module binary does not change.
Michael posted a similar but different patch a while ago, involving
calling ax_NS8390_reinit():
https://lore.kernel.org/linux-m68k/1528604559-972-3-git-send-email-schmitzmic@xxxxxxxxx/
Ah nice. As far as I can tell, the two versions are functionally equivalent
based on my reading of the port accessors, but he probably tested his
version, so I'll drop mine from this series. Nothing else depends on this
one, I just included it since it was an obvious thing to fix.
Arnd
