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=-20.8 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_PATCH, INCLUDES_PULL_REQUEST,MAILING_LIST_MULTI,MENTIONS_GIT_HOSTING,SPF_HELO_NONE, 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 B0090C432BE for ; Thu, 26 Aug 2021 15:05:56 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8A25660F44 for ; Thu, 26 Aug 2021 15:05:56 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S242916AbhHZPGl (ORCPT ); Thu, 26 Aug 2021 11:06:41 -0400 Received: from ozlabs.org ([203.11.71.1]:48935 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S242910AbhHZPGj (ORCPT ); Thu, 26 Aug 2021 11:06:39 -0400 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 4GwR4G6tkqz9sRN; Fri, 27 Aug 2021 01:05:50 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ellerman.id.au; s=201909; t=1629990351; bh=P1ggop+Wpzp+p41o8DUX9QEAna0imAFZLWPlXGklRsw=; h=From:To:Cc:Subject:In-Reply-To:References:Date:From; b=QEOwnB73olOBHQnq/LjP8y5hNffJV5EeYWomV9fF5oGjaYfLwsgZ1c+DgCHrnl8NS OeJS2xQP8dix5Si+cDCbO26RiHn59ETQMvSk+GQfnpyUkUOLdNd2zzXIqTc6mPYCcU lNKdJBD397QHWtD/cNooKNODw7lJ7WwQixmem+peP1QRHzhHQuKeyCIITxPiE+zE5A hg5rGvBTKBsOi20XBG6UKIEqbrNse8ynqSSGVIqG+ZQQNniQKrmorJxY87EEOK29s2 DLC9A7VjJeQ0Y/nLxpNjCQmuiKx9ZlcvGWO5wh5j33ADCSduoYEEQRj5SPfjRzT+At h0bmhfgs2fpGA== From: Michael Ellerman To: Paul Gortmaker Cc: Benjamin Herrenschmidt , Paul Mackerras , Scott Wood , linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [GIT PULL] retire legacy WR sbc8548 and sbc8641 platforms In-Reply-To: <20210824174209.GB160508@windriver.com> References: <20210824174209.GB160508@windriver.com> Date: Fri, 27 Aug 2021 01:05:50 +1000 Message-ID: <87pmu09rg1.fsf@mpe.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Paul Gortmaker writes: > This is unchanged from the original wr_sbc-delete branch sent in January, > other than to add the Acks from Scott in July, and update the baseline. > > Built with ppc64 defconfig and mpc85xx_cds_defconfig and mpc86xx_defconfig > just to make sure I didn't fat finger anything in the baseline update. Thanks for following up on this. I ended up cherry-picking the patches into my branch. I like to keep my next based on rc2, and merging this would have pulled in everything up to rc7 into my branch. I don't think you were planning to merge this branch anywhere else, so it shouldn't make any difference, but let me know if it's a problem. It should appear here soon: https://git.kernel.org/pub/scm/linux/kernel/git/powerpc/linux.git/log/?h=next cheers > Original v1 text follows below, from: > > https://lore.kernel.org/lkml/20210111082823.99562-1-paul.gortmaker@windriver.com > > It would be nice to get this in and off our collective to-do list. > > Thanks, > Paul. > > --- > > In v2.6.27 (2008, 917f0af9e5a9) the sbc8260 support was implicitly > retired by not being carried forward through the ppc --> powerpc > device tree transition. > > Then, in v3.6 (2012, b048b4e17cbb) we retired the support for the > sbc8560 boards. > > Next, in v4.18 (2017, 3bc6cf5a86e5) we retired the support for the > 2006 vintage sbc834x boards. > > The sbc8548 and sbc8641d boards were maybe 1-2 years newer than the > sbc834x boards, but it is also 3+ years later, so it makes sense to > now retire them as well - which is what is done here. > > These two remaining WR boards were based on the Freescale MPC8548-CDS > and the MPC8641D-HPCN reference board implementations. Having had the > chance to use these and many other Fsl ref boards, I know this: The > Freescale reference boards were typically produced in limited quantity > and primarily available to BSP developers and hardware designers, and > not likely to have found a 2nd life with hobbyists and/or collectors. > > It was good to have that BSP code subjected to mainline review and > hence also widely available back in the day. But given the above, we > should probably also be giving serious consideration to retiring > additional similar age/type reference board platforms as well. > > I've always felt it is important for us to be proactive in retiring > old code, since it has a genuine non-zero carrying cost, as described > in the 930d52c012b8 merge log. But for the here and now, we just > clean up the remaining BSP code that I had added for SBC platforms. > > --- > > The following changes since commit e22ce8eb631bdc47a4a4ea7ecf4e4ba499db4f93: > > Linux 5.14-rc7 (2021-08-22 14:24:56 -0700) > > are available in the Git repository at: > > git://git.kernel.org/pub/scm/linux/kernel/git/paulg/linux.git wr_sbc-delete-v2 > > for you to fetch changes up to d44e2dc12ea2112e74cdd25090eeda2727ed09cc: > > MAINTAINERS: update for Paul Gortmaker (2021-08-24 08:19:01 -0400) > > ---------------------------------------------------------------- > Paul Gortmaker (3): > powerpc: retire sbc8548 board support > powerpc: retire sbc8641d board support > MAINTAINERS: update for Paul Gortmaker > > MAINTAINERS | 1 - > arch/powerpc/boot/Makefile | 1 - > arch/powerpc/boot/dts/fsl/sbc8641d.dts | 176 ----------------- > arch/powerpc/boot/dts/sbc8548-altflash.dts | 111 ----------- > arch/powerpc/boot/dts/sbc8548-post.dtsi | 289 ---------------------------- > arch/powerpc/boot/dts/sbc8548-pre.dtsi | 48 ----- > arch/powerpc/boot/dts/sbc8548.dts | 106 ---------- > arch/powerpc/boot/wrapper | 2 +- > arch/powerpc/configs/85xx/sbc8548_defconfig | 50 ----- > arch/powerpc/configs/mpc85xx_base.config | 1 - > arch/powerpc/configs/mpc86xx_base.config | 1 - > arch/powerpc/configs/ppc6xx_defconfig | 1 - > arch/powerpc/platforms/85xx/Kconfig | 6 - > arch/powerpc/platforms/85xx/Makefile | 1 - > arch/powerpc/platforms/85xx/sbc8548.c | 134 ------------- > arch/powerpc/platforms/86xx/Kconfig | 8 +- > arch/powerpc/platforms/86xx/Makefile | 1 - > arch/powerpc/platforms/86xx/sbc8641d.c | 87 --------- > 18 files changed, 2 insertions(+), 1022 deletions(-) > delete mode 100644 arch/powerpc/boot/dts/fsl/sbc8641d.dts > delete mode 100644 arch/powerpc/boot/dts/sbc8548-altflash.dts > delete mode 100644 arch/powerpc/boot/dts/sbc8548-post.dtsi > delete mode 100644 arch/powerpc/boot/dts/sbc8548-pre.dtsi > delete mode 100644 arch/powerpc/boot/dts/sbc8548.dts > delete mode 100644 arch/powerpc/configs/85xx/sbc8548_defconfig > delete mode 100644 arch/powerpc/platforms/85xx/sbc8548.c > delete mode 100644 arch/powerpc/platforms/86xx/sbc8641d.c