[PATCH v1 4/7] arm: dts: imx8mp-skov: Simplify SD partition definition to 1 cell
Oleksij Rempel
o.rempel at pengutronix.de
Thu Sep 28 21:38:15 PDT 2023
Hello Ahmad,
On Thu, Sep 28, 2023 at 10:13:44PM +0200, Ahmad Fatoum wrote:
> Hello Oleksij,
>
> On 28.09.23 12:58, Oleksij Rempel wrote:
> > Update the partition definitions for SD card in imx8mp-skov to use
> > single cell for address and size, streamlining the format.
>
> Is your eMMC really smaller than 4G? If it's >= 4G, barebox will fix it up in
> the kernel DT as 2/2 cells, so keeping it 2/2 in barebox removes some noise
> in of_diff - + output.
Ok, i see. Thx for the tip.
It is SD card, not eMMC. So, most probably it will be >= 4G.
I just realized, that SD card can be used as simple storage, not only as
the boot source. So, it is probably better to remove partition table from the
devicetree for SD too?
> > Signed-off-by: Oleksij Rempel <o.rempel at pengutronix.de>
> > ---
> > arch/arm/dts/imx8mp-skov.dts | 10 +++++-----
> > 1 file changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/dts/imx8mp-skov.dts b/arch/arm/dts/imx8mp-skov.dts
> > index 1d4738dbd7..b94b9dd51d 100644
> > --- a/arch/arm/dts/imx8mp-skov.dts
> > +++ b/arch/arm/dts/imx8mp-skov.dts
> > @@ -88,17 +88,17 @@ reg_usdhc2_vmmc: regulator-usdhc2 {
> > &usdhc2 {
> > partitions {
> > compatible = "fixed-partitions";
> > - #address-cells = <2>;
> > - #size-cells = <2>;
> > + #address-cells = <1>;
> > + #size-cells = <1>;
> >
> > partition at 0 {
> > label = "barebox";
> > - reg = <0x0 0x0 0x0 0x100000>;
> > + reg = <0x0 0x100000>;
> > };
> >
> > - env_sd: partition at e0000 {
> > + env_sd: partition at 100000 {
> > label = "barebox-environment";
> > - reg = <0x0 0x100000 0x0 0x100000>;
> > + reg = <0x100000 0x100000>;
> > };
> > };
> > };
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the barebox
mailing list