lkml.org 
[lkml]   [2023]   [Feb]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: linux-next: build failure after merge of the iio tree
    On Thu, 9 Feb 2023 14:38:21 +1100
    Stephen Rothwell <sfr@canb.auug.org.au> wrote:

    > Hi all,
    >
    > After merging the iio tree, today's linux-next build (powerpc
    > allyesconfig) failed like this:
    >
    > drivers/iio/proximity/sx_common.c: In function 'sx_common_get_raw_register_config':
    > drivers/iio/proximity/sx_common.c:438:55: error: implicit declaration of function 'acpi_device_hid'; did you mean 'dmi_device_id'? [-Werror=implicit-function-declaration]
    > 438 | snprintf(prop, ARRAY_SIZE(prop), "%s,reg_%s", acpi_device_hid(adev), reg_def->property);
    > | ^~~~~~~~~~~~~~~
    > | dmi_device_id
    > drivers/iio/proximity/sx_common.c:438:44: error: format '%s' expects argument of type 'char *', but argument 4 has type 'int' [-Werror=format=]
    > 438 | snprintf(prop, ARRAY_SIZE(prop), "%s,reg_%s", acpi_device_hid(adev), reg_def->property);
    > | ~^ ~~~~~~~~~~~~~~~~~~~~~
    > | | |
    > | char * int
    > | %d
    > cc1: all warnings being treated as errors
    >
    > Caused by commit
    >
    > 91af1d6c497b ("iio: proximity: sx_common: Add old register mapping")
    >
    > I have reverted that commit for today.
    >

    Thanks, I misdiagnosed the issue in previous attempt to fix this.
    Problem is lack of stub for that function in !CONFIG_ACPI
    (not as I thought a missing include - which was also the case).

    As that will need an ACPI related header change to avoid ifdef nastiness
    in the driver I'll drop the patch from my tree and we can get back
    to this next cycle.

    Thanks again,

    Jonathan



    \
     
     \ /
      Last update: 2023-03-27 00:16    [W:4.177 / U:0.032 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site