diff mbox

[v2] ARM: dts: imx28-m28: Remove unneeded partition nodes

Message ID 1461770872-26510-1-git-send-email-fabio.estevam@nxp.com (mailing list archive)
State New, archived
Headers show

Commit Message

Fabio Estevam April 27, 2016, 3:27 p.m. UTC
mtdparts is passed from command line, so there is no need to have a
default partitioning in device-tree.

Suggested-by: Marek Vasut <marex@denx.de>
Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
---
Changes since v1:
- Remove all partition nodes

 arch/arm/boot/dts/imx28-m28.dtsi | 26 --------------------------
 1 file changed, 26 deletions(-)

Comments

Marek Vasut April 27, 2016, 3:32 p.m. UTC | #1
On 04/27/2016 05:27 PM, Fabio Estevam wrote:
> mtdparts is passed from command line, so there is no need to have a
> default partitioning in device-tree.
> 
> Suggested-by: Marek Vasut <marex@denx.de>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>

Acked-by: Marek Vasut <marex@denx.de>

Thanks for doing this :)

> ---
> Changes since v1:
> - Remove all partition nodes
> 
>  arch/arm/boot/dts/imx28-m28.dtsi | 26 --------------------------
>  1 file changed, 26 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx28-m28.dtsi b/arch/arm/boot/dts/imx28-m28.dtsi
> index 759cc56..6cebaa6 100644
> --- a/arch/arm/boot/dts/imx28-m28.dtsi
> +++ b/arch/arm/boot/dts/imx28-m28.dtsi
> @@ -27,32 +27,6 @@
>  				pinctrl-names = "default";
>  				pinctrl-0 = <&gpmi_pins_a &gpmi_status_cfg>;
>  				status = "okay";
> -
> -				partition@0 {
> -					label = "bootloader";
> -					reg = <0x00000000 0x00300000>;
> -					read-only;
> -				};
> -
> -				partition@1 {
> -					label = "environment";
> -					reg = <0x00300000 0x00080000>;
> -				};
> -
> -				partition@2 {
> -					label = "redundant-environment";
> -					reg = <0x00380000 0x00080000>;
> -				};
> -
> -				partition@3 {
> -					label = "kernel";
> -					reg = <0x00400000 0x00400000>;
> -				};
> -
> -				partition@4 {
> -					label = "filesystem";
> -					reg = <0x00800000 0x0f800000>;
> -				};
>  			};
>  		};
>  
>
Shawn Guo May 3, 2016, 8:27 a.m. UTC | #2
On Wed, Apr 27, 2016 at 12:27:52PM -0300, Fabio Estevam wrote:
> mtdparts is passed from command line, so there is no need to have a
> default partitioning in device-tree.
> 
> Suggested-by: Marek Vasut <marex@denx.de>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>

Applied, thanks.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx28-m28.dtsi b/arch/arm/boot/dts/imx28-m28.dtsi
index 759cc56..6cebaa6 100644
--- a/arch/arm/boot/dts/imx28-m28.dtsi
+++ b/arch/arm/boot/dts/imx28-m28.dtsi
@@ -27,32 +27,6 @@ 
 				pinctrl-names = "default";
 				pinctrl-0 = <&gpmi_pins_a &gpmi_status_cfg>;
 				status = "okay";
-
-				partition@0 {
-					label = "bootloader";
-					reg = <0x00000000 0x00300000>;
-					read-only;
-				};
-
-				partition@1 {
-					label = "environment";
-					reg = <0x00300000 0x00080000>;
-				};
-
-				partition@2 {
-					label = "redundant-environment";
-					reg = <0x00380000 0x00080000>;
-				};
-
-				partition@3 {
-					label = "kernel";
-					reg = <0x00400000 0x00400000>;
-				};
-
-				partition@4 {
-					label = "filesystem";
-					reg = <0x00800000 0x0f800000>;
-				};
 			};
 		};