Hi Mark,
On Mon, Sep 25, 2017 at 8:45 PM, Mark Brown <broonie@xxxxxxxxxx> wrote:
> On Mon, Sep 25, 2017 at 07:48:44PM +0200, Geert Uytterhoeven wrote:
>> Please drop this patch, as there's no need to add explicit matching for these
>> compatible values. The family-specific compatible values (which the driver
>> already matches against) are sufficient.
>
> While the patch is not needed if people list the fallback property it
> also does no harm and provides a marginal documentation benefit in
> saying that someone has considered if any special handling is useful and
> decided that it isn't.
All true.
My rebuttal is threefold:
1. Listing the fallback property is mandatory for new SoCs. We only keep
the per-SoC compatible values in the driver for older SoCs that predate
the introduction of fallback properties.
2. Some harm is involved, in the form of increased kernel image size.
3. When updating DT bindings for new SoCs, we usually add "No driver
update is needed" to the patch description to clarify. Unfortunately
that was missed here.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
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
