Thread (79 messages) 79 messages, 15 authors, 2018-12-03

Re: [PATCH 13/36] dt-bindings: arm: Convert PMU binding to json-schema

From: Will Deacon <hidden>
Date: 2018-10-19 10:34:51
Also in: linux-arm-kernel, linux-devicetree, lkml

Hi Rob,

On Wed, Oct 10, 2018 at 01:51:24PM -0500, Rob Herring wrote:
On Wed, Oct 10, 2018 at 11:50 AM Will Deacon [off-list ref] wrote:
quoted
On Tue, Oct 09, 2018 at 01:14:02PM -0500, Rob Herring wrote:
quoted
I guess the single interrupt case is less obvious now with no
description (it's the first list item of 'oneOf'). The schema If the
single interrupt is not supported, then we can drop it here.
Well the description says "1 interrupt per core" which is incorrect.
You are reading the schema wrong. There are 2 cases supported as
defined by each '-'. The 2nd case is all the keywords until the
indentation decreases. So 'description' is just description of the 2nd
case. The first case is just "maxItems: 1". I probably didn't put a
description because why write in free form text what the schema says
(other than of course no one knows json-schema...).
Apologies, I've not read one of these things before and looks like I
completely misread it.
YAML combines the best of Makefiles and python. You can't have tabs
and Indentation is significant. :)
Oh wow, I'm in way over my head here!
quoted
I also
don't understand why maxItems is 8.
Humm, I probably just made that up based on GICv2 limitations. What
should it be? If there's not any inherit maximum, can we put something
reasonable? There's not really any way to express that it should match
the number of cores in the system.
What's the largest number you can think of?

Will
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help