On Tue, Feb 11, 2020 at 9:32 AM Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote:
>
> On 2/10/20 6:00 PM, Stephen Rothwell wrote:
> > Hi all,
> >
> > Changes since 20200210:
> >
>
> on i386:
>
> Function args are reversed (offset and status);
>
> ../drivers/platform/x86/intel_pmc_core.c: In function 'pmc_core_resume':
> ../drivers/platform/x86/intel_pmc_core.c:1329:43: warning: passing argument 4 of 'pmc_core_lpm_display' makes integer from pointer without a cast [-Wint-conversion]
> pmc_core_lpm_display(pmcdev, dev, NULL, "STATUS", offset, maps);
> ^~~~~~~~
> ../drivers/platform/x86/intel_pmc_core.c:977:13: note: expected 'u32 {aka unsigned int}' but argument is of type 'char *'
> static void pmc_core_lpm_display(struct pmc_dev *pmcdev, struct device *dev,
> ^~~~~~~~~~~~~~~~~~~~
> ../drivers/platform/x86/intel_pmc_core.c:1329:53: warning: passing argument 5 of 'pmc_core_lpm_display' makes pointer from integer without a cast [-Wint-conversion]
> pmc_core_lpm_display(pmcdev, dev, NULL, "STATUS", offset, maps);
> ^~~~~~
> ../drivers/platform/x86/intel_pmc_core.c:977:13: note: expected 'const char *' but argument is of type 'int'
> static void pmc_core_lpm_display(struct pmc_dev *pmcdev, struct device *dev,
> ^~~~~~~~~~~~~~~~~~~~
Thank you, it should be fixed in today's Linux Next.
--
With Best Regards,
Andy Shevchenko
