Re: [PATCH 1/3] mfd: use to_i2c_client

2015-12-28 Thread Chanwoo Choi
Hi Geliang,

On 2015년 12월 29일 00:00, Geliang Tang wrote:
> Use to_i2c_client() instead of open-coding it.
> 
> Signed-off-by: Geliang Tang 
> ---
>  drivers/mfd/88pm80x.c   | 4 ++--
>  drivers/mfd/88pm860x-core.c | 4 ++--
>  drivers/mfd/max14577.c  | 4 ++--
>  drivers/mfd/max77686.c  | 4 ++--
>  drivers/mfd/max77693.c  | 4 ++--
>  drivers/mfd/max77843.c  | 4 ++--
>  drivers/mfd/max8925-i2c.c   | 4 ++--
>  drivers/mfd/max8997.c   | 8 
>  drivers/mfd/max8998.c   | 8 
>  drivers/mfd/sec-core.c  | 4 ++--
>  10 files changed, 24 insertions(+), 24 deletions(-)

Reviewed-by: Chanwoo Choi 

Thanks,
Chanwoo Choi

[snip]
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/3] mfd: use to_i2c_client

2015-12-28 Thread Krzysztof Kozlowski
On 29.12.2015 00:00, Geliang Tang wrote:
> Use to_i2c_client() instead of open-coding it.
> 
> Signed-off-by: Geliang Tang 
> ---
>  drivers/mfd/88pm80x.c   | 4 ++--
>  drivers/mfd/88pm860x-core.c | 4 ++--
>  drivers/mfd/max14577.c  | 4 ++--
>  drivers/mfd/max77686.c  | 4 ++--
>  drivers/mfd/max77693.c  | 4 ++--
>  drivers/mfd/max77843.c  | 4 ++--
>  drivers/mfd/max8925-i2c.c   | 4 ++--
>  drivers/mfd/max8997.c   | 8 
>  drivers/mfd/max8998.c   | 8 
>  drivers/mfd/sec-core.c  | 4 ++--
>  10 files changed, 24 insertions(+), 24 deletions(-)

Reviewed-by: Krzysztof Kozlowski 

Best regards,
Krzysztof

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 1/3] mfd: use to_i2c_client

2015-12-28 Thread Geliang Tang
Use to_i2c_client() instead of open-coding it.

Signed-off-by: Geliang Tang 
---
 drivers/mfd/88pm80x.c   | 4 ++--
 drivers/mfd/88pm860x-core.c | 4 ++--
 drivers/mfd/max14577.c  | 4 ++--
 drivers/mfd/max77686.c  | 4 ++--
 drivers/mfd/max77693.c  | 4 ++--
 drivers/mfd/max77843.c  | 4 ++--
 drivers/mfd/max8925-i2c.c   | 4 ++--
 drivers/mfd/max8997.c   | 8 
 drivers/mfd/max8998.c   | 8 
 drivers/mfd/sec-core.c  | 4 ++--
 10 files changed, 24 insertions(+), 24 deletions(-)

diff --git a/drivers/mfd/88pm80x.c b/drivers/mfd/88pm80x.c
index 63445ea..3f24ecb 100644
--- a/drivers/mfd/88pm80x.c
+++ b/drivers/mfd/88pm80x.c
@@ -135,7 +135,7 @@ EXPORT_SYMBOL_GPL(pm80x_deinit);
 #ifdef CONFIG_PM_SLEEP
 static int pm80x_suspend(struct device *dev)
 {
-   struct i2c_client *client = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *client = to_i2c_client(dev);
struct pm80x_chip *chip = i2c_get_clientdata(client);
 
if (chip && chip->wu_flag)
@@ -147,7 +147,7 @@ static int pm80x_suspend(struct device *dev)
 
 static int pm80x_resume(struct device *dev)
 {
-   struct i2c_client *client = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *client = to_i2c_client(dev);
struct pm80x_chip *chip = i2c_get_clientdata(client);
 
if (chip && chip->wu_flag)
diff --git a/drivers/mfd/88pm860x-core.c b/drivers/mfd/88pm860x-core.c
index 3269a99..e497cee 100644
--- a/drivers/mfd/88pm860x-core.c
+++ b/drivers/mfd/88pm860x-core.c
@@ -1218,7 +1218,7 @@ static int pm860x_remove(struct i2c_client *client)
 #ifdef CONFIG_PM_SLEEP
 static int pm860x_suspend(struct device *dev)
 {
-   struct i2c_client *client = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *client = to_i2c_client(dev);
struct pm860x_chip *chip = i2c_get_clientdata(client);
 
if (device_may_wakeup(dev) && chip->wakeup_flag)
@@ -1228,7 +1228,7 @@ static int pm860x_suspend(struct device *dev)
 
 static int pm860x_resume(struct device *dev)
 {
-   struct i2c_client *client = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *client = to_i2c_client(dev);
struct pm860x_chip *chip = i2c_get_clientdata(client);
 
if (device_may_wakeup(dev) && chip->wakeup_flag)
diff --git a/drivers/mfd/max14577.c b/drivers/mfd/max14577.c
index 56e216d..2280b3f 100644
--- a/drivers/mfd/max14577.c
+++ b/drivers/mfd/max14577.c
@@ -495,7 +495,7 @@ MODULE_DEVICE_TABLE(i2c, max14577_i2c_id);
 #ifdef CONFIG_PM_SLEEP
 static int max14577_suspend(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max14577 *max14577 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev))
@@ -516,7 +516,7 @@ static int max14577_suspend(struct device *dev)
 
 static int max14577_resume(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max14577 *max14577 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev))
diff --git a/drivers/mfd/max77686.c b/drivers/mfd/max77686.c
index d19be64..d959ebb 100644
--- a/drivers/mfd/max77686.c
+++ b/drivers/mfd/max77686.c
@@ -352,7 +352,7 @@ MODULE_DEVICE_TABLE(i2c, max77686_i2c_id);
 #ifdef CONFIG_PM_SLEEP
 static int max77686_suspend(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max77686_dev *max77686 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev))
@@ -374,7 +374,7 @@ static int max77686_suspend(struct device *dev)
 
 static int max77686_resume(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max77686_dev *max77686 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev))
diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
index 007f729..b83b7a7 100644
--- a/drivers/mfd/max77693.c
+++ b/drivers/mfd/max77693.c
@@ -334,7 +334,7 @@ MODULE_DEVICE_TABLE(i2c, max77693_i2c_id);
 
 static int max77693_suspend(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max77693_dev *max77693 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev)) {
@@ -347,7 +347,7 @@ static int max77693_suspend(struct device *dev)
 
 static int max77693_resume(struct device *dev)
 {
-   struct i2c_client *i2c = container_of(dev, struct i2c_client, dev);
+   struct i2c_client *i2c = to_i2c_client(dev);
struct max77693_dev *max77693 = i2c_get_clientdata(i2c);
 
if (device_may_wakeup(dev)) {
diff --git a/drivers/mfd/max77843.c b/drivers/mfd/max77843.c
index 586098f..7cf