Re: [U-Boot] [PATCH 2/3] Kconfig: i2c: remove wrong help message related to dm i2c

2015-03-26 Thread Przemyslaw Marczak

Hello Simon,

On 03/24/2015 12:39 AM, Simon Glass wrote:

Hi,

On 10 March 2015 at 04:30, Przemyslaw Marczak p.marc...@samsung.com wrote:

Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
Cc: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Mike Frysinger vap...@gentoo.org
Cc: Simon Glass s...@chromium.org
Cc: Heiko Schocher h...@denx.de
---
  drivers/i2c/Kconfig | 11 +--
  1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/drivers/i2c/Kconfig b/drivers/i2c/Kconfig
index 692810d..0a52ed9 100644
--- a/drivers/i2c/Kconfig
+++ b/drivers/i2c/Kconfig
@@ -2,16 +2,7 @@ config DM_I2C
 bool Enable Driver Model for I2C drivers
 depends on DM
 help
- Enable driver model for I2C. This SPI flash interface
- (spi_flash_probe(), spi_flash_write(), etc.) is then
- implemented by the SPI flash uclass. There is one standard
- SPI flash driver which knows how to probe most chips
- supported by U-Boot. The uclass interface is defined in
- include/spi_flash.h, but is currently fully compatible
- with the old interface to avoid confusion and duplication
- during the transition parent. SPI and SPI flash must be
- enabled together (it is not possible to use driver model
- for one and not the other).
+ Enable driver model for I2C.


That's too short IMO :-)

Can you add a few more details along the lines of what is described
for SPI, and also mention the COMPAT option?

Thanks for fixing this.



Ok, will add more details :)



  config DM_I2C_COMPAT
 bool Enable I2C compatibility layer
--
1.9.1



Regards,
Simon



Best regards,
--
Przemyslaw Marczak
Samsung RD Institute Poland
Samsung Electronics
p.marc...@samsung.com
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/3] Kconfig: i2c: remove wrong help message related to dm i2c

2015-03-23 Thread Lukasz Majewski
Hi Przemyslaw,

 Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
 Cc: Masahiro Yamada yamad...@jp.panasonic.com
 Cc: Mike Frysinger vap...@gentoo.org
 Cc: Simon Glass s...@chromium.org
 Cc: Heiko Schocher h...@denx.de
 ---
  drivers/i2c/Kconfig | 11 +--
  1 file changed, 1 insertion(+), 10 deletions(-)
 
 diff --git a/drivers/i2c/Kconfig b/drivers/i2c/Kconfig
 index 692810d..0a52ed9 100644
 --- a/drivers/i2c/Kconfig
 +++ b/drivers/i2c/Kconfig
 @@ -2,16 +2,7 @@ config DM_I2C
   bool Enable Driver Model for I2C drivers
   depends on DM
   help
 -   Enable driver model for I2C. This SPI flash interface
 -   (spi_flash_probe(), spi_flash_write(), etc.) is then
 -   implemented by the SPI flash uclass. There is one standard
 -   SPI flash driver which knows how to probe most chips
 -   supported by U-Boot. The uclass interface is defined in
 -   include/spi_flash.h, but is currently fully compatible
 -   with the old interface to avoid confusion and duplication
 -   during the transition parent. SPI and SPI flash must be
 -   enabled together (it is not possible to use driver model
 -   for one and not the other).
 +   Enable driver model for I2C.
  
  config DM_I2C_COMPAT
   bool Enable I2C compatibility layer

Reviewed-by: Lukasz Majewski l.majew...@samsung.com

-- 
Best regards,

Lukasz Majewski

Samsung RD Institute Poland (SRPOL) | Linux Platform Group
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCH 2/3] Kconfig: i2c: remove wrong help message related to dm i2c

2015-03-23 Thread Simon Glass
Hi,

On 10 March 2015 at 04:30, Przemyslaw Marczak p.marc...@samsung.com wrote:
 Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
 Cc: Masahiro Yamada yamad...@jp.panasonic.com
 Cc: Mike Frysinger vap...@gentoo.org
 Cc: Simon Glass s...@chromium.org
 Cc: Heiko Schocher h...@denx.de
 ---
  drivers/i2c/Kconfig | 11 +--
  1 file changed, 1 insertion(+), 10 deletions(-)

 diff --git a/drivers/i2c/Kconfig b/drivers/i2c/Kconfig
 index 692810d..0a52ed9 100644
 --- a/drivers/i2c/Kconfig
 +++ b/drivers/i2c/Kconfig
 @@ -2,16 +2,7 @@ config DM_I2C
 bool Enable Driver Model for I2C drivers
 depends on DM
 help
 - Enable driver model for I2C. This SPI flash interface
 - (spi_flash_probe(), spi_flash_write(), etc.) is then
 - implemented by the SPI flash uclass. There is one standard
 - SPI flash driver which knows how to probe most chips
 - supported by U-Boot. The uclass interface is defined in
 - include/spi_flash.h, but is currently fully compatible
 - with the old interface to avoid confusion and duplication
 - during the transition parent. SPI and SPI flash must be
 - enabled together (it is not possible to use driver model
 - for one and not the other).
 + Enable driver model for I2C.

That's too short IMO :-)

Can you add a few more details along the lines of what is described
for SPI, and also mention the COMPAT option?

Thanks for fixing this.


  config DM_I2C_COMPAT
 bool Enable I2C compatibility layer
 --
 1.9.1


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


[U-Boot] [PATCH 2/3] Kconfig: i2c: remove wrong help message related to dm i2c

2015-03-10 Thread Przemyslaw Marczak
Signed-off-by: Przemyslaw Marczak p.marc...@samsung.com
Cc: Masahiro Yamada yamad...@jp.panasonic.com
Cc: Mike Frysinger vap...@gentoo.org
Cc: Simon Glass s...@chromium.org
Cc: Heiko Schocher h...@denx.de
---
 drivers/i2c/Kconfig | 11 +--
 1 file changed, 1 insertion(+), 10 deletions(-)

diff --git a/drivers/i2c/Kconfig b/drivers/i2c/Kconfig
index 692810d..0a52ed9 100644
--- a/drivers/i2c/Kconfig
+++ b/drivers/i2c/Kconfig
@@ -2,16 +2,7 @@ config DM_I2C
bool Enable Driver Model for I2C drivers
depends on DM
help
- Enable driver model for I2C. This SPI flash interface
- (spi_flash_probe(), spi_flash_write(), etc.) is then
- implemented by the SPI flash uclass. There is one standard
- SPI flash driver which knows how to probe most chips
- supported by U-Boot. The uclass interface is defined in
- include/spi_flash.h, but is currently fully compatible
- with the old interface to avoid confusion and duplication
- during the transition parent. SPI and SPI flash must be
- enabled together (it is not possible to use driver model
- for one and not the other).
+ Enable driver model for I2C.
 
 config DM_I2C_COMPAT
bool Enable I2C compatibility layer
-- 
1.9.1

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