mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Yunus Bas <Y.Bas@phytec.de>
To: "barebox@lists.infradead.org" <barebox@lists.infradead.org>,
	"a.fatoum@pengutronix.de" <a.fatoum@pengutronix.de>
Subject: Re: [PATCH] ARM: dts: imx6qdl: pfla02: Update spi-nor naming for kernel compatibility
Date: Mon, 3 May 2021 21:50:56 +0000	[thread overview]
Message-ID: <b62f44635d096f564b98fc7e8f8539298d30e5f2.camel@phytec.de> (raw)
In-Reply-To: <1001f579-e35d-9133-1abc-846d242684b0@pengutronix.de>

Hello Ahmad,

Am Montag, dem 03.05.2021 um 20:36 +0200 schrieb Ahmad Fatoum:
> Hello Yunus,
> 
> On 03.05.21 16:38, Yunus Bas wrote:
> > Hi Ahmad,
> > 
> > The main purpose behind adding MTD-partitions as NVMEM-devices was
> > to
> > have a fast and uncomplicated access on partitions with sensitive
> > data,
> > especially on embedded systems. I think, there is a reasoning for
> > it,
> > but nevertheless, the original kernel patch is from 2018.
> 
> So it's a patch you carry with you and not Linux-upstream
> functionality?

Registering MTD-partitions as NVMEM devices is Linux-upstream. We just
need to adapt the partition names in barebox.

> 
> > To say the truth, there is already an upstream fix for the name-
> > clashing, which registers the NVMEM devices with the unique device
> > names, instead of the MTD-names. I had two options here, one was to
> > revert the first patch with the change for phyCore, the other was
> > to
> > follow up with the phyFLEX-module. I've chosen the latter one,
> > since I
> > felt it has more readability.
> 
> Thanks for the clarification,
> Ahmad
> 

Maybe it is better to drop this and revert the change on phyCore
instead. Prepending the device before the partition would only apply to
the NOR flash. Also, this solution doesn't work out in case there are
two flash memories of the same kind, e.g. two NOR flashes.

Please drop this patch.

Regards,
Yunus

> > 
> > Regards,
> > Yunus
> > 
> > Am Montag, dem 03.05.2021 um 14:38 +0200 schrieb Ahmad Fatoum:
> > > Hello Yunus,
> > > 
> > > On 03.05.21 14:25, Yunus Bas wrote:
> > > > The kernel NVMEM-subsystem prevents the usage of duplicate
> > > > partition names.
> > > > Change the SPI-NOR partition names to be compatible with the
> > > > kernel
> > > > driver.
> > > 
> > > I don't mind the change, but I can't follow the reasoning.
> > > These are MTD partitions, so they shouldn't have been registered
> > > with
> > > the NVMEM
> > > framework in the first place?
> > > 
> > > Cheers,
> > > Ahmad
> > > 
> > > > 
> > > > Commit 72ba815bf6fe ("ARM: dts: phycore-imx6: change mtd-
> > > > partition
> > > > names for
> > > > compatibility with kernel") applies the change for phyCORE
> > > > SOMs.
> > > > The
> > > > same applies for phyFLEX-modules.
> > > > 
> > > > Signed-off-by: Yunus Bas <y.bas@phytec.de>
> > > > ---
> > > >  arch/arm/dts/imx6qdl-phytec-pfla02.dtsi | 11 ++++++-----
> > > >  1 file changed, 6 insertions(+), 5 deletions(-)
> > > > 
> > > > diff --git a/arch/arm/dts/imx6qdl-phytec-pfla02.dtsi
> > > > b/arch/arm/dts/imx6qdl-phytec-pfla02.dtsi
> > > > index e1aa3183b..6df8d247b 100644
> > > > --- a/arch/arm/dts/imx6qdl-phytec-pfla02.dtsi
> > > > +++ b/arch/arm/dts/imx6qdl-phytec-pfla02.dtsi
> > > > @@ -21,7 +21,7 @@
> > > >  
> > > >                 environment-spinor {
> > > >                         compatible = "barebox,environment";
> > > > -                       device-path = &som_flash,
> > > > "partname:barebox-environment";
> > > > +                       device-path = &som_flash,
> > > > "partname:nor.barebox-environment";
> > > >                         status = "disabled";
> > > >                 };
> > > >  
> > > > @@ -54,28 +54,29 @@
> > > >  /delete-node/ &{/memory@10000000};
> > > >  
> > > >  &som_flash {
> > > > +
> > > >         partitions {
> > > >                 compatible = "fixed-partitions";
> > > >                 #address-cells = <1>;
> > > >                 #size-cells = <1>;
> > > >  
> > > >                 partition@0 {
> > > > -                       label = "barebox";
> > > > +                       label = "nor.barebox";
> > > >                         reg = <0x0 0x100000>;
> > > >                 };
> > > >  
> > > >                 partition@100000 {
> > > > -                       label = "barebox-environment";
> > > > +                       label = "nor.barebox-environment";
> > > >                         reg = <0x100000 0x20000>;
> > > >                 };
> > > >  
> > > >                 partition@120000 {
> > > > -                       label = "oftree";
> > > > +                       label = "nor.oftree";
> > > >                         reg = <0x120000 0x20000>;
> > > >                 };
> > > >  
> > > >                 partition@140000 {
> > > > -                       label = "kernel";
> > > > +                       label = "nor.kernel";
> > > >                         reg = <0x140000 0x0>;
> > > >                 };
> > > >         };
> > > > 
> > > 
> > 
> 

-- 
Mit freundlichen Grüßen
Yunus Bas

-Software Engineer-
PHYTEC Messtechnik GmbH
Robert-Koch-Str. 39
55129 Mainz
Germany
Tel.: +49 (0)6131 9221- 466
Web: www.phytec.de

Sie finden uns auch auf: Facebook, LinkedIn, Xing, YouTube

PHYTEC Messtechnik GmbH | Robert-Koch-Str. 39 | 55129 Mainz, Germany
Geschäftsführer: Dipl.-Ing. Michael Mitezki, Dipl.-Ing. Bodo Huber |
Handelsregister Mainz HRB 4656 | Finanzamt Mainz | St.Nr. 266500608, DE
149059855
This E-Mail may contain confidential or privileged information. If you
are not the intended recipient (or have received this E-Mail in error)
please notify the sender immediately and destroy this E-Mail. Any
unauthorized copying, disclosure or distribution of the material in
this E-Mail is strictly forbidden.
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

      reply	other threads:[~2021-05-03 21:52 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-05-03 12:25 Yunus Bas
2021-05-03 12:38 ` Ahmad Fatoum
2021-05-03 14:38   ` Yunus Bas
2021-05-03 18:36     ` Ahmad Fatoum
2021-05-03 21:50       ` Yunus Bas [this message]

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=b62f44635d096f564b98fc7e8f8539298d30e5f2.camel@phytec.de \
    --to=y.bas@phytec.de \
    --cc=a.fatoum@pengutronix.de \
    --cc=barebox@lists.infradead.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox