linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v1 0/3] Remove unnecessary version number
@ 2022-04-14  9:22 Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string Bharat Kumar Gogada
                   ` (2 more replies)
  0 siblings, 3 replies; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-14  9:22 UTC (permalink / raw)
  To: linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada

This patch series removes unnecessary version number in compatible string.
The device tree documentation reg names are rearranged to align with
node name.

Bharat Kumar Gogada (3):
  dt-bindings: PCI: xilinx-cpm: Remove version number in compatible
    string
  PCI: xilinx-cpm: Remove version number in compatible string
  dt-bindings: PCI: xilinx-cpm: Change reg property order

 .../devicetree/bindings/pci/xilinx-versal-cpm.yaml | 14 +++++++-------
 drivers/pci/controller/pcie-xilinx-cpm.c           |  2 +-
 2 files changed, 8 insertions(+), 8 deletions(-)

-- 
2.17.1


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-14  9:22 [PATCH v1 0/3] Remove unnecessary version number Bharat Kumar Gogada
@ 2022-04-14  9:22 ` Bharat Kumar Gogada
  2022-04-14 12:46   ` Michal Simek
  2022-04-14  9:22 ` [PATCH v1 2/3] " Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order Bharat Kumar Gogada
  2 siblings, 1 reply; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-14  9:22 UTC (permalink / raw)
  To: linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada

Removing unnecessary version number in compatible string.

Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com>
---
 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


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v1 2/3] PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-14  9:22 [PATCH v1 0/3] Remove unnecessary version number Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string Bharat Kumar Gogada
@ 2022-04-14  9:22 ` Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order Bharat Kumar Gogada
  2 siblings, 0 replies; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-14  9:22 UTC (permalink / raw)
  To: linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada

Remove unnecessary version number in compatible string.

Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com>
---
 drivers/pci/controller/pcie-xilinx-cpm.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pci/controller/pcie-xilinx-cpm.c b/drivers/pci/controller/pcie-xilinx-cpm.c
index c7cd44ed4dfc..52bc74335584 100644
--- a/drivers/pci/controller/pcie-xilinx-cpm.c
+++ b/drivers/pci/controller/pcie-xilinx-cpm.c
@@ -592,7 +592,7 @@ static int xilinx_cpm_pcie_probe(struct platform_device *pdev)
 }
 
 static const struct of_device_id xilinx_cpm_pcie_of_match[] = {
-	{ .compatible = "xlnx,versal-cpm-host-1.00", },
+	{ .compatible = "xlnx,versal-cpm-host", },
 	{}
 };
 
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order
  2022-04-14  9:22 [PATCH v1 0/3] Remove unnecessary version number Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string Bharat Kumar Gogada
  2022-04-14  9:22 ` [PATCH v1 2/3] " Bharat Kumar Gogada
@ 2022-04-14  9:22 ` Bharat Kumar Gogada
  2022-04-14 12:47   ` Michal Simek
  2 siblings, 1 reply; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-14  9:22 UTC (permalink / raw)
  To: linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, michals, robh, Bharat Kumar Gogada

Describe cpm reg property name before cfg reg property to align with
node name.

Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com>
---
 .../devicetree/bindings/pci/xilinx-versal-cpm.yaml     | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
index 4ebcc838a1f6..a3bc6ed9cf95 100644
--- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
+++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
@@ -18,13 +18,13 @@ properties:
 
   reg:
     items:
-      - description: Configuration space region and bridge registers.
       - description: CPM system level control and status registers.
+      - description: Configuration space region and bridge registers.
 
   reg-names:
     items:
-      - const: cfg
       - const: cpm_slcr
+      - const: cfg
 
   interrupts:
     maxItems: 1
@@ -86,9 +86,9 @@ examples:
                        ranges = <0x02000000 0x0 0xe0000000 0x0 0xe0000000 0x0 0x10000000>,
                                 <0x43000000 0x80 0x00000000 0x80 0x00000000 0x0 0x80000000>;
                        msi-map = <0x0 &its_gic 0x0 0x10000>;
