Re: Applied "spi: sh-msiof: Add compatible strings for r8a774[35]" to the spi tree

[Magnus Damm <magnus.damm@xxxxxxxxx>]

Hi Mark, Geert, everyone,

On Wed, Sep 27, 2017 at 2:30 AM, Mark Brown <broonie@xxxxxxxxxx> wrote:
> On Mon, Sep 25, 2017 at 09:15:53PM +0200, Geert Uytterhoeven wrote:
>
>> 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.
>
> So good practice is always good practice, but I'd argue that it's not
> bad practice to also explicitly enumerate all the documented compatibles
> in the driver.  Being liberal in what you accept and all that.
>
>>   2. Some harm is involved, in the form of increased kernel image size.
>
> Is this really a meaningful impact?
>
>>   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.
>
> That's basically the same good practice thing, it's just documenting
> what you're trying to do here with not putting things in code but
> writing things in changelogs doesn't make them so!

My opinion that good practice is to document the per-driver supported
hardware by explicitly listing the per-SoC compat strings in the DT
binding document.

Then exactly which compat strings the driver matches on is really part
of the software implementation and it will most likely vary over time.

So my view is that only updating the DT binding document should be
enough in most cases when fall-back compat strings are used. I guess
other people see it differently?

Is it too much detail to let the MAINTAINERS file point out both
driver files and the DT binding files?

Cheers,

/ magnus
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html