On Sun, Jul 22, 2018 at 9:54 PM Vinod <vkoul@xxxxxxxxxx> wrote: > > On 20-07-18, 12:03, Rob Herring wrote: > > On Tue, Jul 17, 2018 at 06:27:28PM +0530, Vinod Koul wrote: > > > From: Todor Tomov <todor.tomov@xxxxxxxxxx> > > > > +Required properties: > > > + - compatible: Should be one of: > > > + - "qcom,cci-v1.0.8" for 8916; > > > + - "qcom,cci-v1.4.0" for 8996. > > > > It's not very convincing to use version numbers instead of normal SoC > > part# convention when there is a 1-1 mapping. > > I was under the impression that we don't have 1-1 mapping. An IP version > is found in multiple SoCs Maybe so, but I'm only looking at what is getting upstream. We already discussed this at length on the thermal binding[1]. As you've written it, we'll still have to update the doc to add other chips for an already documented version. Rob [1] https://lkml.org/lkml/2018/7/5/830
