On Wed, Sep 16, 2020 at 2:58 AM Barnabás Pőcze <pobrn@xxxxxxxxxxxxxx> wrote:
...
> > + dev_err(dev, "failed to set GPE wake mask: %d\n", status);
>
> I'm not sure if it's technically safe to print acpi_status with the %d format
> specifier since 'acpi_status' is defined as 'u32' at the moment.
> func("%lu", (unsigned long) status)
> would be safer.
Please, no explicit castings for printf(). In 99% cases it means that
something is not being used correctly.
--
With Best Regards,
Andy Shevchenko
