From: <Hari.PrasathGE@microchip.com> To: <alexandre.belloni@bootlin.com> Cc: <Nicolas.Ferre@microchip.com>, <Ludovic.Desroches@microchip.com>, <robh@kernel.org>, <linux-arm-kernel@lists.infradead.org>, <devicetree@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <linux@armlinux.org.uk> Subject: Re: [PATCH 2/2] ARM: at91: dts: at91-sama5d2_xplained: Add comments for sama5d29 Date: Thu, 19 Aug 2021 10:56:21 +0000 [thread overview] Message-ID: <78b26b01-b66f-5779-9f67-fdd13e473dfc@microchip.com> (raw) In-Reply-To: <YRV13nfgpEEuOOxB@piout.net> 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 <Hari.PrasathGE@microchip.com> >> --- >> 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 <nicolas.ferre@atmel.com> >> */ >> /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. 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. Regards, Hari > -- > Alexandre Belloni, co-owner and COO, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com >
WARNING: multiple messages have this Message-ID (diff)
From: <Hari.PrasathGE@microchip.com> To: <alexandre.belloni@bootlin.com> 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 Subject: Re: [PATCH 2/2] ARM: at91: dts: at91-sama5d2_xplained: Add comments for sama5d29 Date: Thu, 19 Aug 2021 10:56:21 +0000 [thread overview] Message-ID: <78b26b01-b66f-5779-9f67-fdd13e473dfc@microchip.com> (raw) In-Reply-To: <YRV13nfgpEEuOOxB@piout.net> 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 <Hari.PrasathGE@microchip.com> >> --- >> 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 <nicolas.ferre@atmel.com> >> */ >> /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. 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. Regards, Hari > -- > 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
next prev parent reply other threads:[~2021-08-19 10:56 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-12 14:07 [PATCH 1/2] ARM: at91: dts: sama5d29: Add dtsi file for sama5d29 Hari Prasath 2021-08-12 14:07 ` Hari Prasath 2021-08-12 14:07 ` [PATCH 2/2] ARM: at91: dts: at91-sama5d2_xplained: Add comments " Hari Prasath 2021-08-12 14:07 ` Hari Prasath 2021-08-12 19:26 ` Alexandre Belloni 2021-08-12 19:26 ` Alexandre Belloni 2021-08-19 10:56 ` Hari.PrasathGE [this message] 2021-08-19 10:56 ` Hari.PrasathGE 2021-08-19 11:20 ` Alexandre Belloni 2021-08-19 11:20 ` Alexandre Belloni 2021-09-21 9:44 ` Nicolas Ferre 2021-09-21 9:44 ` Nicolas Ferre 2021-09-21 10:34 ` [PATCH 1/2] ARM: at91: dts: sama5d29: Add dtsi file " Nicolas Ferre 2021-09-21 10:34 ` Nicolas Ferre
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=78b26b01-b66f-5779-9f67-fdd13e473dfc@microchip.com \ --to=hari.prasathge@microchip.com \ --cc=Ludovic.Desroches@microchip.com \ --cc=Nicolas.Ferre@microchip.com \ --cc=alexandre.belloni@bootlin.com \ --cc=devicetree@vger.kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=robh@kernel.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.