-                       reg = <0x6 0x00000000 0x0 0x10000000>,
-                             <0x0 0xfca10000 0x0 0x1000>;
-                       reg-names = "cfg", "cpm_slcr";
+                       reg = <0x0 0xfca10000 0x0 0x1000>,
+                             <0x6 0x00000000 0x0 0x10000000>;
+                       reg-names = "cpm_slcr", "cfg";
                        pcie_intc_0: interrupt-controller {
                                #address-cells = <0>;
                                #interrupt-cells = <1>;
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 10+ messages in thread

* Re: [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-14  9:22 ` [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string Bharat Kumar Gogada
@ 2022-04-14 12:46   ` Michal Simek
  2022-04-14 16:45     ` Bjorn Helgaas
  0 siblings, 1 reply; 10+ messages in thread
From: Michal Simek @ 2022-04-14 12:46 UTC (permalink / raw)
  To: Bharat Kumar Gogada, linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, robh



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 <bharat.kumar.gogada@xilinx.com>
> ---
>   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>;

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order
  2022-04-14  9:22 ` [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order Bharat Kumar Gogada
@ 2022-04-14 12:47   ` Michal Simek
  0 siblings, 0 replies; 10+ messages in thread
From: Michal Simek @ 2022-04-14 12:47 UTC (permalink / raw)
  To: Bharat Kumar Gogada, linux-pci, linux-kernel, devicetree
  Cc: lorenzo.pieralisi, bhelgaas, robh



On 4/14/22 11:22, Bharat Kumar Gogada wrote:
> Describe cpm reg property name before cfg reg property to align with
> node name.
> 
> Signed-off-by: Bharat Kumar Gogada <bharat.kumar.gogada@xilinx.com>
> ---
>   .../devicetree/bindings/pci/xilinx-versal-cpm.yaml     | 10 +++++-----
>   1 file changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> index 4ebcc838a1f6..a3bc6ed9cf95 100644
> --- a/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> +++ b/Documentation/devicetree/bindings/pci/xilinx-versal-cpm.yaml
> @@ -18,13 +18,13 @@ properties:
>   
>     reg:
>       items:
> -      - description: Configuration space region and bridge registers.
>         - description: CPM system level control and status registers.
> +      - description: Configuration space region and bridge registers.
>   
>     reg-names:
>       items:
> -      - const: cfg
>         - const: cpm_slcr
> +      - const: cfg
>   
>     interrupts:
>       maxItems: 1
> @@ -86,9 +86,9 @@ examples:
>                          ranges = <0x02000000 0x0 0xe0000000 0x0 0xe0000000 0x0 0x10000000>,
>                                   <0x43000000 0x80 0x00000000 0x80 0x00000000 0x0 0x80000000>;
>                          msi-map = <0x0 &its_gic 0x0 0x10000>;
> -                       reg = <0x6 0x00000000 0x0 0x10000000>,
> -                             <0x0 0xfca10000 0x0 0x1000>;
> -                       reg-names = "cfg", "cpm_slcr";
> +                       reg = <0x0 0xfca10000 0x0 0x1000>,
> +                             <0x6 0x00000000 0x0 0x10000000>;
> +                       reg-names = "cpm_slcr", "cfg";
>                          pcie_intc_0: interrupt-controller {
>                                  #address-cells = <0>;
>                                  #interrupt-cells = <1>;


Acked-by: Michal Simek <michal.simek@xilinx.com>

Thanks,
Michal

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-14 12:46   ` Michal Simek
@ 2022-04-14 16:45     ` Bjorn Helgaas
  2022-04-19 12:22       ` Bharat Kumar Gogada
  0 siblings, 1 reply; 10+ messages in thread
From: Bjorn Helgaas @ 2022-04-14 16:45 UTC (permalink / raw)
  To: Michal Simek
  Cc: Bharat Kumar Gogada, linux-pci, linux-kernel, devicetree,
	lorenzo.pieralisi, bhelgaas, robh

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 <bharat.kumar.gogada@xilinx.com>
> > ---
> >   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>;

^ permalink raw reply	[flat|nested] 10+ messages in thread

* RE: [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-14 16:45     ` Bjorn Helgaas
@ 2022-04-19 12:22       ` Bharat Kumar Gogada
  2022-04-19 14:12         ` Rob Herring
  0 siblings, 1 reply; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-19 12:22 UTC (permalink / raw)
  To: Bjorn Helgaas, Michal Simek
  Cc: linux-pci, linux-kernel, devicetree, lorenzo.pieralisi, bhelgaas, robh

> 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@xilinx.com>
> > > ---
> > >   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>;

^ permalink raw reply	[flat|nested] 10+ messages in thread

* Re: [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-19 12:22       ` Bharat Kumar Gogada
@ 2022-04-19 14:12         ` Rob Herring
  2022-04-26  4:04           ` Bharat Kumar Gogada
  0 siblings, 1 reply; 10+ messages in thread
From: Rob Herring @ 2022-04-19 14:12 UTC (permalink / raw)
  To: Bharat Kumar Gogada
  Cc: Bjorn Helgaas, Michal Simek, linux-pci, linux-kernel, devicetree,
	lorenzo.pieralisi, bhelgaas

On Tue, Apr 19, 2022 at 7:22 AM Bharat Kumar Gogada <bharatku@xilinx.com> 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

^ permalink raw reply	[flat|nested] 10+ messages in thread

* RE: [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string
  2022-04-19 14:12         ` Rob Herring
@ 2022-04-26  4:04           ` Bharat Kumar Gogada
  0 siblings, 0 replies; 10+ messages in thread
From: Bharat Kumar Gogada @ 2022-04-26  4:04 UTC (permalink / raw)
  To: Rob Herring
  Cc: Bjorn Helgaas, Michal Simek, linux-pci, linux-kernel, devicetree,
	lorenzo.pieralisi, bhelgaas

 
> On Tue, Apr 19, 2022 at 7:22 AM Bharat Kumar Gogada
> <bharatku@xilinx.com> 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.

^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2022-04-26  4:04 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-14  9:22 [PATCH v1 0/3] Remove unnecessary version number Bharat Kumar Gogada
2022-04-14  9:22 ` [PATCH v1 1/3] dt-bindings: PCI: xilinx-cpm: Remove version number in compatible string Bharat Kumar Gogada
2022-04-14 12:46   ` Michal Simek
2022-04-14 16:45     ` Bjorn Helgaas
2022-04-19 12:22       ` Bharat Kumar Gogada
2022-04-19 14:12         ` Rob Herring
2022-04-26  4:04           ` Bharat Kumar Gogada
2022-04-14  9:22 ` [PATCH v1 2/3] " Bharat Kumar Gogada
2022-04-14  9:22 ` [PATCH v1 3/3] dt-bindings: PCI: xilinx-cpm: Change reg property order Bharat Kumar Gogada
2022-04-14 12:47   ` Michal Simek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).