All of lore.kernel.org
 help / color / mirror / Atom feed
* [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration
@ 2019-11-27 14:28 djordje senicic
  2019-12-05 19:55 ` Denys Dmytriyenko
  0 siblings, 1 reply; 3+ messages in thread
From: djordje senicic @ 2019-11-27 14:28 UTC (permalink / raw)
  To: meta-ti; +Cc: d-senicic1, Djordje Senicic

- CMEM memory block increased to 512MB to enable TIDL using 4xEVE and 2xDSP

Signed-off-by: Djordje Senicic <x0157990@ti.com>
---
 recipes-kernel/linux/cmem.inc                 |  9 +++--
 .../linux/files/dra7xx/cmem-am572i.dtsi       | 40 +++++++++++++++++++
 2 files changed, 45 insertions(+), 4 deletions(-)
 create mode 100644 recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi

diff --git a/recipes-kernel/linux/cmem.inc b/recipes-kernel/linux/cmem.inc
index 8bd643f4..8d33fe35 100644
--- a/recipes-kernel/linux/cmem.inc
+++ b/recipes-kernel/linux/cmem.inc
@@ -2,14 +2,15 @@
 #
 # Use these for temporary overrides
 CMEM_MACHINE = "${MACHINE}"
-CMEM_MACHINE_am57xx-evm = "am571x am572x am574x"
-CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am574x"
+CMEM_MACHINE_am57xx-evm = "am571x am572x am572i am574x"
+CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am572i am574x"
 CMEM_MACHINE_dra7xx-evm = "dra71x dra72x dra74x dra76x"
 CMEM_MACHINE_dra7xx-hs-evm = "dra71x dra72x dra74x dra76x"
 
 # Set cmem.dtsi per machine or machine variant
 CMEM_DTSI = "cmem.dtsi"
 CMEM_DTSI_am571x = "cmem-am571x.dtsi"
+CMEM_DTSI_am572i = "cmem-am572i.dtsi"
 CMEM_DTSI_am574x = "cmem-am574x.dtsi"
 CMEM_DTSI_dra71x = "cmem-dra71x.dtsi"
 CMEM_DTSI_dra72x = "cmem-dra72x.dtsi"
@@ -18,8 +19,8 @@ CMEM_DTSI_dra76x = "cmem-dra76x.dtsi"
 # Split device trees between variants
 CMEM_DEVICETREE = "${KERNEL_DEVICETREE}"
 CMEM_DEVICETREE_am571x = "am571x-idk.dtb am571x-idk-lcd-osd101t2045.dtb am571x-idk-lcd-osd101t2587.dtb"
-CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb \
-                          am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
+CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb"
+CMEM_DEVICETREE_am572i = "am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
 CMEM_DEVICETREE_am574x = "am574x-idk.dtb am574x-idk-lcd-osd101t2587.dtb"
 CMEM_DEVICETREE_dra71x = "dra71-evm.dtb"
 CMEM_DEVICETREE_dra72x = "dra72-evm.dtb dra72-evm-lcd-osd101t2045.dtb dra72-evm-lcd-osd101t2587.dtb \
diff --git a/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
new file mode 100644
index 00000000..6c39e22c
--- /dev/null
+++ b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
@@ -0,0 +1,40 @@
+/ {
+        reserved-memory {
+                #address-cells = <2>;
+                #size-cells = <2>;
+                ranges;
+
+                cmem_block_mem_0: cmem_block_mem@a0000000 {
+                        reg = <0x0 0xa0000000 0x0 0x20000000>;
+                        no-map;
+                        status = "okay";
+                };
+
+		cmem_block_mem_1_ocmc3: cmem_block_mem@40500000 {
+			reg = <0x0 0x40500000 0x0 0x100000>;
+			no-map;
+			status = "okay";
+		};
+        };
+
+        cmem {
+                compatible = "ti,cmem";
+                #address-cells = <1>;
+                #size-cells = <0>;
+
+		#pool-size-cells = <2>;
+
+                status = "okay";
+
+                cmem_block_0: cmem_block@0 {
+                        reg = <0>;
+                        memory-region = <&cmem_block_mem_0>;
+                        cmem-buf-pools = <1 0x0 0x20000000>;
+                };
+
+		cmem_block_1: cmem_block@1 {
+			reg = <1>;
+			memory-region = <&cmem_block_mem_1_ocmc3>;
+		};
+        };
+};
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration
  2019-11-27 14:28 [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration djordje senicic
@ 2019-12-05 19:55 ` Denys Dmytriyenko
  2019-12-05 22:30   ` djordje senicic
  0 siblings, 1 reply; 3+ messages in thread
From: Denys Dmytriyenko @ 2019-12-05 19:55 UTC (permalink / raw)
  To: d-senicic1; +Cc: meta-ti, meta-ti

On Wed, Nov 27, 2019 at 09:28:16AM -0500, djordje senicic via Lists.Yoctoproject.Org wrote:
> - CMEM memory block increased to 512MB to enable TIDL using 4xEVE and 2xDSP
> 
> Signed-off-by: Djordje Senicic <x0157990@ti.com>
> ---
>  recipes-kernel/linux/cmem.inc                 |  9 +++--
>  .../linux/files/dra7xx/cmem-am572i.dtsi       | 40 +++++++++++++++++++
>  2 files changed, 45 insertions(+), 4 deletions(-)
>  create mode 100644 recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> 
> diff --git a/recipes-kernel/linux/cmem.inc b/recipes-kernel/linux/cmem.inc
> index 8bd643f4..8d33fe35 100644
> --- a/recipes-kernel/linux/cmem.inc
> +++ b/recipes-kernel/linux/cmem.inc
> @@ -2,14 +2,15 @@
>  #
>  # Use these for temporary overrides
>  CMEM_MACHINE = "${MACHINE}"
> -CMEM_MACHINE_am57xx-evm = "am571x am572x am574x"
> -CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am574x"
> +CMEM_MACHINE_am57xx-evm = "am571x am572x am572i am574x"
> +CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am572i am574x"
>  CMEM_MACHINE_dra7xx-evm = "dra71x dra72x dra74x dra76x"
>  CMEM_MACHINE_dra7xx-hs-evm = "dra71x dra72x dra74x dra76x"
>  
>  # Set cmem.dtsi per machine or machine variant
>  CMEM_DTSI = "cmem.dtsi"
>  CMEM_DTSI_am571x = "cmem-am571x.dtsi"
> +CMEM_DTSI_am572i = "cmem-am572i.dtsi"

Why not use the full "idk" instead of a single "i" in dtsi and machine names - 
a single change from "x" to "i" is too subtle and can be missed/forgotten in 
the future...


>  CMEM_DTSI_am574x = "cmem-am574x.dtsi"
>  CMEM_DTSI_dra71x = "cmem-dra71x.dtsi"
>  CMEM_DTSI_dra72x = "cmem-dra72x.dtsi"
> @@ -18,8 +19,8 @@ CMEM_DTSI_dra76x = "cmem-dra76x.dtsi"
>  # Split device trees between variants
>  CMEM_DEVICETREE = "${KERNEL_DEVICETREE}"
>  CMEM_DEVICETREE_am571x = "am571x-idk.dtb am571x-idk-lcd-osd101t2045.dtb am571x-idk-lcd-osd101t2587.dtb"
> -CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb \
> -                          am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
> +CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb"
> +CMEM_DEVICETREE_am572i = "am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
>  CMEM_DEVICETREE_am574x = "am574x-idk.dtb am574x-idk-lcd-osd101t2587.dtb"
>  CMEM_DEVICETREE_dra71x = "dra71-evm.dtb"
>  CMEM_DEVICETREE_dra72x = "dra72-evm.dtb dra72-evm-lcd-osd101t2045.dtb dra72-evm-lcd-osd101t2587.dtb \
> diff --git a/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> new file mode 100644
> index 00000000..6c39e22c
> --- /dev/null
> +++ b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> @@ -0,0 +1,40 @@
> +/ {
> +        reserved-memory {
> +                #address-cells = <2>;
> +                #size-cells = <2>;
> +                ranges;
> +
> +                cmem_block_mem_0: cmem_block_mem@a0000000 {
> +                        reg = <0x0 0xa0000000 0x0 0x20000000>;
> +                        no-map;
> +                        status = "okay";
> +                };
> +
> +		cmem_block_mem_1_ocmc3: cmem_block_mem@40500000 {
> +			reg = <0x0 0x40500000 0x0 0x100000>;
> +			no-map;
> +			status = "okay";
> +		};
> +        };
> +
> +        cmem {
> +                compatible = "ti,cmem";
> +                #address-cells = <1>;
> +                #size-cells = <0>;
> +
> +		#pool-size-cells = <2>;
> +
> +                status = "okay";
> +
> +                cmem_block_0: cmem_block@0 {
> +                        reg = <0>;
> +                        memory-region = <&cmem_block_mem_0>;
> +                        cmem-buf-pools = <1 0x0 0x20000000>;
> +                };
> +
> +		cmem_block_1: cmem_block@1 {
> +			reg = <1>;
> +			memory-region = <&cmem_block_mem_1_ocmc3>;
> +		};
> +        };
> +};
> -- 
> 2.17.1
> 

> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> 
> View/Reply Online (#12533): https://lists.yoctoproject.org/g/meta-ti/message/12533
> Mute This Topic: https://lists.yoctoproject.org/mt/62541901/1051543
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub  [denys@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration
  2019-12-05 19:55 ` Denys Dmytriyenko
@ 2019-12-05 22:30   ` djordje senicic
  0 siblings, 0 replies; 3+ messages in thread
From: djordje senicic @ 2019-12-05 22:30 UTC (permalink / raw)
  To: Dmytriyenko, Denys
  Cc: meta-ti@yoctoproject.org, meta-ti@lists.yoctoproject.org

Addressed naming ("i"->"idk") in PATCHv2: https://lists.yoctoproject.org/g/meta-ti/message/12552 

-----Original Message-----
From: Dmytriyenko, Denys 
Sent: Thursday, December 5, 2019 2:56 PM
To: Senicic, Djordje
Cc: meta-ti@yoctoproject.org; meta-ti@lists.yoctoproject.org
Subject: Re: [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration

On Wed, Nov 27, 2019 at 09:28:16AM -0500, djordje senicic via Lists.Yoctoproject.Org wrote:
> - CMEM memory block increased to 512MB to enable TIDL using 4xEVE and 2xDSP
> 
> Signed-off-by: Djordje Senicic <x0157990@ti.com>
> ---
>  recipes-kernel/linux/cmem.inc                 |  9 +++--
>  .../linux/files/dra7xx/cmem-am572i.dtsi       | 40 +++++++++++++++++++
>  2 files changed, 45 insertions(+), 4 deletions(-)
>  create mode 100644 recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> 
> diff --git a/recipes-kernel/linux/cmem.inc b/recipes-kernel/linux/cmem.inc
> index 8bd643f4..8d33fe35 100644
> --- a/recipes-kernel/linux/cmem.inc
> +++ b/recipes-kernel/linux/cmem.inc
> @@ -2,14 +2,15 @@
>  #
>  # Use these for temporary overrides
>  CMEM_MACHINE = "${MACHINE}"
> -CMEM_MACHINE_am57xx-evm = "am571x am572x am574x"
> -CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am574x"
> +CMEM_MACHINE_am57xx-evm = "am571x am572x am572i am574x"
> +CMEM_MACHINE_am57xx-hs-evm = "am571x am572x am572i am574x"
>  CMEM_MACHINE_dra7xx-evm = "dra71x dra72x dra74x dra76x"
>  CMEM_MACHINE_dra7xx-hs-evm = "dra71x dra72x dra74x dra76x"
>  
>  # Set cmem.dtsi per machine or machine variant
>  CMEM_DTSI = "cmem.dtsi"
>  CMEM_DTSI_am571x = "cmem-am571x.dtsi"
> +CMEM_DTSI_am572i = "cmem-am572i.dtsi"

Why not use the full "idk" instead of a single "i" in dtsi and machine names - 
a single change from "x" to "i" is too subtle and can be missed/forgotten in 
the future...


>  CMEM_DTSI_am574x = "cmem-am574x.dtsi"
>  CMEM_DTSI_dra71x = "cmem-dra71x.dtsi"
>  CMEM_DTSI_dra72x = "cmem-dra72x.dtsi"
> @@ -18,8 +19,8 @@ CMEM_DTSI_dra76x = "cmem-dra76x.dtsi"
>  # Split device trees between variants
>  CMEM_DEVICETREE = "${KERNEL_DEVICETREE}"
>  CMEM_DEVICETREE_am571x = "am571x-idk.dtb am571x-idk-lcd-osd101t2045.dtb am571x-idk-lcd-osd101t2587.dtb"
> -CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb \
> -                          am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
> +CMEM_DEVICETREE_am572x = "am57xx-beagle-x15.dtb am57xx-beagle-x15-revc.dtb"
> +CMEM_DEVICETREE_am572i = "am572x-idk.dtb am572x-idk-lcd-osd101t2045.dtb am572x-idk-lcd-osd101t2587.dtb"
>  CMEM_DEVICETREE_am574x = "am574x-idk.dtb am574x-idk-lcd-osd101t2587.dtb"
>  CMEM_DEVICETREE_dra71x = "dra71-evm.dtb"
>  CMEM_DEVICETREE_dra72x = "dra72-evm.dtb dra72-evm-lcd-osd101t2045.dtb dra72-evm-lcd-osd101t2587.dtb \
> diff --git a/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> new file mode 100644
> index 00000000..6c39e22c
> --- /dev/null
> +++ b/recipes-kernel/linux/files/dra7xx/cmem-am572i.dtsi
> @@ -0,0 +1,40 @@
> +/ {
> +        reserved-memory {
> +                #address-cells = <2>;
> +                #size-cells = <2>;
> +                ranges;
> +
> +                cmem_block_mem_0: cmem_block_mem@a0000000 {
> +                        reg = <0x0 0xa0000000 0x0 0x20000000>;
> +                        no-map;
> +                        status = "okay";
> +                };
> +
> +		cmem_block_mem_1_ocmc3: cmem_block_mem@40500000 {
> +			reg = <0x0 0x40500000 0x0 0x100000>;
> +			no-map;
> +			status = "okay";
> +		};
> +        };
> +
> +        cmem {
> +                compatible = "ti,cmem";
> +                #address-cells = <1>;
> +                #size-cells = <0>;
> +
> +		#pool-size-cells = <2>;
> +
> +                status = "okay";
> +
> +                cmem_block_0: cmem_block@0 {
> +                        reg = <0>;
> +                        memory-region = <&cmem_block_mem_0>;
> +                        cmem-buf-pools = <1 0x0 0x20000000>;
> +                };
> +
> +		cmem_block_1: cmem_block@1 {
> +			reg = <1>;
> +			memory-region = <&cmem_block_mem_1_ocmc3>;
> +		};
> +        };
> +};
> -- 
> 2.17.1
> 

> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> 
> View/Reply Online (#12533): https://lists.yoctoproject.org/g/meta-ti/message/12533
> Mute This Topic: https://lists.yoctoproject.org/mt/62541901/1051543
> Group Owner: meta-ti+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub  [denys@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2019-12-05 22:30 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-11-27 14:28 [meta-ti][thud/master][PATCH] cmem: Adding am5729idk specific CMEM configuration djordje senicic
2019-12-05 19:55 ` Denys Dmytriyenko
2019-12-05 22:30   ` djordje senicic

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.