[PATCH 1/3] ARM: dts: at91: replace microchip, 24aa02e48 with atmel, at24c02

[PATCH 1/3] ARM: dts: at91: replace microchip, 24aa02e48 with atmel, at24c02

[Eugen Hristev]

microchip,24aa025e48 does not exist in the bindings of this driver.
It can be replaced with atmel,at24c02 which is a standard compatible
and the memory is compatible with this one, depending on the page size.
microchip 24aa02e48 has a page size of 8, while 24aa025e48 has a page
size of 16 bytes.

Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
---
 arch/arm/dts/at91-sama5d2_icp.dts | 6 +++---
 arch/arm/dts/at91-sama7g5ek.dts   | 4 ++--
 arch/arm/dts/sama5d27_som1.dtsi   | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/arch/arm/dts/at91-sama5d2_icp.dts b/arch/arm/dts/at91-sama5d2_icp.dts
index 44522197ff..0b0db1b2be 100644
--- a/arch/arm/dts/at91-sama5d2_icp.dts
+++ b/arch/arm/dts/at91-sama5d2_icp.dts
@@ -68,19 +68,19 @@
 				status = "okay";
 
 				eeprom@50 {
-					compatible = "microchip,24aa02e48";
+					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 					reg = <0x50>;
 					pagesize = <16>;
 				};
 
 				eeprom@52 {
-					compatible = "microchip,24aa02e48";
+					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 					reg = <0x52>;
 					pagesize = <16>;
 				};
 
 				eeprom@53 {
-					compatible = "microchip,24aa02e48";
+					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 					reg = <0x53>;
 					pagesize = <16>;
 				};
diff --git a/arch/arm/dts/at91-sama7g5ek.dts b/arch/arm/dts/at91-sama7g5ek.dts
index 086ee45005..eaba0de3f7 100644
--- a/arch/arm/dts/at91-sama7g5ek.dts
+++ b/arch/arm/dts/at91-sama7g5ek.dts
@@ -405,13 +405,13 @@
 		status = "okay";
 
 		eeprom@52 {
-			compatible = "microchip,24aa02e48";
+			compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 			reg = <0x52>;
 			pagesize = <16>;
 		};
 
 		eeprom@53 {
-			compatible = "microchip,24aa02e48";
+			compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 			reg = <0x53>;
 			pagesize = <16>;
 		};
diff --git a/arch/arm/dts/sama5d27_som1.dtsi b/arch/arm/dts/sama5d27_som1.dtsi
index ea7540bcfc..db4fefadcd 100644
--- a/arch/arm/dts/sama5d27_som1.dtsi
+++ b/arch/arm/dts/sama5d27_som1.dtsi
@@ -92,7 +92,7 @@
 				status = "okay";
 
 				i2c_eeprom: i2c_eeprom@50 {
-					compatible = "microchip,24aa02e48";
+					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa02e48 */
 					reg = <0x50>;
 				};
 			};
-- 
2.25.1

[PATCH 2/3] misc: i2c_eeprom: remove 24aa02e48

[Eugen Hristev]

This compatible does not exist in the bindings.
All occurences in DT have been replaced by at24c02 which is equivalent.

Fixes: 7264066707 ("misc: i2c_eeprom: Add compatible for 24AA02E48")
Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
---
 drivers/misc/i2c_eeprom.c | 8 --------
 1 file changed, 8 deletions(-)

diff --git a/drivers/misc/i2c_eeprom.c b/drivers/misc/i2c_eeprom.c
index 89a450d0f8..05bc803d8b 100644
--- a/drivers/misc/i2c_eeprom.c
+++ b/drivers/misc/i2c_eeprom.c
@@ -169,13 +169,6 @@ static const struct i2c_eeprom_drv_data eeprom_data = {
 	.offset_len = 1,
 };
 
-static const struct i2c_eeprom_drv_data mc24aa02e48_data = {
-	.size = 256,
-	.pagesize = 8,
-	.addr_offset_mask = 0,
-	.offset_len = 1,
-};
-
 static const struct i2c_eeprom_drv_data atmel24c01a_data = {
 	.size = 128,
 	.pagesize = 8,
@@ -263,7 +256,6 @@ static const struct i2c_eeprom_drv_data atmel24c512_data = {
 
 static const struct udevice_id i2c_eeprom_std_ids[] = {
 	{ .compatible = "i2c-eeprom", (ulong)&eeprom_data },
-	{ .compatible = "microchip,24aa02e48", (ulong)&mc24aa02e48_data },
 	{ .compatible = "atmel,24c01", (ulong)&atmel24c01a_data },
 	{ .compatible = "atmel,24c01a", (ulong)&atmel24c01a_data },
 	{ .compatible = "atmel,24c02", (ulong)&atmel24c02_data },
-- 
2.25.1

[PATCH 3/3] ARM: dts: at91: sam9x60ek: fix eeprom compatible

[Eugen Hristev]

The memory on this board is microchip 24aa025e48 which is compatible with
at24c02 with a page size of 16.
Fix the compatible accordingly.

Reported-by: Sergiu Moga <sergiu.moga@microchip.com>
Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
Tested-by: Sergiu Moga <sergiu.moga@microchip.com>
---
Hi,

I will also send a patch to Linux to fix this

Eugen

 arch/arm/dts/sam9x60ek.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/arm/dts/sam9x60ek.dts b/arch/arm/dts/sam9x60ek.dts
index 4258e8e1ab..54c694bd78 100644
--- a/arch/arm/dts/sam9x60ek.dts
+++ b/arch/arm/dts/sam9x60ek.dts
@@ -72,7 +72,7 @@
 					status = "okay";
 
 					eeprom@53 {
-						compatible = "atmel,24c32";
+						compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 24aa025e48, an at24c02 with page size of 16 */
 						reg = <0x53>;
 						pagesize = <16>;
 					};
-- 
2.25.1

Re: [PATCH 1/3] ARM: dts: at91: replace microchip,24aa02e48 with atmel,at24c02

[Michael Walle]

Am 2022-05-26 16:04, schrieb Eugen Hristev:
> microchip,24aa025e48 does not exist in the bindings of this driver.
> It can be replaced with atmel,at24c02 which is a standard compatible
> and the memory is compatible with this one, depending on the page size.
> microchip 24aa02e48 has a page size of 8, while 24aa025e48 has a page
> size of 16 bytes.

Nice. Thanks for taking care! One minor nit below.

> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
Reviewed-by: Michael Walle <michael@walle.cc>


> ---
>  arch/arm/dts/at91-sama5d2_icp.dts | 6 +++---
>  arch/arm/dts/at91-sama7g5ek.dts   | 4 ++--
>  arch/arm/dts/sama5d27_som1.dtsi   | 2 +-
>  3 files changed, 6 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm/dts/at91-sama5d2_icp.dts
> b/arch/arm/dts/at91-sama5d2_icp.dts
> index 44522197ff..0b0db1b2be 100644
> --- a/arch/arm/dts/at91-sama5d2_icp.dts
> +++ b/arch/arm/dts/at91-sama5d2_icp.dts
> @@ -68,19 +68,19 @@
>  				status = "okay";
> 
>  				eeprom@50 {
> -					compatible = "microchip,24aa02e48";
> +					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip
> 24aa025e48, an at24c02 with page size of 16 */
>  					reg = <0x50>;
>  					pagesize = <16>;
>  				};
> 
>  				eeprom@52 {
> -					compatible = "microchip,24aa02e48";
> +					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip
> 24aa025e48, an at24c02 with page size of 16 */
>  					reg = <0x52>;
>  					pagesize = <16>;
>  				};
> 
>  				eeprom@53 {
> -					compatible = "microchip,24aa02e48";
> +					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip
> 24aa025e48, an at24c02 with page size of 16 */
>  					reg = <0x53>;
>  					pagesize = <16>;
>  				};
> diff --git a/arch/arm/dts/at91-sama7g5ek.dts 
> b/arch/arm/dts/at91-sama7g5ek.dts
> index 086ee45005..eaba0de3f7 100644
> --- a/arch/arm/dts/at91-sama7g5ek.dts
> +++ b/arch/arm/dts/at91-sama7g5ek.dts
> @@ -405,13 +405,13 @@
>  		status = "okay";
> 
>  		eeprom@52 {
> -			compatible = "microchip,24aa02e48";
> +			compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip
> 24aa025e48, an at24c02 with page size of 16 */
>  			reg = <0x52>;
>  			pagesize = <16>;
>  		};
> 
>  		eeprom@53 {
> -			compatible = "microchip,24aa02e48";
> +			compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip
> 24aa025e48, an at24c02 with page size of 16 */
>  			reg = <0x53>;
>  			pagesize = <16>;
>  		};
> diff --git a/arch/arm/dts/sama5d27_som1.dtsi 
> b/arch/arm/dts/sama5d27_som1.dtsi
> index ea7540bcfc..db4fefadcd 100644
> --- a/arch/arm/dts/sama5d27_som1.dtsi
> +++ b/arch/arm/dts/sama5d27_som1.dtsi
> @@ -92,7 +92,7 @@
>  				status = "okay";
> 
>  				i2c_eeprom: i2c_eeprom@50 {

Nit: that one should actually be just "eeprom@50".

-michael

> -					compatible = "microchip,24aa02e48";
> +					compatible = "atmel,24c02"; /* EEPROM is 2Kbits microchip 
> 24aa02e48 */
>  					reg = <0x50>;
>  				};
>  			};

Re: [PATCH 2/3] misc: i2c_eeprom: remove 24aa02e48

[Michael Walle]

Am 2022-05-26 16:04, schrieb Eugen Hristev:
> This compatible does not exist in the bindings.
> All occurences in DT have been replaced by at24c02 which is equivalent.
> 
> Fixes: 7264066707 ("misc: i2c_eeprom: Add compatible for 24AA02E48")
> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>

Reviewed-by: Michael Walle <michael@walle.cc>

-michael

Re: [PATCH 1/3] ARM: dts: at91: replace microchip,24aa02e48 with atmel,at24c02

[Eugen.Hristev]

On 5/26/22 5:11 PM, Michael Walle wrote:

[snip]

>> @@ -92,7 +92,7 @@
>>                               status = "okay";
>>
>>                               i2c_eeprom: i2c_eeprom@50 {
> 
> Nit: that one should actually be just "eeprom@50".

Hi Michael,

That's true, but my patch just changes the compatible .

With time, I want to completely sync the DT with Linux. It will take 
some effort, but I am gathering the pieces of the puzzle slowly.

Thanks !

Eugen


> 
> -michael
> 
>> -                                     compatible = "microchip,24aa02e48";
>> +                                     compatible = "atmel,24c02"; /* 
>> EEPROM is 2Kbits microchip
>> 24aa02e48 */
>>                                       reg = <0x50>;
>>                               };
>>                       };

Re: [PATCH 1/3] ARM: dts: at91: replace microchip,24aa02e48 with atmel,at24c02

[Heiko Schocher]

Hello Eugen,

On 26.05.22 16:18, Eugen.Hristev@microchip.com wrote:
> On 5/26/22 5:11 PM, Michael Walle wrote:
> 
> [snip]
> 
>>> @@ -92,7 +92,7 @@
>>>                               status = "okay";
>>>
>>>                               i2c_eeprom: i2c_eeprom@50 {
>>
>> Nit: that one should actually be just "eeprom@50".
> 
> Hi Michael,
> 
> That's true, but my patch just changes the compatible .
> 
> With time, I want to completely sync the DT with Linux. It will take 
> some effort, but I am gathering the pieces of the puzzle slowly.
> 
> Thanks !

Fine with me.

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
> 
> Eugen
> 
> 
>>
>> -michael
>>
>>> -                                     compatible = "microchip,24aa02e48";
>>> +                                     compatible = "atmel,24c02"; /* 
>>> EEPROM is 2Kbits microchip
>>> 24aa02e48 */
>>>                                       reg = <0x50>;
>>>                               };
>>>                       };
> 

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

Re: [PATCH 2/3] misc: i2c_eeprom: remove 24aa02e48

[Heiko Schocher]

Hello Eugen,

On 26.05.22 16:04, Eugen Hristev wrote:
> This compatible does not exist in the bindings.
> All occurences in DT have been replaced by at24c02 which is equivalent.
> 
> Fixes: 7264066707 ("misc: i2c_eeprom: Add compatible for 24AA02E48")
> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
> ---
>  drivers/misc/i2c_eeprom.c | 8 --------
>  1 file changed, 8 deletions(-)

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

Re: [PATCH 3/3] ARM: dts: at91: sam9x60ek: fix eeprom compatible

[Heiko Schocher]

Hello Eugen,

On 26.05.22 16:04, Eugen Hristev wrote:
> The memory on this board is microchip 24aa025e48 which is compatible with
> at24c02 with a page size of 16.
> Fix the compatible accordingly.
> 
> Reported-by: Sergiu Moga <sergiu.moga@microchip.com>
> Signed-off-by: Eugen Hristev <eugen.hristev@microchip.com>
> Tested-by: Sergiu Moga <sergiu.moga@microchip.com>
> ---
> Hi,
> 
> I will also send a patch to Linux to fix this

Thanks!

> 
> Eugen
> 
>  arch/arm/dts/sam9x60ek.dts | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de

Re: [PATCH 1/3] ARM: dts: at91: replace microchip,24aa02e48 with atmel,at24c02

[Eugen.Hristev]

On 5/31/22 7:43 AM, Heiko Schocher wrote:
> Hello Eugen,
> 
> On 26.05.22 16:18, Eugen.Hristev@microchip.com wrote:
>> On 5/26/22 5:11 PM, Michael Walle wrote:
>>
>> [snip]
>>
>>>> @@ -92,7 +92,7 @@
>>>>                                status = "okay";
>>>>
>>>>                                i2c_eeprom: i2c_eeprom@50 {
>>>
>>> Nit: that one should actually be just "eeprom@50".
>>
>> Hi Michael,
>>
>> That's true, but my patch just changes the compatible .
>>
>> With time, I want to completely sync the DT with Linux. It will take
>> some effort, but I am gathering the pieces of the puzzle slowly.
>>
>> Thanks !
> 
> Fine with me.
> 
> Reviewed-by: Heiko Schocher <hs@denx.de>

Applied the series to u-boot-at91/next
Thanks for reviewing.

Eugen

> 
> bye,
> Heiko
>>
>> Eugen
>>
>>
>>>
>>> -michael
>>>
>>>> -                                     compatible = "microchip,24aa02e48";
>>>> +                                     compatible = "atmel,24c02"; /*
>>>> EEPROM is 2Kbits microchip
>>>> 24aa02e48 */
>>>>                                        reg = <0x50>;
>>>>                                };
>>>>                        };
>>
> 
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-52   Fax: +49-8142-66989-80   Email: hs@denx.de
>