Re: [PATCH 1/1] dt-bindings: eliminate yamllint warnings

2020-12-05 Thread Leizhen (ThunderTown)



On 2020/12/5 1:41, Mark Brown wrote:
> On Fri, Dec 04, 2020 at 10:42:26AM +0800, Zhen Lei wrote:
>> All warnings are related only to "wrong indentation", except one:
>> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml:4:1: \
>> [error] missing document start "---" (document-start)
> 
> It would make life easier (and be more normal practice) to split this up
> by driver/subsystem and send a bunch of separate patches to the relevant
> maintainers, this makes it much easier to review and handle things.

Okay, I'll split this patch and send them separately. I'm not going to mark
the new patches as v2 to save trouble.

> 



Re: [PATCH 1/1] dt-bindings: eliminate yamllint warnings

2020-12-05 Thread Sam Ravnborg
On Fri, Dec 04, 2020 at 10:42:26AM +0800, Zhen Lei wrote:
> All warnings are related only to "wrong indentation", except one:
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml:4:1: \
> [error] missing document start "---" (document-start)
> 
> Signed-off-by: Zhen Lei 
> Cc: Rob Herring 
> Cc: Michael Turquette 
> Cc: Stephen Boyd 
> Cc: Shawn Guo 
> Cc: Sascha Hauer 
> Cc: Pengutronix Kernel Team 
> Cc: Fabio Estevam 
> Cc: NXP Linux Team 
> Cc: David Airlie 
> Cc: Daniel Vetter 
> Cc: Sumit Semwal 
> Cc: Thierry Reding 
> Cc: Sam Ravnborg 
> Cc: Hans Verkuil 
> Cc: Mauro Carvalho Chehab 
> Cc: Sakari Ailus 
> Cc: Ricardo Ribalda 
> Cc: Ulf Hansson 
> Cc: Matthias Brugger 
> Cc: Liam Girdwood 
> Cc: Mark Brown 

For the bindings/display/* parts:
Acked-by: Sam Ravnborg 


Re: [PATCH 1/1] dt-bindings: eliminate yamllint warnings

2020-12-04 Thread Mark Brown
On Fri, Dec 04, 2020 at 10:42:26AM +0800, Zhen Lei wrote:
> All warnings are related only to "wrong indentation", except one:
> Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml:4:1: \
> [error] missing document start "---" (document-start)

It would make life easier (and be more normal practice) to split this up
by driver/subsystem and send a bunch of separate patches to the relevant
maintainers, this makes it much easier to review and handle things.


signature.asc
Description: PGP signature