Re: [PATCH v4 3/3] regulator: qcom-rpm: Regulator driver for the Qualcomm RPM

From: Bjorn Andersson
Date: Wed Jul 16 2014 - 18:22:30 EST


On Wed, Jul 16, 2014 at 12:01 AM, pramod gurav
<pramod.gurav.etc@xxxxxxxxx> wrote:
> Hi Bjorn,
>
> On Wed, Jul 16, 2014 at 4:30 AM, Bjorn Andersson
> <bjorn.andersson@xxxxxxxxxxxxxx> wrote:
[...]
>> +static const struct regulator_linear_range pldo_ranges[] = {
>> + REGULATOR_LINEAR_RANGE( 750000, 0, 59, 12500),
>
> ERROR: space prohibited after that open parenthesis '('
>

I prefer the nicely aligned columns over checkpatch correctness here.

>> + REGULATOR_LINEAR_RANGE(1500000, 60, 123, 25000),
>> + REGULATOR_LINEAR_RANGE(3100000, 124, 160, 50000),
>> +};
>> +

Regards,
Bjorn
--
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/