Re: [PATCH] [v2] platform/surface: aggregator: avoid clang -Wconstant-conversion warning

Subject: Re: [PATCH] [v2] platform/surface: aggregator: avoid clang -Wconstant-conversion warning
From: Maximilian Luz <luzmaximilian@xxxxxxxxx>
Date: Mon, 17 May 2021 14:40:59 +0200
Cc: Arnd Bergmann <arnd@xxxxxxxx>, "platform-driver-x86@xxxxxxxxxxxxxxx" <platform-driver-x86@xxxxxxxxxxxxxxx>, Nick Desaulniers <ndesaulniers@xxxxxxxxxx>, "linux-kernel@xxxxxxxxxxxxxxx" <linux-kernel@xxxxxxxxxxxxxxx>, "clang-built-linux@xxxxxxxxxxxxxxxx" <clang-built-linux@xxxxxxxxxxxxxxxx>
In-reply-to: <16280aaedc40425295d202431ac3adc0@AcuMS.aculab.com>
References: <20210514200453.1542978-1-arnd@kernel.org> <057b5568-c4b8-820c-3dd7-2200f61a4d58@kernel.org> <16280aaedc40425295d202431ac3adc0@AcuMS.aculab.com>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1

On 5/17/21 10:31 AM, David Laight wrote:

From: Nathan Chancellor

Sent: 14 May 2021 22:23


Clang complains about the assignment of SSAM_ANY_IID to
ssam_device_uid->instance:


Has this been raised with clang?


I believe so, see this earlier report:

  https://lore.kernel.org/linux-mm/20210311185154.6uysryumfho73zlb@archlinux-ax161/

Regards,
Max

References:
- [PATCH] [v2] platform/surface: aggregator: avoid clang -Wconstant-conversion warning
  - From: Arnd Bergmann
- Re: [PATCH] [v2] platform/surface: aggregator: avoid clang -Wconstant-conversion warning
  - From: Nathan Chancellor
- RE: [PATCH] [v2] platform/surface: aggregator: avoid clang -Wconstant-conversion warning
  - From: David Laight