Re: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge

Re: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge

[MyungJoo Ham]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=euc-kr, Size: 1622 bytes --]

Sender : Pallala, Ramakrishna<ramakrishna.pallala@intel.com> Date : 2011-09-07 17:30 (GMT+09:00)
> Hi MyungJoo,
> 
> > -----Original Message-----
> > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > Sent: Monday, September 05, 2011 6:55 AM
> > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > Subject: Re: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge
> > 
> > Sender : Ramakrishna Pallala<ramakrishna.pallala@intel.com> Date : 2011-09-02 15:45
> > (GMT+09:00)
> > >
> > > In max17042_get_property(...), the values returned by
> > > max17042_read_reg are directly assigned to the variables,
> > > even if the read results in an error.
> > >
> > > This patch checks for the return code from max17042_read_reg and
> > > exits the function if there is any error.
> > >
> > > Signed-off-by: Ramakrishna Pallala
> > > ---
> > > drivers/power/max17042_battery.c |   85 ++++++++++++++++++++++++++------------
> > > 1 files changed, 58 insertions(+), 27 deletions(-)
> > >
> > 
> > Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
> 
> Thanks for the Ack.
> When can I expect the patch to be merged ?
> 
> Thanks,
> Ram
> 

Hello Ram,

Although I'd be so glad to be able to merge patches of MAX17042 (and MAX8997/8998/8952/8903/...?) by myself; however, I'm not a maintainer of MAX17042.

I guess you need to wait for Anton.

Cheers,
MyungJoo

ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge

[Pallala, Ramakrishna]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="ks_c_5601-1987", Size: 2133 bytes --]

Hi Anton,

> -----Original Message-----
> From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> Sent: Wednesday, September 07, 2011 2:17 PM
> To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> Subject: Re: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge
> 
> Sender : Pallala, Ramakrishna<ramakrishna.pallala@intel.com> Date : 2011-09-07 17:30
> (GMT+09:00)
> > Hi MyungJoo,
> >
> > > -----Original Message-----
> > > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > > Sent: Monday, September 05, 2011 6:55 AM
> > > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > > Subject: Re: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge
> > >
> > > Sender : Ramakrishna Pallala<ramakrishna.pallala@intel.com> Date : 2011-09-02 15:45
> > > (GMT+09:00)
> > > >
> > > > In max17042_get_property(...), the values returned by
> > > > max17042_read_reg are directly assigned to the variables,
> > > > even if the read results in an error.
> > > >
> > > > This patch checks for the return code from max17042_read_reg and
> > > > exits the function if there is any error.
> > > >
> > > > Signed-off-by: Ramakrishna Pallala
> > > > ---
> > > > drivers/power/max17042_battery.c |   85 ++++++++++++++++++++++++++------------
> > > > 1 files changed, 58 insertions(+), 27 deletions(-)
> > > >
> > >
> > > Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
> >
> > Thanks for the Ack.
> > When can I expect the patch to be merged ?
> >
> > Thanks,
> > Ram
> >
> 
> Hello Ram,
> 
> Although I'd be so glad to be able to merge patches of MAX17042 (and
> MAX8997/8998/8952/8903/...?) by myself; however, I'm not a maintainer of MAX17042.
> 
> I guess you need to wait for Anton.
> 
> Cheers,
> MyungJoo

When can I expect the patch to be merged ?

Thanks,
Ram
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge

[Pallala, Ramakrishna]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="ks_c_5601-1987", Size: 2663 bytes --]

Hi Anton,

> -----Original Message-----
> From: Pallala, Ramakrishna
> Sent: Thursday, September 08, 2011 11:07 PM
> To: myungjoo.ham@samsung.com; linux-kernel@vger.kernel.org; Anton Vorontsov
> Cc: dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> Subject: RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge
> 
> Hi Anton,
> 
> > -----Original Message-----
> > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > Sent: Wednesday, September 07, 2011 2:17 PM
> > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > Subject: Re: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel
> gauge
> >
> > Sender : Pallala, Ramakrishna<ramakrishna.pallala@intel.com> Date : 2011-09-07 17:30
> > (GMT+09:00)
> > > Hi MyungJoo,
> > >
> > > > -----Original Message-----
> > > > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > > > Sent: Monday, September 05, 2011 6:55 AM
> > > > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > > > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > > > Subject: Re: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel
> gauge
> > > >
> > > > Sender : Ramakrishna Pallala<ramakrishna.pallala@intel.com> Date : 2011-09-02
> 15:45
> > > > (GMT+09:00)
> > > > >
> > > > > In max17042_get_property(...), the values returned by
> > > > > max17042_read_reg are directly assigned to the variables,
> > > > > even if the read results in an error.
> > > > >
> > > > > This patch checks for the return code from max17042_read_reg and
> > > > > exits the function if there is any error.
> > > > >
> > > > > Signed-off-by: Ramakrishna Pallala
> > > > > ---
> > > > > drivers/power/max17042_battery.c |   85 ++++++++++++++++++++++++++------------
> > > > > 1 files changed, 58 insertions(+), 27 deletions(-)
> > > > >
> > > >
> > > > Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
> > >
> > > Thanks for the Ack.
> > > When can I expect the patch to be merged ?
> > >
> > > Thanks,
> > > Ram
> > >
> >
> > Hello Ram,
> >
> > Although I'd be so glad to be able to merge patches of MAX17042 (and
> > MAX8997/8998/8952/8903/...?) by myself; however, I'm not a maintainer of MAX17042.
> >
> > I guess you need to wait for Anton.
> >
> > Cheers,
> > MyungJoo
> 
> When can I expect the patch to be merged ?
> 
> Thanks,
> Ram

