From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Fri, 3 May 2019 16:27:18 +0200 From: Heiko Carstens Subject: Re: [PATCH v2 4/4] s390: fix clang -Wpointer-sign warnigns in boot code References: <20190415083605.2560074-1-arnd@arndb.de> <20190415083605.2560074-4-arnd@arndb.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20190415083605.2560074-4-arnd@arndb.de> Message-Id: <20190503142718.GF5602@osiris> Sender: linux-kernel-owner@vger.kernel.org List-Archive: List-Post: To: Arnd Bergmann Cc: Martin Schwidefsky , clang-built-linux@googlegroups.com, Nick Desaulniers , Nathan Chancellor , linux-s390@vger.kernel.org, Vasily Gorbik , Christian Borntraeger , Janosch Frank , Collin Walling , linux-kernel@vger.kernel.org List-ID: On Mon, Apr 15, 2019 at 10:35:54AM +0200, Arnd Bergmann wrote: > The arch/s390/boot directory is built with its own set of compiler > options that does not include -Wno-pointer-sign like the rest of > the kernel does, this causes a lot of harmess but correct warnings > when building with clang. > > For the atomics, we can add type casts to avoid the warnings, for > everything else the easiest way is to slightly adapt the types > to be more consistent. > > Signed-off-by: Arnd Bergmann > --- > arch/s390/boot/ipl_parm.c | 2 +- > arch/s390/include/asm/bitops.h | 12 ++++++------ > arch/s390/include/asm/ebcdic.h | 2 +- > arch/s390/include/asm/lowcore.h | 2 +- > drivers/s390/char/sclp.h | 4 ++-- > 5 files changed, 11 insertions(+), 11 deletions(-) Applied, with typo fix and a simple coding style issue. Thanks!