Re: [PATCH 1/2] mfd: ab8500-debugfs: add function declaration
From: Arnd Bergmann
Date: Sun Sep 25 2016 - 19:30:11 EST
On Sunday 25 September 2016, Baoyou Xie wrote:
> On 24 September 2016 at 16:30, Arnd Bergmann <arnd@xxxxxxxx> wrote:
> > This can't be right for two reasons:
> > - the suspend_test_wake_cause_interrupt_is_mine function again
> > is only used in the file it is defined in
> but, it is declared as weak, so other modules can implement it.
In theory that is correct, but doing that would be very unusual
and bad style. As nobody does this at the moment, and the driver is for
obsolete hardware, just remove it now, and if we ever need want to
override it, that should be done using a runtime callback handler
or something completely different.