Removing unnecessary version number in compatible string.
Signed-off-by: Bharat Kumar Gogada <[email protected]>
---
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
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>;
--
2.17.1
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.
>
> Signed-off-by: Bharat Kumar Gogada <[email protected]>
> ---
> 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>;
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.
> > Signed-off-by: Bharat Kumar Gogada <[email protected]>
> > ---
> > 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>;
> 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 <[email protected]>
> > > ---
> > > 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>;
On Tue, Apr 19, 2022 at 7:22 AM Bharat Kumar Gogada <[email protected]> wrote:
>
> > 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.
I did not say to remove the existing version breaking compatibility.
Just don't continue to add new version numbers.
Rob
> On Tue, Apr 19, 2022 at 7:22 AM Bharat Kumar Gogada
> <[email protected]> wrote:
> >
> > > 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.
>
> I did not say to remove the existing version breaking compatibility.
> Just don't continue to add new version numbers.
>
Thanks Rob.