Re: [kbuild-all] Re: sound/soc/intel/catpt/loader.c:654 catpt_first_boot_firmware() warn: consider using resource_size() here

From: Andy Shevchenko
Date: Tue Nov 24 2020 - 04:12:39 EST


On Tue, Nov 24, 2020 at 10:06 AM Rong Chen <rong.a.chen@xxxxxxxxx> wrote:
> On 11/23/20 7:41 PM, Rojewski, Cezary wrote:
> > On 2020-11-23 11:53 AM, Andy Shevchenko wrote:
> >> On Sun, Nov 22, 2020 at 03:52:27AM +0800, kernel test robot wrote:

...

> >> This sounds like false positive. From where it gets the idea of resource_size()
> >> for the *start* offset?!
> >>
> > Indeed it is false positive. I've already explained this in:
> >
> > RE: [bug report] ASoC: Intel: catpt: Firmware loading and context restore
> > https://www.spinics.net/lists/alsa-devel/msg117145.html

> Thanks a lot, we'll ignore the warning next time.

I think the proper solution here is to notify smatch upstream to fix the tool.

--
With Best Regards,
Andy Shevchenko