Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-11-20 Thread Stefano Babic
On 31/10/2014 04:08, Peng Fan wrote:
> The PFUZ probe failed with the following msg:
> " wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=0
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=1
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=2
>   i2c_init_transfer: give up i2c_regs=021a
>   Can't find PMIC:PFUZE100 "
> 
> board_early_init_f is too early to call i2c related setting, because
> init_func_i2c is called after board_early_init_f being invoked. Thus
> move setup_i2c into board_init.
> 
> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
> 
> Using this patch PFUZ can be correctly probed:
> "PMIC:  PFUZE100 ID=0x11"
> 
> Signed-off-by: Peng Fan 
> ---

Applied to u-boot-imx, thanks !

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-11-17 Thread Stefano Babic
Hi Peng,

On 16/11/2014 12:42, Peng Fan wrote:

>>> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
>>>
>>> Using this patch PFUZ can be correctly probed:
>>> "PMIC:  PFUZE100 ID=0x11"
>>>
>>> Signed-off-by: Peng Fan 
>>
>> Acked-by: Fabio Estevam 

Patch is ok for me, too.

Acked-by: Stefano Babic 

Best regards,
Stefano Babic


-- 
=
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sba...@denx.de
=
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-11-16 Thread Peng Fan

Hi Stefano,

Would you please help reviewing this patch?

On 11/1/2014 8:16 AM, Fabio Estevam wrote:

On Fri, Oct 31, 2014 at 1:08 AM, Peng Fan  wrote:

The PFUZ probe failed with the following msg:
" wait_for_sr_state: failed sr=81 cr=a0 state=2020
   i2c_init_transfer: failed for chip 0x8 retry=0
   wait_for_sr_state: failed sr=81 cr=a0 state=2020
   i2c_init_transfer: failed for chip 0x8 retry=1
   wait_for_sr_state: failed sr=81 cr=a0 state=2020
   i2c_init_transfer: failed for chip 0x8 retry=2
   i2c_init_transfer: give up i2c_regs=021a
   Can't find PMIC:PFUZE100 "

board_early_init_f is too early to call i2c related setting, because
init_func_i2c is called after board_early_init_f being invoked. Thus
move setup_i2c into board_init.

Also PFUZ is connected to I2C bus 0, so change "1" -> "0".

Using this patch PFUZ can be correctly probed:
"PMIC:  PFUZE100 ID=0x11"

Signed-off-by: Peng Fan 


Acked-by: Fabio Estevam 


Regards,
Peng.
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-10-31 Thread Fabio Estevam
On Fri, Oct 31, 2014 at 1:08 AM, Peng Fan  wrote:
> The PFUZ probe failed with the following msg:
> " wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=0
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=1
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=2
>   i2c_init_transfer: give up i2c_regs=021a
>   Can't find PMIC:PFUZE100 "
>
> board_early_init_f is too early to call i2c related setting, because
> init_func_i2c is called after board_early_init_f being invoked. Thus
> move setup_i2c into board_init.
>
> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
>
> Using this patch PFUZ can be correctly probed:
> "PMIC:  PFUZE100 ID=0x11"
>
> Signed-off-by: Peng Fan 

Acked-by: Fabio Estevam 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-10-31 Thread Fabio Estevam
On Fri, Oct 31, 2014 at 10:01 PM, Peng Fan  wrote:
>
>
> On 10/31/2014 08:01 PM, Fabio Estevam wrote:
>> On Fri, Oct 31, 2014 at 1:08 AM, Peng Fan  wrote:
>>> The PFUZ probe failed with the following msg:
>>> " wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>>   i2c_init_transfer: failed for chip 0x8 retry=0
>>>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>>   i2c_init_transfer: failed for chip 0x8 retry=1
>>>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>>   i2c_init_transfer: failed for chip 0x8 retry=2
>>>   i2c_init_transfer: give up i2c_regs=021a
>>>   Can't find PMIC:PFUZE100 "
>>>
>>> board_early_init_f is too early to call i2c related setting, because
>>> init_func_i2c is called after board_early_init_f being invoked. Thus
>>> move setup_i2c into board_init.
>>>
>>> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
>>>
>>> Using this patch PFUZ can be correctly probed:
>>> "PMIC:  PFUZE100 ID=0x11"
>>>
>>> Signed-off-by: Peng Fan 
>>
>> Maybe this is a difference between RevA versus RevB board?
>
> I checked SCH-27962 REV A and SCH-27962 REV B. There is no different to I2C 
> part. PMIC is connected to the first I2C bus. You can check page 22.

