[lkml]   [2011]   [Oct]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH 2/5] input/cma3000_d0x: Check silicon version
Hello Jonathan

>> +     if (rev != CMA3000_REV) {
>> +             error = -EINVAL;
>> +             pr_err("CMA3000 Accelerometer: Unknown Revision %x\n", rev);
> warning perhaps rather than error?

IMHO it should be an error. If the revision is different, either you
are in front of a unknown chip or the chip is going to behave
differently and lead into an unstable system. Better be safe than

> Patch could do with a statement of why anyone would care about this.
>>       error = request_threaded_irq(irq, NULL, cma3000_thread_irq,
>>                                    pdata->irqflags | IRQF_ONESHOT,
>>                                    "cma3000_d0x", data);

You are right... not so useful information :). Maybe as part of a
sysfs file, but definately not as as a printk


Ricardo Ribalda
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to
More majordomo info at
Please read the FAQ at

 \ /
  Last update: 2011-10-18 14:09    [W:0.046 / U:3.228 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site