From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933774AbdBQLcI (ORCPT ); Fri, 17 Feb 2017 06:32:08 -0500 Received: from mail-oi0-f67.google.com ([209.85.218.67]:35987 "EHLO mail-oi0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932867AbdBQLcH (ORCPT ); Fri, 17 Feb 2017 06:32:07 -0500 MIME-Version: 1.0 In-Reply-To: <07efabbe-1c1f-40ea-339e-1c09dbbfce30@suse.de> References: <20170215165528.10052-1-afaerber@suse.de> <20170215165528.10052-5-afaerber@suse.de> <6175161.4SuqJTY84p@wuerfel> <07efabbe-1c1f-40ea-339e-1c09dbbfce30@suse.de> From: Arnd Bergmann Date: Fri, 17 Feb 2017 12:32:05 +0100 X-Google-Sender-Auth: lk7lUQR3HUOu-nRH8fnr0B21vsE Message-ID: Subject: Re: [PATCH 04/11] ARM64: Prepare Actions Semi S900 To: =?UTF-8?Q?Andreas_F=C3=A4rber?= Cc: Linux ARM , mp-cs@actions-semi.com, support@lemaker.org, info@ucrobotics.com, Will Deacon , Linux Kernel Mailing List , Catalin Marinas 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-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v1HBWKe7029373 On Fri, Feb 17, 2017 at 1:34 AM, Andreas Färber wrote: > Am 16.02.2017 um 14:43 schrieb Arnd Bergmann: >> On Wednesday, February 15, 2017 5:55:21 PM CET Andreas Färber wrote: >>> >>> +config ARCH_OWL >>> + bool "Actions Semi S900 SoC Family" >>> + help >>> + This enables support for the Actions Semiconductor S900 SoC family. >>> + >>> >> >> There seem to be a couple of other SoCs in the same family, so >> maybe use a more generic name than S900 in the one-line >> Kconfig description. > > Good point. I only tested with `make oldconfig`, so using "Owl" should > tidy the sort order in `make menuconfig` while at it. (I believe there > were more inconsistent examples, such as Armada 3700 vs. MVEBU...) > > Note that I have not come across any explanation for "OWL" as an > acronym, so I'm guessing it's the bird and therefore capitalizing it. > > Apart from S900, actions-semi.com shows V series S900VR and S700 as well > as GT series GT7 Cortex-A53 based SoCs. My guess[*] is these could all > be handled as ARCH_OWL, adding further differentiation below if > necessary; but Bubblegum-96 seemed the only available arm64 board for > testing so far, so any comments from Actions will be appreciated. > > [*] The S500 vendor device tree was actually using ATM series models for > various nodes, so I'm inferring that the product series are closely > enough related. > > However, if you would prefer ARCH_ACTIONS I wouldn't mind either - we > just need to keep it consistent across arm and arm64. ARCH_ACTIONS seems best to me. We tend to go for the broader names per company these days even when things are not closely related. This avoids having to rename things when a new line of products comes up that shares a subset of the components but doesn't fit under the old name. Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Fri, 17 Feb 2017 12:32:05 +0100 Subject: [PATCH 04/11] ARM64: Prepare Actions Semi S900 In-Reply-To: <07efabbe-1c1f-40ea-339e-1c09dbbfce30@suse.de> References: <20170215165528.10052-1-afaerber@suse.de> <20170215165528.10052-5-afaerber@suse.de> <6175161.4SuqJTY84p@wuerfel> <07efabbe-1c1f-40ea-339e-1c09dbbfce30@suse.de> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Feb 17, 2017 at 1:34 AM, Andreas F?rber wrote: > Am 16.02.2017 um 14:43 schrieb Arnd Bergmann: >> On Wednesday, February 15, 2017 5:55:21 PM CET Andreas F?rber wrote: >>> >>> +config ARCH_OWL >>> + bool "Actions Semi S900 SoC Family" >>> + help >>> + This enables support for the Actions Semiconductor S900 SoC family. >>> + >>> >> >> There seem to be a couple of other SoCs in the same family, so >> maybe use a more generic name than S900 in the one-line >> Kconfig description. > > Good point. I only tested with `make oldconfig`, so using "Owl" should > tidy the sort order in `make menuconfig` while at it. (I believe there > were more inconsistent examples, such as Armada 3700 vs. MVEBU...) > > Note that I have not come across any explanation for "OWL" as an > acronym, so I'm guessing it's the bird and therefore capitalizing it. > > Apart from S900, actions-semi.com shows V series S900VR and S700 as well > as GT series GT7 Cortex-A53 based SoCs. My guess[*] is these could all > be handled as ARCH_OWL, adding further differentiation below if > necessary; but Bubblegum-96 seemed the only available arm64 board for > testing so far, so any comments from Actions will be appreciated. > > [*] The S500 vendor device tree was actually using ATM series models for > various nodes, so I'm inferring that the product series are closely > enough related. > > However, if you would prefer ARCH_ACTIONS I wouldn't mind either - we > just need to keep it consistent across arm and arm64. ARCH_ACTIONS seems best to me. We tend to go for the broader names per company these days even when things are not closely related. This avoids having to rename things when a new line of products comes up that shares a subset of the components but doesn't fit under the old name. Arnd