You are right. I was surprised that PMIC was working originally. I
tested your change and it also works fine here.

Thanks!
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-10-31 Thread Peng Fan


On 10/31/2014 08:01 PM, Fabio Estevam wrote:
> On Fri, Oct 31, 2014 at 1:08 AM, Peng Fan  wrote:
>> The PFUZ probe failed with the following msg:
>> " wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>   i2c_init_transfer: failed for chip 0x8 retry=0
>>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>   i2c_init_transfer: failed for chip 0x8 retry=1
>>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>>   i2c_init_transfer: failed for chip 0x8 retry=2
>>   i2c_init_transfer: give up i2c_regs=021a
>>   Can't find PMIC:PFUZE100 "
>>
>> board_early_init_f is too early to call i2c related setting, because
>> init_func_i2c is called after board_early_init_f being invoked. Thus
>> move setup_i2c into board_init.
>>
>> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
>>
>> Using this patch PFUZ can be correctly probed:
>> "PMIC:  PFUZE100 ID=0x11"
>>
>> Signed-off-by: Peng Fan 
> 
> Maybe this is a difference between RevA versus RevB board?

I checked SCH-27962 REV A and SCH-27962 REV B. There is no different to I2C 
part. PMIC is connected to the first I2C bus. You can check page 22.

> 
> I have a RevA and PMIC works fine here:
I tried on RevA and RevB. Both can not work.
The following is the msg on RevA:

U-Boot 2014.10-00435-g571bdf1 (Nov 01 2014 - 07:39:47)

CPU:   Freescale i.MX6SX rev1.0 at 792 MHz
Reset cause: POR
Board: MX6SX SABRE SDB
I2C:   ready
DRAM:  1 GiB
MMC:   FSL_SDHC: 0
*** Warning - bad CRC, using default environment

In:serial
Out:   serial
Err:   serial
wait_for_sr_state: failed sr=81 cr=a0 state=2020
i2c_init_transfer: failed for chip 0x8 retry=0
wait_for_sr_state: failed sr=81 cr=a0 state=2020
i2c_init_transfer: failed for chip 0x8 retry=1
wait_for_sr_state: failed sr=81 cr=a0 state=2020
i2c_init_transfer: failed for chip 0x8 retry=2
i2c_init_transfer: give up i2c_regs=021a
Can't find PMIC:PFUZE100
Net:   FEC [PRIME]
Hit any key to stop autoboot:  0

After applying this patch, all is fine. I am not sure whether your board has 
been reworked. But PMIC is connected to the first I2C bus, thus I think 
setup_i2c(0,) is correct, but not setup_i2c(1,xxx). And I think it is too 
early to call setup_i2c in board_early_init_f.

> 
> U-Boot 2014.10-rc2-17115-g718b923 (Sep 30 2014 - 14:03:24)
> 
> CPU:   Freescale i.MX6SX rev1.0 at 792 MHz
> Reset cause: WDOG
> Board: MX6SX SABRE SDB
> I2C:   ready
> DRAM:  1 GiB
> MMC:   FSL_SDHC: 0
> In:serial
> Out:   serial
> Err:   serial
> PMIC:  PFUZE100 ID=0x10
> Net:   FEC [PRIME]
> Hit any key to stop autoboot:  0
> =>
> 
> => pmic PFUZE100 dump
> PMIC: PFUZE100
> 
> 0x00: 0010   0021  0001 003f 0001
> 0x08:  007f      0081
> 0x10:   003f     
> 0x18:    0010    
> 0x20: 002b 001b 002b 0008 0044   
> 0x28:       002b 001b
> 0x30: 002b 0008 0044   0072 0072 0072
> 0x38: 0008 0054   0026 0026 0026 0008
> 0x40: 0064   0026 0026 0026 0008 0064
> 0x48:   0038 0038 0038 0008 0074 
> 0x50:        
> 0x58:        
> 0x60:        
> 0x68:   0010 0006 0018 000e 001a 0010
> 0x70: 001f 001c      
> 0x78:       
> =>
> 
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-10-31 Thread Fabio Estevam
On Fri, Oct 31, 2014 at 1:08 AM, Peng Fan  wrote:
> The PFUZ probe failed with the following msg:
> " wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=0
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=1
>   wait_for_sr_state: failed sr=81 cr=a0 state=2020
>   i2c_init_transfer: failed for chip 0x8 retry=2
>   i2c_init_transfer: give up i2c_regs=021a
>   Can't find PMIC:PFUZE100 "
>
> board_early_init_f is too early to call i2c related setting, because
> init_func_i2c is called after board_early_init_f being invoked. Thus
> move setup_i2c into board_init.
>
> Also PFUZ is connected to I2C bus 0, so change "1" -> "0".
>
> Using this patch PFUZ can be correctly probed:
> "PMIC:  PFUZE100 ID=0x11"
>
> Signed-off-by: Peng Fan 

