[PATCH RFC] imx6ul-phytec-phycore-som: don't use fsl, no-blockmark-swap for the NAND

[PATCH RFC] imx6ul-phytec-phycore-som: don't use fsl, no-blockmark-swap for the NAND

[Uwe Kleine-König]

barebox doesn't honor that property which yields to inconsitencies
between Linux and barebox after for example filling an UBI with a file
system with barebox and then trying to mount that from Linux.

Note this is an interruptive change as it requires to rewrite the flash
to make it working with Linux again.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
Hello,

I wonder why Phytec chose to use this configuration. Was this just
copied from another board?

Best regards
Uwe

 arch/arm/dts/imx6ul-phytec-phycore-som.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi b/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
index d829fdd6fb29..398546d7b92e 100644
--- a/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
+++ b/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
@@ -49,7 +49,6 @@
 	pinctrl-names = "default";
 	pinctrl-0 = <&pinctrl_gpmi_nand>;
 	nand-on-flash-bbt;
-	fsl,no-blockmark-swap;
 	status = "disabled";
 
 	#address-cells = <1>;
-- 
2.20.1


_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH RFC] imx6ul-phytec-phycore-som: don't use fsl,no-blockmark-swap for the NAND

[Stefan Riedmüller]

Hi Uwe,

On 08.01.19 11:24, Uwe Kleine-König wrote:
> barebox doesn't honor that property which yields to inconsitencies
> between Linux and barebox after for example filling an UBI with a file
> system with barebox and then trying to mount that from Linux.
> 
> Note this is an interruptive change as it requires to rewrite the flash
> to make it working with Linux again.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> Hello,
> 
> I wonder why Phytec chose to use this configuration. Was this just
> copied from another board?

I have to admit, I don't know why this configuration was added. I guess it was 
added unintentionally since we do not use this configuration in the kernel.

So from my side there is no objection to this patch.

Thanks for spoting this and best regards
Stefan

> 
> Best regards
> Uwe
> 
>   arch/arm/dts/imx6ul-phytec-phycore-som.dtsi | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi b/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
> index d829fdd6fb29..398546d7b92e 100644
> --- a/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
> +++ b/arch/arm/dts/imx6ul-phytec-phycore-som.dtsi
> @@ -49,7 +49,6 @@
>   	pinctrl-names = "default";
>   	pinctrl-0 = <&pinctrl_gpmi_nand>;
>   	nand-on-flash-bbt;
> -	fsl,no-blockmark-swap;
>   	status = "disabled";
>   
>   	#address-cells = <1>;
> 

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

Re: [PATCH RFC] imx6ul-phytec-phycore-som: don't use fsl, no-blockmark-swap for the NAND

[Sascha Hauer]

On Tue, Jan 08, 2019 at 11:24:12AM +0100, Uwe Kleine-König wrote:
> barebox doesn't honor that property which yields to inconsitencies
> between Linux and barebox after for example filling an UBI with a file
> system with barebox and then trying to mount that from Linux.
> 
> Note this is an interruptive change as it requires to rewrite the flash
> to make it working with Linux again.
> 
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
> ---
> Hello,
> 
> I wonder why Phytec chose to use this configuration. Was this just
> copied from another board?

Applied, thanks

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox