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=-14.5 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER, INCLUDES_PATCH,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 7CAE6C43214 for ; Thu, 19 Aug 2021 11:23:06 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 390FD610D2 for ; Thu, 19 Aug 2021 11:23:06 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.1 mail.kernel.org 390FD610D2 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=bootlin.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=OUUD5oggWsEJ7/hRG24WL7rAoB9V6FAiX7xvS3fJl3Q=; b=j9gIZeazQopSiJ ZDq78B9vxEPICHCMLLNncga8i3leCYCDC3znds0jHbom8EpJgpMwCSCTq14kKkcZ2c+8rN6crYDxp lnuf/GEfEYkrCPy5cZAta0mDrjqXIGGesXsL9kSI/X+WBsOqsbzxKhZdgyifWQSvc+ueiPdYLOQZb +JBNjvesrnRIn0gWPBaWa5ofVHlC2miMtV/soA7RoQk6em2qwDF5L+PxHMLzGNy3GBKvU2IjYHqKT B9kZC3uCAayzXT9Hx9cmQD1DaujH2P1GEqY5dhSIQ5sTbP15weQfdEqG+hsXlSmtR1naj4dnHdbpW 8IE1bNxAz0Xy7JPqedAA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1mGg69-00899t-UK; Thu, 19 Aug 2021 11:20:38 +0000 Received: from relay7-d.mail.gandi.net ([217.70.183.200]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1mGg63-00898P-9E for linux-arm-kernel@lists.infradead.org; Thu, 19 Aug 2021 11:20:35 +0000 Received: (Authenticated sender: alexandre.belloni@bootlin.com) by relay7-d.mail.gandi.net (Postfix) with ESMTPSA id 602BE20005; Thu, 19 Aug 2021 11:20:26 +0000 (UTC) Date: Thu, 19 Aug 2021 13:20:25 +0200 From: Alexandre Belloni To: Hari.PrasathGE@microchip.com Subject: Re: [PATCH 2/2] ARM: at91: dts: at91-sama5d2_xplained: Add comments for sama5d29 Message-ID: References: <20210812140758.28273-1-Hari.PrasathGE@microchip.com> <20210812140758.28273-2-Hari.PrasathGE@microchip.com> <78b26b01-b66f-5779-9f67-fdd13e473dfc@microchip.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <78b26b01-b66f-5779-9f67-fdd13e473dfc@microchip.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20210819_042031_502440_3AC0B1E4 X-CRM114-Status: GOOD ( 29.08 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: robh@kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Ludovic.Desroches@microchip.com, linux@armlinux.org.uk, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 19/08/2021 10:56:21+0000, Hari.PrasathGE@microchip.com wrote: > Hello, > > On 13/08/21 12:56 am, Alexandre Belloni wrote: > > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > > > Hello, > > > > On 12/08/2021 19:37:58+0530, Hari Prasath wrote: > >> Add comments for the end user for modifying the DTS file for > >> instantiating the sama5d29 SoC. > >> > >> Signed-off-by: Hari Prasath > >> --- > >> arch/arm/boot/dts/at91-sama5d2_xplained.dts | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts > >> index 627b7bf88d83..faa30063d9a9 100644 > >> --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts > >> +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts > >> @@ -6,6 +6,11 @@ > >> * 2015 Nicolas Ferre > >> */ > >> /dts-v1/; > >> +/* > >> + * Replace the line below with "sama5d29.dtsi" in order to instantiate the > >> + * sama5d29 SoC of the sama5d2 family.Otherwise, leave it unchanged when > >> + * using sama5d27 SoC for instance. > >> + */ > > > > I guess it would be better to have a at91-sama5d29_xplained.dts (and we > > should have had a at91-sama5d27_xplained.dts), else, you can't create an > > image that will support both variants. > > > > > The new SoC 'sama5d29' belongs to the same sama5d2 family. It is > essentially a new chip revision of the existing sama5d27 with a new GMAC > controller IP that addresses few silicon issues. There wouldn't be > separate evaluation boards that shall be made but we will be using the > existing sama5d2 XPlained boards itself but with the new SoC populated. That's what I understood but a board with a different SoC is a different board. > Hence we have taken this approach of having a separate DTSI file and > reuse the existing board specific DTS file. > > We don't want to create single image that will support both variants. In > fact, we don't want our customers to enable certain features that are > broken at the silicon level in the existing revision of the SoC i.e > sama5d27. Instead, they could do this change manually and use it in the > new SoC i.e sama5d29. > This will be confusing to your customers while you could make their lives simpler by having the bootloader chose the proper dtb instead of having them change that manually. They will then have to regenerate images with that change, see how your customers struggle to do that: https://stackoverflow.com/questions/68222619/update-custom-device-tree-on-yocto https://stackoverflow.com/questions/37347808/how-to-use-an-own-device-tree-and-modified-kernel-config-in-yocto https://stackoverflow.com/questions/63047955/overriding-defconfig-in-bbappend-file -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel