diff mbox series

clk: gcc-qcs404: Add PCIe resets

Message ID 20190502002138.10646-1-bjorn.andersson@linaro.org
State Superseded
Headers show
Series clk: gcc-qcs404: Add PCIe resets | expand

Commit Message

Bjorn Andersson May 2, 2019, 12:21 a.m. UTC
Enabling PCIe requires several of the PCIe related resets from GCC, so
add them all.

Reviewed-by: Niklas Cassel <niklas.cassel@linaro.org>

Acked-by: Stephen Boyd <sboyd@kernel.org>

Acked-by: Rob Herring <robh@kernel.org>

Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>

---

Changes since v2:
- Rebased patch

 drivers/clk/qcom/gcc-qcs404.c               | 7 +++++++
 include/dt-bindings/clock/qcom,gcc-qcs404.h | 7 +++++++
 2 files changed, 14 insertions(+)

-- 
2.18.0

Comments

Marc Gonzalez May 2, 2019, 10:53 a.m. UTC | #1
On 02/05/2019 02:21, Bjorn Andersson wrote:

> diff --git a/include/dt-bindings/clock/qcom,gcc-qcs404.h b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> index 454b3f43f538..5959399fed2e 100644

> --- a/include/dt-bindings/clock/qcom,gcc-qcs404.h

> +++ b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> @@ -166,5 +166,12 @@

>  #define GCC_PCIEPHY_0_PHY_BCR				12

>  #define GCC_EMAC_BCR					13

>  #define GCC_CDSP_RESTART				14

> +#define GCC_PCIE_0_AXI_MASTER_STICKY_ARES		14


Seems weird that there would be two names for the same entry at index 14?
Niklas Cassel May 2, 2019, 11:20 a.m. UTC | #2
On Thu, May 02, 2019 at 12:53:33PM +0200, Marc Gonzalez wrote:
> On 02/05/2019 02:21, Bjorn Andersson wrote:

> 

> > diff --git a/include/dt-bindings/clock/qcom,gcc-qcs404.h b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > index 454b3f43f538..5959399fed2e 100644

> > --- a/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > +++ b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > @@ -166,5 +166,12 @@

> >  #define GCC_PCIEPHY_0_PHY_BCR				12

> >  #define GCC_EMAC_BCR					13

> >  #define GCC_CDSP_RESTART				14

> > +#define GCC_PCIE_0_AXI_MASTER_STICKY_ARES		14

> 

> Seems weird that there would be two names for the same entry at index 14?


Changes since v2:
- Rebased patch

The proper tag in the subject should have been [PATCH v2].

This is most likely an issue caused by the rebase.


Kind regards,
Niklas
Stephen Boyd May 2, 2019, 8:43 p.m. UTC | #3
Quoting Niklas Cassel (2019-05-02 04:20:24)
> On Thu, May 02, 2019 at 12:53:33PM +0200, Marc Gonzalez wrote:

> > On 02/05/2019 02:21, Bjorn Andersson wrote:

> > 

> > > diff --git a/include/dt-bindings/clock/qcom,gcc-qcs404.h b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > index 454b3f43f538..5959399fed2e 100644

> > > --- a/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > +++ b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > @@ -166,5 +166,12 @@

> > >  #define GCC_PCIEPHY_0_PHY_BCR                              12

> > >  #define GCC_EMAC_BCR                                       13

> > >  #define GCC_CDSP_RESTART                           14

> > > +#define GCC_PCIE_0_AXI_MASTER_STICKY_ARES          14

> > 

> > Seems weird that there would be two names for the same entry at index 14?

> 

> Changes since v2:

> - Rebased patch

> 

> The proper tag in the subject should have been [PATCH v2].

> 

> This is most likely an issue caused by the rebase.

> 


Please resend then.
Bjorn Andersson May 2, 2019, 9:53 p.m. UTC | #4
On Thu 02 May 13:43 PDT 2019, Stephen Boyd wrote:

