[PATCH v2 1/1] nvmem: imx-ocotp: Reverse MAC addresses on all i.MX derivates

[Srinivas Kandagatla]

On 04/05/2023 07:05, Alexander Stein wrote:
> Not just i.MX8M, but all i.MX6/7 (and subtypes) need to reverse the
> MAC address read from fuses. Exceptions are i.MX6SLL and i.MX7ULP which
> do not support ethernet at all.
> 
> Fixes: d0221a780cbc ("nvmem: imx-ocotp: add support for post processing")
> Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>
> ---

Applied thanks,

--srini
> Changes in v2:
> * Instead of adding the flag for all platforms, apply post processing by
>    default
> 
>   drivers/nvmem/imx-ocotp.c | 8 +-------
>   1 file changed, 1 insertion(+), 7 deletions(-)
> 
> diff --git a/drivers/nvmem/imx-ocotp.c b/drivers/nvmem/imx-ocotp.c
> index ac0edb6398f1..c1af27105227 100644
> --- a/drivers/nvmem/imx-ocotp.c
> +++ b/drivers/nvmem/imx-ocotp.c
> @@ -97,7 +97,6 @@ struct ocotp_params {
>   	unsigned int bank_address_words;
>   	void (*set_timing)(struct ocotp_priv *priv);
>   	struct ocotp_ctrl_reg ctrl;
> -	bool reverse_mac_address;
>   };
>   
>   static int imx_ocotp_wait_for_busy(struct ocotp_priv *priv, u32 flags)
> @@ -545,7 +544,6 @@ static const struct ocotp_params imx8mq_params = {
>   	.bank_address_words = 0,
>   	.set_timing = imx_ocotp_set_imx6_timing,
>   	.ctrl = IMX_OCOTP_BM_CTRL_DEFAULT,
> -	.reverse_mac_address = true,
>   };
>   
>   static const struct ocotp_params imx8mm_params = {
> @@ -553,7 +551,6 @@ static const struct ocotp_params imx8mm_params = {
>   	.bank_address_words = 0,
>   	.set_timing = imx_ocotp_set_imx6_timing,
>   	.ctrl = IMX_OCOTP_BM_CTRL_DEFAULT,
> -	.reverse_mac_address = true,
>   };
>   
>   static const struct ocotp_params imx8mn_params = {
> @@ -561,7 +558,6 @@ static const struct ocotp_params imx8mn_params = {
>   	.bank_address_words = 0,
>   	.set_timing = imx_ocotp_set_imx6_timing,
>   	.ctrl = IMX_OCOTP_BM_CTRL_DEFAULT,
> -	.reverse_mac_address = true,
>   };
>   
>   static const struct ocotp_params imx8mp_params = {
> @@ -569,7 +565,6 @@ static const struct ocotp_params imx8mp_params = {
>   	.bank_address_words = 0,
>   	.set_timing = imx_ocotp_set_imx6_timing,
>   	.ctrl = IMX_OCOTP_BM_CTRL_8MP,
> -	.reverse_mac_address = true,
>   };
>   
>   static const struct of_device_id imx_ocotp_dt_ids[] = {
> @@ -624,8 +619,7 @@ static int imx_ocotp_probe(struct platform_device *pdev)
>   	imx_ocotp_nvmem_config.size = 4 * priv->params->nregs;
>   	imx_ocotp_nvmem_config.dev = dev;
>   	imx_ocotp_nvmem_config.priv = priv;
> -	if (priv->params->reverse_mac_address)
> -		imx_ocotp_nvmem_config.layout = &imx_ocotp_layout;
> +	imx_ocotp_nvmem_config.layout = &imx_ocotp_layout;
>   
>   	priv->config = &imx_ocotp_nvmem_config;
>