Please let me know your feedback on this patch.

Thanks,
Ram
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥

RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge

[Pallala, Ramakrishna]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="ks_c_5601-1987", Size: 3337 bytes --]

Hi Anton/Sang/David,

> -----Original Message-----
> From: Pallala, Ramakrishna
> Sent: Monday, September 12, 2011 6:18 PM
> To: Pallala, Ramakrishna; myungjoo.ham@samsung.com; linux-kernel@vger.kernel.org; Anton
> Vorontsov; cbou@mail.ru
> Cc: dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> Subject: RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel gauge
> 
> Hi Anton,
> 
> > -----Original Message-----
> > From: Pallala, Ramakrishna
> > Sent: Thursday, September 08, 2011 11:07 PM
> > To: myungjoo.ham@samsung.com; linux-kernel@vger.kernel.org; Anton Vorontsov
> > Cc: dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > Subject: RE: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel
> gauge
> >
> > Hi Anton,
> >
> > > -----Original Message-----
> > > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > > Sent: Wednesday, September 07, 2011 2:17 PM
> > > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > > Subject: Re: RE: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel
> > gauge
> > >
> > > Sender : Pallala, Ramakrishna<ramakrishna.pallala@intel.com> Date : 2011-09-07 17:30
> > > (GMT+09:00)
> > > > Hi MyungJoo,
> > > >
> > > > > -----Original Message-----
> > > > > From: MyungJoo Ham [mailto:myungjoo.ham@samsung.com]
> > > > > Sent: Monday, September 05, 2011 6:55 AM
> > > > > To: Pallala, Ramakrishna; linux-kernel@vger.kernel.org
> > > > > Cc: Anton Vorontsov; dwmw2@infradead.org; ¹Ú°æ¹Î; Wolfram Sang; R, Durgadoss
> > > > > Subject: Re: [PATCHv1 1/1] [Power Supply]: Fix error handling in max17042 fuel
> > gauge
> > > > >
> > > > > Sender : Ramakrishna Pallala<ramakrishna.pallala@intel.com> Date : 2011-09-02
> > 15:45
> > > > > (GMT+09:00)
> > > > > >
> > > > > > In max17042_get_property(...), the values returned by
> > > > > > max17042_read_reg are directly assigned to the variables,
> > > > > > even if the read results in an error.
> > > > > >
> > > > > > This patch checks for the return code from max17042_read_reg and
> > > > > > exits the function if there is any error.
> > > > > >
> > > > > > Signed-off-by: Ramakrishna Pallala
> > > > > > ---
> > > > > > drivers/power/max17042_battery.c |   85 ++++++++++++++++++++++++++------------
> > > > > > 1 files changed, 58 insertions(+), 27 deletions(-)
> > > > > >
> > > > >
> > > > > Acked-by: MyungJoo Ham <myungjoo.ham@samsung.com>
> > > >
> > > > Thanks for the Ack.
> > > > When can I expect the patch to be merged ?
> > > >
> > > > Thanks,
> > > > Ram
> > > >
> > >
> > > Hello Ram,
> > >
> > > Although I'd be so glad to be able to merge patches of MAX17042 (and
> > > MAX8997/8998/8952/8903/...?) by myself; however, I'm not a maintainer of MAX17042.
> > >
> > > I guess you need to wait for Anton.
> > >
> > > Cheers,
> > > MyungJoo
> >
> > When can I expect the patch to be merged ?
> >
> > Thanks,
> > Ram
> 
> Please let me know your feedback on this patch.
> 
> Thanks,
> Ram

I have been waiting for your comments for two weeks.
Please let me know when can I get the feedback on this patch.

Thanks,
Ram
ÿôèº{.nÇ+‰·Ÿ®‰­†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a­¢f£¢·hšïêÿ‘êçz_è®\x03(­éšŽŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨è­Ú&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