From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chen Gang Subject: Re: [PATCH v2] arch: use ASM_NL instead of ';' for assembler new line character in the macro Date: Tue, 29 Oct 2013 14:07:58 +0800 Message-ID: <526F50BE.6020505@asianux.com> References: <526E3CE2.2070405@asianux.com> <526E6A92.6090301@synopsys.com> <526F09B8.9030202@asianux.com> <526F11C0.1030809@asianux.com> <526F149A.3000306@asianux.com> <526F4DCF.5010406@synopsys.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Return-path: Received: from intranet.asianux.com ([58.214.24.6]:37081 "EHLO intranet.asianux.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751330Ab3J2GJA (ORCPT ); Tue, 29 Oct 2013 02:09:00 -0400 In-Reply-To: <526F4DCF.5010406@synopsys.com> Sender: linux-arch-owner@vger.kernel.org List-ID: To: Vineet Gupta Cc: James Hogan , Al Viro , Arnd Bergmann , "linux-kernel@vger.kernel.org" , Linux-Arch , Max Filippov On 10/29/2013 01:55 PM, Vineet Gupta wrote: > On 10/29/2013 07:21 AM, Chen Gang wrote: >> For some assemblers, they use another character as newline in a macro >> (e.g. arc uses '`'), so for generic assembly code, need use ASM_NL (a >> macro) instead of ';' for it. >> >> Also need notice about code styles ('\t' for each line). >> >> >> Signed-off-by: Chen Gang >> --- >> arch/arc/include/asm/linkage.h | 2 ++ >> include/linux/linkage.h | 19 ++++++++++++------- > > Acked-by: Vineet Gupta > Thank you very much! -- Chen Gang