Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751827AbdC0FcG (ORCPT ); Mon, 27 Mar 2017 01:32:06 -0400 Received: from conssluserg-03.nifty.com ([210.131.2.82]:49002 "EHLO conssluserg-03.nifty.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751854AbdC0Fb5 (ORCPT ); Mon, 27 Mar 2017 01:31:57 -0400 DKIM-Filter: OpenDKIM Filter v2.10.3 conssluserg-03.nifty.com v2R5VM7x019060 X-Nifty-SrcIP: [209.85.161.178] MIME-Version: 1.0 In-Reply-To: <1489496093-15315-2-git-send-email-nicolas.dichtel@6wind.com> References: <1489496093-15315-1-git-send-email-nicolas.dichtel@6wind.com> <1489496093-15315-2-git-send-email-nicolas.dichtel@6wind.com> From: Masahiro Yamada Date: Mon, 27 Mar 2017 14:31:21 +0900 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH v10 01/11] h8300: put bitsperlong.h in uapi To: Nicolas Dichtel Cc: Arnd Bergmann , Michal Marek , Linux Kbuild mailing list , Linux Kernel Mailing List , linux-arch@vger.kernel.org, David Airlie , "David S. Miller" , Russell King , bp@alien8.de, slash.tmp@free.fr, Daniel Vetter , rmk+kernel@armlinux.org.uk, msalter@redhat.com, jengelh@inai.de, hch@infradead.org, Tobias Klauser , mpe@ellerman.id.au, Ingo Molnar , Thomas Gleixner , "Dmitry V. Levin" , Herbert Xu , linux-rdma@vger.kernel.org Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 933 Lines: 35 Hi Nicolas, 2017-03-14 21:54 GMT+09:00 Nicolas Dichtel : > This header file is exported, thus move it to uapi. > > Signed-off-by: Nicolas Dichtel > -#endif /* __ASM_H8300_BITS_PER_LONG */ > diff --git a/arch/h8300/include/uapi/asm/bitsperlong.h b/arch/h8300/include/uapi/asm/bitsperlong.h > new file mode 100644 > index 000000000000..e56cf72369b6 > --- /dev/null > +++ b/arch/h8300/include/uapi/asm/bitsperlong.h > @@ -0,0 +1,14 @@ > +#ifndef _UAPI_ASM_H8300_BITS_PER_LONG > +#define _UAPI_ASM_H8300_BITS_PER_LONG Just a minor comment. The include guard was originally __ASM_H8300_BITS_PER_LONG. This will change it into _ASM_H8300_BITS_PER_LONG (after _UAPI is stripped by headers_install.sh) I just thought _UAPI__ASM_H8300_BITS_PER_LONG might be even safer, but I know I am nit-picking. I can apply this as-is. I leave it to you. -- Best Regards Masahiro Yamada