Maybe this is a difference between RevA versus RevB board?

I have a RevA and PMIC works fine here:

U-Boot 2014.10-rc2-17115-g718b923 (Sep 30 2014 - 14:03:24)

CPU:   Freescale i.MX6SX rev1.0 at 792 MHz
Reset cause: WDOG
Board: MX6SX SABRE SDB
I2C:   ready
DRAM:  1 GiB
MMC:   FSL_SDHC: 0
In:serial
Out:   serial
Err:   serial
PMIC:  PFUZE100 ID=0x10
Net:   FEC [PRIME]
Hit any key to stop autoboot:  0
=>

=> pmic PFUZE100 dump
PMIC: PFUZE100

0x00: 0010   0021  0001 003f 0001
0x08:  007f      0081
0x10:   003f     
0x18:    0010    
0x20: 002b 001b 002b 0008 0044   
0x28:       002b 001b
0x30: 002b 0008 0044   0072 0072 0072
0x38: 0008 0054   0026 0026 0026 0008
0x40: 0064   0026 0026 0026 0008 0064
0x48:   0038 0038 0038 0008 0074 
0x50:        
0x58:        
0x60:        
0x68:   0010 0006 0018 000e 001a 0010
0x70: 001f 001c      
0x78:       
=>
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


[U-Boot] [PATCH] imx:mx6sxsabresd fix pfuz probe failed

2014-10-30 Thread Peng Fan
The PFUZ probe failed with the following msg:
" wait_for_sr_state: failed sr=81 cr=a0 state=2020
  i2c_init_transfer: failed for chip 0x8 retry=0
  wait_for_sr_state: failed sr=81 cr=a0 state=2020
  i2c_init_transfer: failed for chip 0x8 retry=1
  wait_for_sr_state: failed sr=81 cr=a0 state=2020
  i2c_init_transfer: failed for chip 0x8 retry=2
  i2c_init_transfer: give up i2c_regs=021a
  Can't find PMIC:PFUZE100 "

board_early_init_f is too early to call i2c related setting, because
init_func_i2c is called after board_early_init_f being invoked. Thus
move setup_i2c into board_init.

Also PFUZ is connected to I2C bus 0, so change "1" -> "0".

Using this patch PFUZ can be correctly probed:
"PMIC:  PFUZE100 ID=0x11"

Signed-off-by: Peng Fan 
---
 board/freescale/mx6sxsabresd/mx6sxsabresd.c | 5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/board/freescale/mx6sxsabresd/mx6sxsabresd.c 
b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
index 68d3718..256ea29 100644
--- a/board/freescale/mx6sxsabresd/mx6sxsabresd.c
+++ b/board/freescale/mx6sxsabresd/mx6sxsabresd.c
@@ -243,7 +243,6 @@ int board_phy_config(struct phy_device *phydev)
 int board_early_init_f(void)
 {
setup_iomux_uart();
-   setup_i2c(1, CONFIG_SYS_I2C_SPEED, 0x7f, &i2c_pad_info1);
 
/* Enable PERI_3V3, which is used by SD2, ENET, LVDS, BT */
imx_iomux_v3_setup_multiple_pads(peri_3v3_pads,
@@ -277,6 +276,10 @@ int board_init(void)
/* Address of boot parameters */
gd->bd->bi_boot_params = PHYS_SDRAM + 0x100;
 
+#ifdef CONFIG_SYS_I2C_MXC
+   setup_i2c(0, CONFIG_SYS_I2C_SPEED, 0x7f, &i2c_pad_info1);
+#endif
+
return 0;
 }
 
-- 
1.8.4


___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot