public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names
@ 2019-04-10 23:00 Pierre Bourdon
  2019-04-11  8:49 ` Stefan Roese
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Pierre Bourdon @ 2019-04-10 23:00 UTC (permalink / raw)
  To: u-boot

Commit c4bd12a7dad4 ("i2c: mux: Generate longer i2c mux name") changed
the naming scheme of i2c devices within a mux. This broke references to
i2c at 0 in the Turris Omnia board initialization code.

Signed-off-by: Pierre Bourdon <delroth@gmail.com>
Cc: Marek Behún <marek.behun@nic.cz>
---
 board/CZ.NIC/turris_omnia/turris_omnia.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c b/board/CZ.NIC/turris_omnia/turris_omnia.c
index c7f6479a0c..c54cd047c1 100644
--- a/board/CZ.NIC/turris_omnia/turris_omnia.c
+++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
@@ -32,13 +32,13 @@
 
 DECLARE_GLOBAL_DATA_PTR;
 
-#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 0"
+#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 11000->i2cmux at 70->i2c at 0"
 #define OMNIA_I2C_EEPROM		0x54
 #define OMNIA_I2C_EEPROM_CONFIG_ADDR	0x0
 #define OMNIA_I2C_EEPROM_ADDRLEN	2
 #define OMNIA_I2C_EEPROM_MAGIC		0x0341a034
 
-#define OMNIA_I2C_MCU_DM_NAME		"i2c at 0"
+#define OMNIA_I2C_MCU_DM_NAME		"i2c at 11000->i2cmux at 70->i2c at 0"
 #define OMNIA_I2C_MCU_ADDR_STATUS	0x1
 #define OMNIA_I2C_MCU_SATA		0x20
 #define OMNIA_I2C_MCU_CARDDET		0x10
-- 
2.19.2

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names
  2019-04-10 23:00 [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names Pierre Bourdon
@ 2019-04-11  8:49 ` Stefan Roese
  2019-04-18 16:39 ` Marek Behún
  2019-04-26 10:41 ` Stefan Roese
  2 siblings, 0 replies; 4+ messages in thread
From: Stefan Roese @ 2019-04-11  8:49 UTC (permalink / raw)
  To: u-boot

Hi Pierre,

On 11.04.19 01:00, Pierre Bourdon wrote:
> Commit c4bd12a7dad4 ("i2c: mux: Generate longer i2c mux name") changed
> the naming scheme of i2c devices within a mux. This broke references to
> i2c at 0 in the Turris Omnia board initialization code.
> 
> Signed-off-by: Pierre Bourdon <delroth@gmail.com>
> Cc: Marek Behún <marek.behun@nic.cz>
> ---
>   board/CZ.NIC/turris_omnia/turris_omnia.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c b/board/CZ.NIC/turris_omnia/turris_omnia.c
> index c7f6479a0c..c54cd047c1 100644
> --- a/board/CZ.NIC/turris_omnia/turris_omnia.c
> +++ b/board/CZ.NIC/turris_omnia/turris_omnia.c
> @@ -32,13 +32,13 @@
>   
>   DECLARE_GLOBAL_DATA_PTR;
>   
> -#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 0"
> +#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 11000->i2cmux at 70->i2c at 0"
>   #define OMNIA_I2C_EEPROM		0x54
>   #define OMNIA_I2C_EEPROM_CONFIG_ADDR	0x0
>   #define OMNIA_I2C_EEPROM_ADDRLEN	2
>   #define OMNIA_I2C_EEPROM_MAGIC		0x0341a034
>   
> -#define OMNIA_I2C_MCU_DM_NAME		"i2c at 0"
> +#define OMNIA_I2C_MCU_DM_NAME		"i2c at 11000->i2cmux at 70->i2c at 0"
>   #define OMNIA_I2C_MCU_ADDR_STATUS	0x1
>   #define OMNIA_I2C_MCU_SATA		0x20
>   #define OMNIA_I2C_MCU_CARDDET		0x10
> 

Look good, so:

Reviewed-by: Stefan Roese <sr@denx.de>

Please use "arm: mvebu: turris_omnia: ..." as the patch subject next
time and Cc me (or the corresponding subsystem maintainer) next time.
No need to re-send now.

Thanks,
Stefan

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names
  2019-04-10 23:00 [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names Pierre Bourdon
  2019-04-11  8:49 ` Stefan Roese
@ 2019-04-18 16:39 ` Marek Behún
  2019-04-26 10:41 ` Stefan Roese
  2 siblings, 0 replies; 4+ messages in thread
From: Marek Behún @ 2019-04-18 16:39 UTC (permalink / raw)
  To: u-boot

Pierre, please add

Reviewed-by: Marek Behún <marek.behun@nic.cz>

and send also to Stefan Roese <sr@denx.de>

Marek

On Thu, 11 Apr 2019 01:00:23 +0200
Pierre Bourdon <delroth@gmail.com> wrote:

> Commit c4bd12a7dad4 ("i2c: mux: Generate longer i2c mux name") changed
> the naming scheme of i2c devices within a mux. This broke references
> to i2c at 0 in the Turris Omnia board initialization code.
> 
> Signed-off-by: Pierre Bourdon <delroth@gmail.com>
> Cc: Marek Behún <marek.behun@nic.cz>
> ---
>  board/CZ.NIC/turris_omnia/turris_omnia.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/board/CZ.NIC/turris_omnia/turris_omnia.c
> b/board/CZ.NIC/turris_omnia/turris_omnia.c index
> c7f6479a0c..c54cd047c1 100644 ---
> a/board/CZ.NIC/turris_omnia/turris_omnia.c +++
> b/board/CZ.NIC/turris_omnia/turris_omnia.c @@ -32,13 +32,13 @@
>  
>  DECLARE_GLOBAL_DATA_PTR;
>  
> -#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 0"
> +#define OMNIA_I2C_EEPROM_DM_NAME	"i2c at 11000->i2cmux at 70->i2c at 0"
>  #define OMNIA_I2C_EEPROM		0x54
>  #define OMNIA_I2C_EEPROM_CONFIG_ADDR	0x0
>  #define OMNIA_I2C_EEPROM_ADDRLEN	2
>  #define OMNIA_I2C_EEPROM_MAGIC		0x0341a034
>  
> -#define OMNIA_I2C_MCU_DM_NAME		"i2c at 0"
> +#define
> OMNIA_I2C_MCU_DM_NAME		"i2c at 11000->i2cmux at 70->i2c at 0"
> #define OMNIA_I2C_MCU_ADDR_STATUS	0x1 #define
> OMNIA_I2C_MCU_SATA		0x20 #define
> OMNIA_I2C_MCU_CARDDET		0x10

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names
  2019-04-10 23:00 [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names Pierre Bourdon
  2019-04-11  8:49 ` Stefan Roese
  2019-04-18 16:39 ` Marek Behún
@ 2019-04-26 10:41 ` Stefan Roese
  2 siblings, 0 replies; 4+ messages in thread
From: Stefan Roese @ 2019-04-26 10:41 UTC (permalink / raw)
  To: u-boot

On 11.04.19 01:00, Pierre Bourdon wrote:
> Commit c4bd12a7dad4 ("i2c: mux: Generate longer i2c mux name") changed
> the naming scheme of i2c devices within a mux. This broke references to
> i2c at 0 in the Turris Omnia board initialization code.
> 
> Signed-off-by: Pierre Bourdon <delroth@gmail.com>
> Cc: Marek Behún <marek.behun@nic.cz>

Applied to u-boot-marvell/master.

Thanks,
Stefan

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2019-04-26 10:41 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-04-10 23:00 [U-Boot] [PATCH] turris_omnia: fix eeprom/mcu device names Pierre Bourdon
2019-04-11  8:49 ` Stefan Roese
2019-04-18 16:39 ` Marek Behún
2019-04-26 10:41 ` Stefan Roese

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox