2020-01-06 13:10:46

by Matwey V. Kornilov

[permalink] [raw]
Subject: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

BeagleBone Black series is equipped with 512MB RAM
whereas only 256MB is included from am335x-bone-common.dtsi

This leads to an issue with unusual setups when devicetree
is loaded by GRUB2 directly.

Signed-off-by: Matwey V. Kornilov <[email protected]>
---
arch/arm/boot/dts/am335x-boneblack-common.dtsi | 5 +++++
1 file changed, 5 insertions(+)

diff --git a/arch/arm/boot/dts/am335x-boneblack-common.dtsi b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
index 7ad079861efd..91f93bc89716 100644
--- a/arch/arm/boot/dts/am335x-boneblack-common.dtsi
+++ b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
@@ -131,6 +131,11 @@
};

/ {
+ memory@80000000 {
+ device_type = "memory";
+ reg = <0x80000000 0x20000000>; /* 512 MB */
+ };
+
clk_mcasp0_fixed: clk_mcasp0_fixed {
#clock-cells = <0>;
compatible = "fixed-clock";
--
2.16.4


2020-01-06 20:45:15

by Robert Nelson

[permalink] [raw]
Subject: Re: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

On Mon, Jan 6, 2020 at 7:10 AM Matwey V. Kornilov <[email protected]> wrote:
>
> BeagleBone Black series is equipped with 512MB RAM
> whereas only 256MB is included from am335x-bone-common.dtsi

FYI: While all versions from the factory are 512MB, some 3rd parties
offered 1GB reballing upgrades..

and the SanCloud variant which uses this file, was built with 1GB:

https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/am335x-sancloud-bbe.dts

>
> This leads to an issue with unusual setups when devicetree
> is loaded by GRUB2 directly.

It's a miracle, since when did GRUB2 gain this ability?

>
> Signed-off-by: Matwey V. Kornilov <[email protected]>
> ---
> arch/arm/boot/dts/am335x-boneblack-common.dtsi | 5 +++++
> 1 file changed, 5 insertions(+)
>
> diff --git a/arch/arm/boot/dts/am335x-boneblack-common.dtsi b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> index 7ad079861efd..91f93bc89716 100644
> --- a/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> +++ b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> @@ -131,6 +131,11 @@
> };
>
> / {
> + memory@80000000 {
> + device_type = "memory";
> + reg = <0x80000000 0x20000000>; /* 512 MB */
> + };
> +
> clk_mcasp0_fixed: clk_mcasp0_fixed {
> #clock-cells = <0>;
> compatible = "fixed-clock";
> --
> 2.16.4
>


--
Robert Nelson
https://rcn-ee.com/

2020-01-06 20:49:02

by Matwey V. Kornilov

[permalink] [raw]
Subject: Re: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

пн, 6 янв. 2020 г. в 23:44, Robert Nelson <[email protected]>:
>
> On Mon, Jan 6, 2020 at 7:10 AM Matwey V. Kornilov <[email protected]> wrote:
> >
> > BeagleBone Black series is equipped with 512MB RAM
> > whereas only 256MB is included from am335x-bone-common.dtsi
>
> FYI: While all versions from the factory are 512MB, some 3rd parties
> offered 1GB reballing upgrades..
>
> and the SanCloud variant which uses this file, was built with 1GB:
>
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/arm/boot/dts/am335x-sancloud-bbe.dts
>
> >
> > This leads to an issue with unusual setups when devicetree
> > is loaded by GRUB2 directly.
>
> It's a miracle, since when did GRUB2 gain this ability?
>

I am not sure about when. But Grub 2.02 has devicetree command, which
can be used as the following:

echo "Loading device tree blob ..."
devicetree (tftp)/dtb/am335x-boneblack.dtb

And you know, you can run Grub itself as EFI application on to of u-boot.

> >
> > Signed-off-by: Matwey V. Kornilov <[email protected]>
> > ---
> > arch/arm/boot/dts/am335x-boneblack-common.dtsi | 5 +++++
> > 1 file changed, 5 insertions(+)
> >
> > diff --git a/arch/arm/boot/dts/am335x-boneblack-common.dtsi b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> > index 7ad079861efd..91f93bc89716 100644
> > --- a/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> > +++ b/arch/arm/boot/dts/am335x-boneblack-common.dtsi
> > @@ -131,6 +131,11 @@
> > };
> >
> > / {
> > + memory@80000000 {
> > + device_type = "memory";
> > + reg = <0x80000000 0x20000000>; /* 512 MB */
> > + };
> > +
> > clk_mcasp0_fixed: clk_mcasp0_fixed {
> > #clock-cells = <0>;
> > compatible = "fixed-clock";
> > --
> > 2.16.4
> >
>
>
> --
> Robert Nelson
> https://rcn-ee.com/



--
With best regards,
Matwey V. Kornilov.
Sternberg Astronomical Institute, Lomonosov Moscow State University, Russia
119234, Moscow, Universitetsky pr-k 13, +7 (495) 9392382

2020-01-06 21:06:17

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

* Matwey V. Kornilov <[email protected]> [200106 20:48]:
> пн, 6 янв. 2020 г. в 23:44, Robert Nelson <[email protected]>:
> >
> > On Mon, Jan 6, 2020 at 7:10 AM Matwey V. Kornilov <[email protected]> wrote:
> > >
> > > BeagleBone Black series is equipped with 512MB RAM
> > > whereas only 256MB is included from am335x-bone-common.dtsi
> >
> > FYI: While all versions from the factory are 512MB, some 3rd parties
> > offered 1GB reballing upgrades..

So what's the conclusion, is it safe to bump the default size to
512MB then?

The custom ones could use their own dts file if bootloader is not
setting the RAM.

Regards,

Tony

2020-01-07 18:35:10

by Robert Nelson

[permalink] [raw]
Subject: Re: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

On Mon, Jan 6, 2020 at 3:05 PM Tony Lindgren <[email protected]> wrote:
>
> * Matwey V. Kornilov <[email protected]> [200106 20:48]:
> > пн, 6 янв. 2020 г. в 23:44, Robert Nelson <[email protected]>:
> > >
> > > On Mon, Jan 6, 2020 at 7:10 AM Matwey V. Kornilov <[email protected]> wrote:
> > > >
> > > > BeagleBone Black series is equipped with 512MB RAM
> > > > whereas only 256MB is included from am335x-bone-common.dtsi
> > >
> > > FYI: While all versions from the factory are 512MB, some 3rd parties
> > > offered 1GB reballing upgrades..
>
> So what's the conclusion, is it safe to bump the default size to
> 512MB then?
>
> The custom ones could use their own dts file if bootloader is not
> setting the RAM.

Yeah this is safe for factory devices. Classic u-boot will update
these for modified boards.

Regards,

--
Robert Nelson
https://rcn-ee.com/

2020-01-13 17:51:53

by Tony Lindgren

[permalink] [raw]
Subject: Re: [PATCH] arm: dts: am335x-boneblack-common: fix memory size

* Robert Nelson <[email protected]> [200107 18:33]:
> On Mon, Jan 6, 2020 at 3:05 PM Tony Lindgren <[email protected]> wrote:
> >
> > * Matwey V. Kornilov <[email protected]> [200106 20:48]:
> > > пн, 6 янв. 2020 г. в 23:44, Robert Nelson <[email protected]>:
> > > >
> > > > On Mon, Jan 6, 2020 at 7:10 AM Matwey V. Kornilov <[email protected]> wrote:
> > > > >
> > > > > BeagleBone Black series is equipped with 512MB RAM
> > > > > whereas only 256MB is included from am335x-bone-common.dtsi
> > > >
> > > > FYI: While all versions from the factory are 512MB, some 3rd parties
> > > > offered 1GB reballing upgrades..
> >
> > So what's the conclusion, is it safe to bump the default size to
> > 512MB then?
> >
> > The custom ones could use their own dts file if bootloader is not
> > setting the RAM.
>
> Yeah this is safe for factory devices. Classic u-boot will update
> these for modified boards.

OK applying into fixes thanks.

Tony