Hi Linus,
On Tue, Dec 1, 2015 at 10:54 AM, Linus Walleij <linus.wall...@linaro.org> wrote:
> On Tue, Nov 24, 2015 at 4:05 PM, Geert Uytterhoeven
> <geert+rene...@glider.be> wrote:
>
>> If the Renesas R-Car GPIO driver cannot find a functional clock, it
>> prints a warning, .e.g.
>>
>> gpio_rcar ffc40000.gpio: unable to get clock
>>
>> and continues, as the clock is optional, depending on the SoC type.
>> This warning may confuse users.
>>
>> To fix this, add a flag to indicate that the clock is mandatory or
>> optional:
>> - If the clock is mandatory (on R-Car Gen2), a missing clock is now
>> treated as a fatal error,
>> - If the clock is optional (on R-Car Gen1), the warning is no longer
>> printed.
>>
>> The new flag is merged with the existing has_both_edge_trigger boolean
>> into a bitfield to save space.
>
> *DON'T* try to save bits in a megabyte kernel like this.
> The better readability is worth the extra bits or bytes it takes.
> Besides, did you really try to compile it before/after that change
> to verify that it actually saved that space?
Is it better readable?
Size also depends on how many more flags we'll add later.
>> Suggested-by: Magnus Damm <magnus.d...@gmail.com>
>> Signed-off-by: Geert Uytterhoeven <geert+rene...@glider.be>
>> ---
>> This depends on "gpio: rcar: Remove obsolete platform data support".
>
> Where is that patch? This does not apply to my tree,
> probably because of this.
IIRC, you said "include it in your pull request" for that series.
> Please sent the patches in a series.
I can send both to you, as they're not pinctrl but gpio.
>> unsigned has_both_edge_trigger:1;
>> + unsigned needs_clk:1;
>
> both should be bool and assigned true/false.
OK, if you prefer it that way.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
