[PATCH 02/12] arm64: dts: imx8qxp: add readonly for ocotp

Frank Li posted 12 patches 1 month, 2 weeks ago
[PATCH 02/12] arm64: dts: imx8qxp: add readonly for ocotp
Posted by Frank Li 1 month, 2 weeks ago
Add readonly for ocotp because i.MX8QXP only support program fuse by scu.

Signed-off-by: Frank Li <Frank.Li@nxp.com>
---
 arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
index def6de8d579d40919e4bf7f88e119611ae4cb69a..cc82cc319159c2558400fa641570cddd3ad3083d 100644
--- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
+++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
@@ -234,6 +234,7 @@ ocotp: ocotp {
 			compatible = "fsl,imx8qxp-scu-ocotp";
 			#address-cells = <1>;
 			#size-cells = <1>;
+			read-only;
 
 			fec_mac0: mac@2c4 {
 				reg = <0x2c4 6>;

-- 
2.34.1
Re: [PATCH 02/12] arm64: dts: imx8qxp: add readonly for ocotp
Posted by John Ernberg 1 month, 1 week ago
Hi Frank,

On Wed, Oct 29, 2025 at 03:54:38PM -0400, Frank Li wrote:
> Add readonly for ocotp because i.MX8QXP only support program fuse by scu.

This is not true? The driver supports writing the fuses via the SCFW API,
and we are using this way to populate the MAC fuses.

Best regards // John Ernberg
> 
> Signed-off-by: Frank Li <Frank.Li@nxp.com>
> ---
>  arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> index def6de8d579d40919e4bf7f88e119611ae4cb69a..cc82cc319159c2558400fa641570cddd3ad3083d 100644
> --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> @@ -234,6 +234,7 @@ ocotp: ocotp {
>  			compatible = "fsl,imx8qxp-scu-ocotp";
>  			#address-cells = <1>;
>  			#size-cells = <1>;
> +			read-only;
>  
>  			fec_mac0: mac@2c4 {
>  				reg = <0x2c4 6>;
> 
> -- 
> 2.34.1
> 
Re: [PATCH 02/12] arm64: dts: imx8qxp: add readonly for ocotp
Posted by Frank Li 1 month, 1 week ago
On Mon, Nov 03, 2025 at 11:22:55AM +0000, John Ernberg wrote:
> Hi Frank,
>
> On Wed, Oct 29, 2025 at 03:54:38PM -0400, Frank Li wrote:
> > Add readonly for ocotp because i.MX8QXP only support program fuse by scu.
>
> This is not true? The driver supports writing the fuses via the SCFW API,
> and we are using this way to populate the MAC fuses.

Yes!

Shanw:
	You can skip this patch. If need respin, I will drop it.

Frank
>
> Best regards // John Ernberg
> >
> > Signed-off-by: Frank Li <Frank.Li@nxp.com>
> > ---
> >  arch/arm64/boot/dts/freescale/imx8qxp.dtsi | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > index def6de8d579d40919e4bf7f88e119611ae4cb69a..cc82cc319159c2558400fa641570cddd3ad3083d 100644
> > --- a/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > +++ b/arch/arm64/boot/dts/freescale/imx8qxp.dtsi
> > @@ -234,6 +234,7 @@ ocotp: ocotp {
> >  			compatible = "fsl,imx8qxp-scu-ocotp";
> >  			#address-cells = <1>;
> >  			#size-cells = <1>;
> > +			read-only;
> >
> >  			fec_mac0: mac@2c4 {
> >  				reg = <0x2c4 6>;
> >
> > --
> > 2.34.1
> >