Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751013AbdGZGwN (ORCPT ); Wed, 26 Jul 2017 02:52:13 -0400 Received: from mail-oi0-f66.google.com ([209.85.218.66]:37384 "EHLO mail-oi0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750813AbdGZGwL (ORCPT ); Wed, 26 Jul 2017 02:52:11 -0400 MIME-Version: 1.0 In-Reply-To: References: <20170726025717.lob7jlrb6ipa6rel@latitude> From: Arnd Bergmann Date: Wed, 26 Jul 2017 08:52:10 +0200 X-Google-Sender-Auth: yUvZGst7m-syaQ4tikt2hx2jwTk Message-ID: Subject: Re: [patches] [PATCH 17/17] RISC-V: Build Infastructure To: Palmer Dabbelt Cc: "nathan=20Neusch=C3=A4fer?=" , patches@groups.riscv.org, Olof Johansson , Andrew Morton , albert@sifive.com, Masahiro Yamada , Michal Marek , Will Deacon , Peter Zijlstra , Boqun Feng , Ingo Molnar , Daniel Lezcano , Thomas Gleixner , Jason Cooper , Marc Zyngier , gregkh , Jiri Slaby , David Miller , Mauro Carvalho Chehab , Stephen Rothwell , =?UTF-8?B?RnLDqWTDqXJpYyBXZWlzYmVja2Vy?= , Al Viro , "Luis R. Rodriguez" , Doug Ledford , Bart Van Assche , sstabellini@kernel.org, Daniel Vetter , Michael Ellerman , Mark Salter , Nicolas Dichtel , James Hogan , Paul Gortmaker , Guenter Roeck , Heiko Carstens , Martin Schwidefsky , Linux Kernel Mailing List 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: 1202 Lines: 27 On Wed, Jul 26, 2017 at 7:20 AM, Palmer Dabbelt wrote: > On Tue, 25 Jul 2017 19:57:17 PDT (-0700), j.neuschaefer@gmx.net wrote: >> On Tue, Jul 11, 2017 at 06:31:30PM -0700, Palmer Dabbelt wrote: >>> This patch contains all the build infastructure that actually enables >>> the RISC-V port. This includes Makefiles, linker scripts, and Kconfig >>> files. It also contains the only top-level change, which adds RISC-V to >>> the list of architectures that need a sed run to produce the ARCH >>> variable when building locally. >>> >>> Signed-off-by: Palmer Dabbelt >>> --- >> [...] >>> +config ISA_C >>> + bool "Emit compressed instructions when building Linux" >> >> As a user, I'd prefer to have slightly more globally-recognisable names >> than ISA_ for RISC-V instruction set architecture >> extensions. A quick "git grep -A2 'config ISA'" shows the following >> Kconfig symbols: > > Good timing: I was about to submit a v6 patch set. I'm cool with > CONFIG_ISA_RVC and friends, do you mind submitting a patch? I'd generally prefix those highly architecture specific ones with the architecture name, e.g. CONFIG_RISCV_ISA_C Arnd