From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.8 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 24C20C67863 for ; Tue, 23 Oct 2018 11:11:09 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id ECD472075D for ; Tue, 23 Oct 2018 11:11:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org ECD472075D Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=arm.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728097AbeJWTeG (ORCPT ); Tue, 23 Oct 2018 15:34:06 -0400 Received: from foss.arm.com ([217.140.101.70]:57304 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726953AbeJWTeG (ORCPT ); Tue, 23 Oct 2018 15:34:06 -0400 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 675F7341; Tue, 23 Oct 2018 04:11:06 -0700 (PDT) Received: from big-swifty.misterjones.org (usa-sjc-mx-foss1.foss.arm.com [217.140.101.70]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 24A9B3F6A8; Tue, 23 Oct 2018 04:10:50 -0700 (PDT) Date: Tue, 23 Oct 2018 12:10:46 +0100 Message-ID: <8636swude1.wl-marc.zyngier@arm.com> From: Marc Zyngier To: Bartosz Golaszewski Cc: Sekhar Nori , Kevin Hilman , David Lechner , Michael Turquette , Stephen Boyd , Arnd Bergmann , Greg Kroah-Hartman , Mark Rutland , Yoshinori Sato , Rich Felker , Andy Shevchenko , "Rafael J . Wysocki" , Peter Rosin , Jiri Slaby , Thomas Gleixner , Daniel Lezcano , Geert Uytterhoeven , Magnus Damm , Johan Hovold , Rob Herring , Frank Rowand , Philippe Ombredanne , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arch@vger.kernel.org, Bartosz Golaszewski Subject: Re: [RESEND PATCH v3 3/3] sh: add the sh_ prefix to early platform symbols In-Reply-To: <20180827090651.25062-4-brgl@bgdev.pl> References: <20180827090651.25062-1-brgl@bgdev.pl> <20180827090651.25062-4-brgl@bgdev.pl> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI-EPG/1.14.7 (Harue) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 EasyPG/1.0.0 Emacs/25.1 (aarch64-unknown-linux-gnu) MULE/6.0 (HANACHIRUSATO) Organization: ARM Ltd MIME-Version: 1.0 (generated by SEMI-EPG 1.14.7 - "Harue") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Bartosz, On Mon, 27 Aug 2018 10:06:51 +0100, Bartosz Golaszewski wrote: > > From: Bartosz Golaszewski > > Old early platform device support is now sh-specific. Before moving on > to implementing new early platform framework based on real platform > devices, prefix all early platform symbols with 'sh_'. > > Signed-off-by: Bartosz Golaszewski > --- > arch/sh/drivers/platform_early.c | 66 +++++++++++++------------- > arch/sh/include/asm/platform_early.h | 30 ++++++------ > arch/sh/kernel/cpu/sh2/setup-sh7619.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-mxg.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-sh7201.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-sh7203.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-sh7206.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-sh7264.c | 2 +- > arch/sh/kernel/cpu/sh2a/setup-sh7269.c | 2 +- > arch/sh/kernel/cpu/sh3/setup-sh7705.c | 2 +- > arch/sh/kernel/cpu/sh3/setup-sh770x.c | 2 +- > arch/sh/kernel/cpu/sh3/setup-sh7710.c | 2 +- > arch/sh/kernel/cpu/sh3/setup-sh7720.c | 2 +- > arch/sh/kernel/cpu/sh4/setup-sh4-202.c | 2 +- > arch/sh/kernel/cpu/sh4/setup-sh7750.c | 8 ++-- > arch/sh/kernel/cpu/sh4/setup-sh7760.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7343.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7366.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7722.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7723.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7724.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7734.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7757.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7763.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7770.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7780.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7785.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-sh7786.c | 2 +- > arch/sh/kernel/cpu/sh4a/setup-shx3.c | 2 +- > arch/sh/kernel/cpu/sh5/setup-sh5.c | 2 +- > arch/sh/kernel/setup.c | 2 +- > arch/sh/kernel/time.c | 4 +- > drivers/clocksource/sh_cmt.c | 6 +-- > drivers/clocksource/sh_mtu2.c | 6 +-- > drivers/clocksource/sh_tmu.c | 6 +-- > drivers/tty/serial/sh-sci.c | 4 +- > include/linux/platform_device.h | 2 +- > 37 files changed, 94 insertions(+), 94 deletions(-) > > diff --git a/arch/sh/drivers/platform_early.c b/arch/sh/drivers/platform_early.c > index 3dfe9f3541bb..f6d148451dfc 100644 > --- a/arch/sh/drivers/platform_early.c > +++ b/arch/sh/drivers/platform_early.c > @@ -4,8 +4,8 @@ > #include > #include > > -static __initdata LIST_HEAD(early_platform_driver_list); > -static __initdata LIST_HEAD(early_platform_device_list); > +static __initdata LIST_HEAD(sh_early_platform_driver_list); > +static __initdata LIST_HEAD(sh_early_platform_device_list); As discussed offline at ELC, I don't think this patch is useful at this stage, and in the context of this particular series. I believe it should be dropped and only the first two patches should be merged. If the SH maintainer is not responsive, then maybe Greg could take them as they remove cruft from core code. As for the whole "early device" debate, my take is that we'd be much better off making the device model available early, rather than trying to architect something that is simply papering over the issue. Thanks, M. -- Jazz is not dead, it just smell funny.