Re: [PATCH Resend] extcon: Fix return value in extcon_register_interest()

2012-09-25 Thread Chanwoo Choi
On 09/25/2012 08:01 PM, Sachin Kamat wrote:

> Propagate the value returned from extcon_find_cable_index()
> instead of -ENODEV. For readability, -EINVAL is returned in place of
> the variable.
> 
> Signed-off-by: Sachin Kamat 
> ---

Applied, thank you.

You can check it after some hours at below git repository for Extcon.
-
http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/extcon-for-next

Thanks,
Chanwoo Choi
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH Resend] extcon: Fix return value in extcon_register_interest()

2012-09-25 Thread Sachin Kamat
Propagate the value returned from extcon_find_cable_index()
instead of -ENODEV. For readability, -EINVAL is returned in place of
the variable.

Signed-off-by: Sachin Kamat 
---
 drivers/extcon/extcon-class.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/extcon/extcon-class.c b/drivers/extcon/extcon-class.c
index 946a318..6348b64 100644
--- a/drivers/extcon/extcon-class.c
+++ b/drivers/extcon/extcon-class.c
@@ -475,7 +475,7 @@ int extcon_register_interest(struct 
extcon_specific_cable_nb *obj,
 
obj->cable_index = extcon_find_cable_index(obj->edev, cable_name);
if (obj->cable_index < 0)
-   return -ENODEV;
+   return -EINVAL;
 
obj->user_nb = nb;
 
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH Resend] extcon: Fix return value in extcon_register_interest()

2012-09-25 Thread Sachin Kamat
Propagate the value returned from extcon_find_cable_index()
instead of -ENODEV. For readability, -EINVAL is returned in place of
the variable.

Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
---
 drivers/extcon/extcon-class.c |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/drivers/extcon/extcon-class.c b/drivers/extcon/extcon-class.c
index 946a318..6348b64 100644
--- a/drivers/extcon/extcon-class.c
+++ b/drivers/extcon/extcon-class.c
@@ -475,7 +475,7 @@ int extcon_register_interest(struct 
extcon_specific_cable_nb *obj,
 
obj-cable_index = extcon_find_cable_index(obj-edev, cable_name);
if (obj-cable_index  0)
-   return -ENODEV;
+   return -EINVAL;
 
obj-user_nb = nb;
 
-- 
1.7.4.1

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH Resend] extcon: Fix return value in extcon_register_interest()

2012-09-25 Thread Chanwoo Choi
On 09/25/2012 08:01 PM, Sachin Kamat wrote:

 Propagate the value returned from extcon_find_cable_index()
 instead of -ENODEV. For readability, -EINVAL is returned in place of
 the variable.
 
 Signed-off-by: Sachin Kamat sachin.ka...@linaro.org
 ---

Applied, thank you.

You can check it after some hours at below git repository for Extcon.
-
http://git.infradead.org/users/kmpark/linux-samsung/shortlog/refs/heads/extcon-for-next

Thanks,
Chanwoo Choi
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/