Re: [PATCH] Input: silead - list all supported compatible strings in binding document

From: Javier Martinez Canillas
Date: Mon Apr 03 2017 - 11:52:08 EST


Hello Rob,

On 04/03/2017 11:25 AM, Rob Herring wrote:
> On Wed, Mar 29, 2017 at 02:25:31PM -0400, Javier Martinez Canillas wrote:
>> The driver contains compatible strings for different models, but the DT
>> binding doc only lists one of them. Add the remaining to the document.
>>
>> Signed-off-by: Javier Martinez Canillas <javier@xxxxxxxxxxxxxxx>
>> ---
>>
>> .../devicetree/bindings/input/touchscreen/silead_gsl1680.txt | 7 ++++++-
>> 1 file changed, 6 insertions(+), 1 deletion(-)
>
> "dt-bindings: input: ..." is preferred for the subject, but no need to
> respin just for that.
>

Can we document it in Documentation/devicetree/bindings/submitting-patches.txt?

I'm asking because is true that at the beginning we used "dt-bindings: foo" for
all DT bindings patches but then many (most?) maintainers started asking for the
subsystem subject line to be used for both drivers and DT bindings docs since
they would be merging both and also they could miss the DT bindings patches if
their subsystem prefix was not used.

> Acked-by: Rob Herring <robh@xxxxxxxxxx>
>

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America