Re: [PATCH] fix some coding style problem in drivers/staging/iio/frequency/ad9832.c
From: Jonathan Cameron
Date: Mon Jun 02 2014 - 13:43:46 EST
On 02/06/14 18:05, Federico Di Pierro wrote:
Fair enough, but you need to spend a little time working
This is my latest task of the eudyptula challenge.
Just a very small patch! It fixes 3 little coding style problems.
I'm using linux-next.
on the 'how' of your patch submission.
See Documentation/SubmittingPatches and repost the patch according
to those guidelines.
Also at a quick glance, unless I am very bored I'm unlikely to take
such a minor white space and bracket adjustment patch. If it were
part of a bigger series doing something more interesting I'd be
If you want to approach this driver, then feel free but there are
many more major issues within it to be looked at such as ABI
compliance. See Documentation/ABI/testing/sysfs-bus-iio and friends
and compare the driver with them. Off the top of my head I can't
remember how bad this driver is but I doubt everything is well
Don't feel disheartened, as I would be happy to see more substantial
cleanup patches from you as part of the challenge or otherwise.
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/