On 08-08-18, 08:48, Rob Herring wrote: > On Wed, Aug 8, 2018 at 8:03 AM Todor Tomov <todor.tomov@xxxxxxxxxx> wrote: > > >>>> So this should either be: > > >>>> > > >>>> - "qcom,cci-v1.0.8" > > > > > > This is supposed to be 1.1.0 for 8916... > > > > There is conflicting information about this in the documentation. > > However the version read from the hw version register is 1.0.8 and > > this matches the value listed in the SWI too so I believe 1.0.8 is > > correct for 8916. > > And this is why we don't use version numbers. Unless you have access > to the RTL repositories with version tags then it is just error prone. Yeah we have a HW_VERSION register too, but someone might forget to populate it, so lets go ahead and use the soc numbers as discussed. Thanks -- ~Vinod -- To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html