linux-kernel - Re: linux-next: build failure after merge of the iio tree

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20230209102007.00002d07@Huawei.com>
Date:   Thu, 9 Feb 2023 10:20:07 +0000
From:   Jonathan Cameron <Jonathan.Cameron@...wei.com>
To:     Stephen Rothwell <sfr@...b.auug.org.au>
CC:     Gwendal Grignou <gwendal@...omium.org>,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
        Linux Next Mailing List <linux-next@...r.kernel.org>
Subject: Re: linux-next: build failure after merge of the iio tree

On Thu, 9 Feb 2023 14:38:21 +1100
Stephen Rothwell <sfr@...b.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