On Tue, Sep 17, 2019 at 07:48:17AM -0500, Rob Herring wrote: > On Tue, Sep 17, 2019 at 6:29 AM Simon Horman <horms@xxxxxxxxxxxx> wrote: > > On Mon, Sep 16, 2019 at 05:33:56PM +0200, Simon Horman wrote: ... > > > +properties: > > > + compatible: > > > + items: > > > + - const: simple-pm-bus > > > > The following patch, for Renesas BSC, also > > describes the simple-pm-bus but also requires other compatible > > strings to be present. > > > > In order to facilitate that I tried the following: > > > > compatible: > > contains: > > const: simple-pm-bus > > additionalItems: true > > > > This allows make dtbs_check DT_SCHEMA_FILES=.../simple-pm-bus.yaml to pass > > in cases where there are more compat strings pesent. However > > make dtbs_check DT_SCHEMA_FILES=...renesas,bsc.yaml (schema file > > introduced in following patch) fails in the case > > where the only compat string is simple-pm-bus. > > > > make dtbs_check DT_SCHEMA_FILES=...renesas,bsc.yaml also fails > > due to the reg property not being present for the same compat case, > > which should be valid according to simple-pm-bus.yaml. > > > > Does anyone have any guidance on how to proceed? > > You'll need a custom 'select' in renesas,bsc.yaml that leaves out > "simple-pm-bus". See "panel-lvds" users in linux-next for an example. Thanks, I think I have that working now.