> Quoting Niklas Cassel (2019-05-02 04:20:24)

> > On Thu, May 02, 2019 at 12:53:33PM +0200, Marc Gonzalez wrote:

> > > On 02/05/2019 02:21, Bjorn Andersson wrote:

> > > 

> > > > diff --git a/include/dt-bindings/clock/qcom,gcc-qcs404.h b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > > index 454b3f43f538..5959399fed2e 100644

> > > > --- a/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > > +++ b/include/dt-bindings/clock/qcom,gcc-qcs404.h

> > > > @@ -166,5 +166,12 @@

> > > >  #define GCC_PCIEPHY_0_PHY_BCR                              12

> > > >  #define GCC_EMAC_BCR                                       13

> > > >  #define GCC_CDSP_RESTART                           14

> > > > +#define GCC_PCIE_0_AXI_MASTER_STICKY_ARES          14

> > > 

> > > Seems weird that there would be two names for the same entry at index 14?

> > 

> > Changes since v2:

> > - Rebased patch

> > 

> > The proper tag in the subject should have been [PATCH v2].

> > 

> > This is most likely an issue caused by the rebase.

> > 

> 

> Please resend then.

> 


Yeah, I screwed up the rebase. v4 is coming.

Thanks Marc,
Bjorn
diff mbox series

Patch

diff --git a/drivers/clk/qcom/gcc-qcs404.c b/drivers/clk/qcom/gcc-qcs404.c
index a54807eb3b28..29cf464dd2c8 100644
--- a/drivers/clk/qcom/gcc-qcs404.c
+++ b/drivers/clk/qcom/gcc-qcs404.c
@@ -2766,6 +2766,13 @@  static const struct qcom_reset_map gcc_qcs404_resets[] = {
 	[GCC_PCIE_0_PHY_BCR] = { 0x3e004 },
 	[GCC_PCIE_0_LINK_DOWN_BCR] = { 0x3e038 },
 	[GCC_PCIEPHY_0_PHY_BCR] = { 0x3e03c },
+	[GCC_PCIE_0_AXI_MASTER_STICKY_ARES] = { 0x3e040, 6},
+	[GCC_PCIE_0_AHB_ARES] = { 0x3e040, 5 },
+	[GCC_PCIE_0_AXI_SLAVE_ARES] = { 0x3e040, 4 },
+	[GCC_PCIE_0_AXI_MASTER_ARES] = { 0x3e040, 3 },
+	[GCC_PCIE_0_CORE_STICKY_ARES] = { 0x3e040, 2 },
+	[GCC_PCIE_0_SLEEP_ARES] = { 0x3e040, 1 },
+	[GCC_PCIE_0_PIPE_ARES] = { 0x3e040, 0 },
 	[GCC_EMAC_BCR] = { 0x4e000 },
 };
 
diff --git a/include/dt-bindings/clock/qcom,gcc-qcs404.h b/include/dt-bindings/clock/qcom,gcc-qcs404.h
index 454b3f43f538..5959399fed2e 100644
--- a/include/dt-bindings/clock/qcom,gcc-qcs404.h
+++ b/include/dt-bindings/clock/qcom,gcc-qcs404.h
@@ -166,5 +166,12 @@ 
 #define GCC_PCIEPHY_0_PHY_BCR				12
 #define GCC_EMAC_BCR					13
 #define GCC_CDSP_RESTART				14
+#define GCC_PCIE_0_AXI_MASTER_STICKY_ARES		14
+#define GCC_PCIE_0_AHB_ARES				15
+#define GCC_PCIE_0_AXI_SLAVE_ARES			16
+#define GCC_PCIE_0_AXI_MASTER_ARES			17
+#define GCC_PCIE_0_CORE_STICKY_ARES			18
+#define GCC_PCIE_0_SLEEP_ARES				19
+#define GCC_PCIE_0_PIPE_ARES				20
 
 #endif