[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20210419165659.GA2363@ashish-NUC8i5BEH>
Date: Mon, 19 Apr 2021 22:26:59 +0530
From: Ashish Kalra <eashishkalra@...il.com>
To: Jérôme Pouiller <jerome.pouiller@...abs.com>
Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org,
Ashish Kalra <eashishkalra@...il.com>
Subject: Re: [PATCH] staging: wfx: silence symbol 'wfx_get_ps_timeout' was
not declared warning
On Mon, Apr 19, 2021 at 05:55:34PM +0200, Jérôme Pouiller wrote:
> On Monday 19 April 2021 17:33:48 CEST Ashish Kalra wrote:
> >
> > Upon running sparse, "warning: symbol 'wfx_get_ps_timeout' was not declared.
> > Should it be static?" and "warning: symbol 'wfx_update_pm' was not declared.
> > Should it be static?" is brought to notice for this file. static keyword
> > should be added to prevent this warning. let's add it to make it cleaner and
> > silence the Sparse warning.
>
> Hi Ashish,
>
> Thank you for your contribution.
>
> It seems that this issue is already fixed by commit ce59858bbc10 "staging:
> wfx: make methods 'wfx_get_ps_timeout' and 'wfx_update_pm' static" (merged
> in master in version 5.8). Can you check you are working on the last tree?
>
>
> --
> Jérôme Pouiller
>
>
Thanks Jerome
It seems some sync issue with my source code.I will update to
latest and correct
Powered by blists - more mailing lists