> On Thu, Apr 14, 2022 at 02:46:25PM +0200, Michal Simek wrote:
> > On 4/14/22 11:22, Bharat Kumar Gogada wrote:
> > > Removing unnecessary version number in compatible string.
> >
> > I am missing reason for this in commit message.
>
> Agreed. The commit log for the pcie-xilinx-cpm.c change also needs to
> explain why removing the version is useful and safe.
HI Bjorn, Michal,
The CPM block is hard block, Rob pointed out that versioning has no value here.
Will resend patch with this detail.
Regards,
Bharat
>
> > > Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xxxxxxxxxx>
> > > ---
> > > Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git
> > > a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> > > b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> > > index 32f4641085bc..4ebcc838a1f6 100644
> > > --- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> > > +++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> > > @@ -14,7 +14,7 @@ allOf:
> > > properties:
> > > compatible:
> > > - const: xlnx,versal-cpm-host-1.00
> > > + const: xlnx,versal-cpm-host
> >
> > And this is likely breaking compatibility for existing DTs.
> >
> > M
> >
> > > reg:
> > > items:
> > > @@ -70,7 +70,7 @@ examples:
> > > #address-cells = <2>;
> > > #size-cells = <2>;
> > > cpm_pcie: pcie@fca10000 {
> > > - compatible = "xlnx,versal-cpm-host-1.00";
> > > + compatible = "xlnx,versal-cpm-host";
> > > device_type = "pci";
> > > #address-cells = <3>;
> > > #interrupt-cells = <1>;
