lkml.org 
[lkml]   [2014]   [Mar]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v2] power: twl4030-madc-battery: Convert to iio consumer.
From
Hi Sebastian,

On Thu, Mar 6, 2014 at 12:25 AM, Sebastian Reichel <sre@debian.org> wrote:
> Hi Marek,
>
> I just have one comment:
>
> On Wed, Mar 05, 2014 at 09:52:17PM +0100, Marek Belisko wrote:
>> [...]
>> -static int madc_read(int index)
>> +static int madc_read(struct iio_channel *channel)
>> {
>> - struct twl4030_madc_request req;
>> - int val;
>> -
>> - req.channels = index;
>> - req.method = TWL4030_MADC_SW2;
>> - req.type = TWL4030_MADC_WAIT;
>> - req.do_avg = 0;
>> - req.raw = false;
>> - req.func_cb = NULL;
>> -
>> - val = twl4030_madc_conversion(&req);
>> - if (val < 0)
>> - return val;
>> -
>> - return req.rbuf[ffs(index) - 1];
>> + int val, err;
>> + err = iio_read_channel_processed(channel, &val);
>> + if (err < 0) {
>> + pr_info("Error:%d\n", err);
>
> This should be "Error: %d\n" (with a space).
Ups this is remain from debugging messages. I'll remove it and re-post patches.
Thanks.
>
> Apart from that you should use dev_info() or dev_err(), so that the
> message is properly prefixed by the device. Currently there is no
> hint where this error message is generated.
>
>> + return err;
>> + }
>> + return val;
>> }
>> [...]
>
> After fixing this you can add
>
> Reviewed-By: Sebastian Reichel <sre@debian.org>
>
> -- Sebastian

BR,

marek

--
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com


\
 
 \ /
  Last update: 2014-03-06 23:01    [W:1.908 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site