From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-qt1-f180.google.com ([209.85.160.180]:36168 "EHLO mail-qt1-f180.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726304AbeI2CgM (ORCPT ); Fri, 28 Sep 2018 22:36:12 -0400 Received: by mail-qt1-f180.google.com with SMTP id e26-v6so8042922qtq.3 for ; Fri, 28 Sep 2018 13:10:50 -0700 (PDT) MIME-Version: 1.0 References: In-Reply-To: From: Arnd Bergmann Date: Fri, 28 Sep 2018 22:10:34 +0200 Message-ID: Subject: Re: [GIT PULL] Renesas ARM64 Based SoC SoC Updates for v4.20 To: Simon Horman Cc: arm-soc , Linux-Renesas , Olof Johansson , Kevin Hilman , Linux ARM , Magnus Damm Content-Type: text/plain; charset="UTF-8" Sender: linux-renesas-soc-owner@vger.kernel.org List-ID: On Fri, Sep 28, 2018 at 12:21 PM Simon Horman wrote: > > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these Renesas ARM64 based SoC SoC updates for v4.20. > > > The following changes since commit 5b394b2ddf0347bef56e50c69a58773c94343ff3: > > Linux 4.19-rc1 (2018-08-26 14:11:59 -0700) > > are available in the git repository at: > > https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-arm64-soc-for-v4.20 > > for you to fetch changes up to 692dce77dfb71b5eaf896d3cdc7ef72f70631b14: > > arm64: Add Renesas R8A774C0 support (2018-09-11 15:50:57 +0200) > > ---------------------------------------------------------------- > Renesas ARM64 Based SoC SoC Updates for v4.20 > > * Add support for RZ/G2E (r8a774c0) and RZ/G2M (r8a774a1) SoCs > * Enable Compare Match Timer (CMT) and Timer Unit (TMU) > for Renesas SoCs > * Remove no longer needed ARCH_SHMOBILE Kconfig symbol The ARCH_SHMOBILE removal and cleanup is fine, but I wonder about the newly added symbols for specific SoCs. I see you already have a couple of those, but the other manufacturers don't. I think what happened here is that I tried to reject those additions normally, but I never noticed yours. From what I see in drivers, you have additional symbols depending on these in clk, pinctrl, drivers/soc, and the dtb files, so at the very least we can't just drop the config options, but I'd still like to see this work more like other platforms. Any ideas what we can do here? Arnd From mboxrd@z Thu Jan 1 00:00:00 1970 From: arnd@arndb.de (Arnd Bergmann) Date: Fri, 28 Sep 2018 22:10:34 +0200 Subject: [GIT PULL] Renesas ARM64 Based SoC SoC Updates for v4.20 In-Reply-To: References: Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Fri, Sep 28, 2018 at 12:21 PM Simon Horman wrote: > > Hi Olof, Hi Kevin, Hi Arnd, > > Please consider these Renesas ARM64 based SoC SoC updates for v4.20. > > > The following changes since commit 5b394b2ddf0347bef56e50c69a58773c94343ff3: > > Linux 4.19-rc1 (2018-08-26 14:11:59 -0700) > > are available in the git repository at: > > https://git.kernel.org/pub/scm/linux/kernel/git/horms/renesas.git tags/renesas-arm64-soc-for-v4.20 > > for you to fetch changes up to 692dce77dfb71b5eaf896d3cdc7ef72f70631b14: > > arm64: Add Renesas R8A774C0 support (2018-09-11 15:50:57 +0200) > > ---------------------------------------------------------------- > Renesas ARM64 Based SoC SoC Updates for v4.20 > > * Add support for RZ/G2E (r8a774c0) and RZ/G2M (r8a774a1) SoCs > * Enable Compare Match Timer (CMT) and Timer Unit (TMU) > for Renesas SoCs > * Remove no longer needed ARCH_SHMOBILE Kconfig symbol The ARCH_SHMOBILE removal and cleanup is fine, but I wonder about the newly added symbols for specific SoCs. I see you already have a couple of those, but the other manufacturers don't. I think what happened here is that I tried to reject those additions normally, but I never noticed yours. From what I see in drivers, you have additional symbols depending on these in clk, pinctrl, drivers/soc, and the dtb files, so at the very least we can't just drop the config options, but I'd still like to see this work more like other platforms. Any ideas what we can do here? Arnd