All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-14  9:54 ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

The goal of this series is to enable the support of MiPHY28lp Generic PHY.

The first patch is to update miphy28lp phy driver to access sysconfig register
offsets via syscfg dt property. It's based on Arnds review comments here
https://lkml.org/lkml/2014/11/13/161
I have updated the miphy28lp phy driver same way as Peter's implementation.

Gabriel Fernandez (3):
  phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
    property.
  ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
  ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
    USB3) PHY

 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
 arch/arm/configs/multi_v7_defconfig                |  1 +
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 5 files changed, 113 insertions(+), 56 deletions(-)

-- 
1.9.1


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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-14  9:54 ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ, Lee Jones, Gabriel Fernandez

The goal of this series is to enable the support of MiPHY28lp Generic PHY.

The first patch is to update miphy28lp phy driver to access sysconfig register
offsets via syscfg dt property. It's based on Arnds review comments here
https://lkml.org/lkml/2014/11/13/161
I have updated the miphy28lp phy driver same way as Peter's implementation.

Gabriel Fernandez (3):
  phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
    property.
  ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
  ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
    USB3) PHY

 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
 arch/arm/configs/multi_v7_defconfig                |  1 +
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 5 files changed, 113 insertions(+), 56 deletions(-)

-- 
1.9.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-14  9:54 ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: linux-arm-kernel

The goal of this series is to enable the support of MiPHY28lp Generic PHY.

The first patch is to update miphy28lp phy driver to access sysconfig register
offsets via syscfg dt property. It's based on Arnds review comments here
https://lkml.org/lkml/2014/11/13/161
I have updated the miphy28lp phy driver same way as Peter's implementation.

Gabriel Fernandez (3):
  phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
    property.
  ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
  ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
    USB3) PHY

 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
 arch/arm/configs/multi_v7_defconfig                |  1 +
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 5 files changed, 113 insertions(+), 56 deletions(-)

-- 
1.9.1

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

* [PATCH 1/3] phy: miphy28lp: Pass sysconfig register offsets via syscfg dt property.
  2015-01-14  9:54 ` Gabriel FERNANDEZ
  (?)
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  -1 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

Based on Arnds review comments here https://lkml.org/lkml/2014/11/13/161,
update the miphy28lp phy driver to access sysconfig register offsets via
syscfg dt property.

This is because the reg property should not be mixing address spaces like
it does currently for miphy28lp. This change then also aligns us to how other
platforms such as keystone and bcm7445 pass there syscon offsets via DT.

I have updated the miphy28lp phy driver same way as Peter's implementation.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 2 files changed, 48 insertions(+), 56 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
index 46a135d..89caa88 100644
--- a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
+++ b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
@@ -26,6 +26,7 @@ Required properties (port (child) node):
 		  filled in "reg". It can also contain the offset of the system configuration
 		  registers used as glue-logic to setup the device for SATA/PCIe or USB3
 		  devices.
+- st,syscfg	: Offset of the parent configuration register.
 - resets	: phandle to the parent reset controller.
 - reset-names	: Associated name must be "miphy-sw-rst".
 
@@ -54,18 +55,12 @@ example:
 			phy_port0: port@9b22000 {
 				reg = <0x9b22000 0xff>,
 				      <0x9b09000 0xff>,
-				      <0x9b04000 0xff>,
-				      <0x114 0x4>, /* sysctrl MiPHY cntrl */
-				      <0x818 0x4>, /* sysctrl MiPHY status*/
-				      <0xe0  0x4>, /* sysctrl PCIe */
-				      <0xec  0x4>; /* sysctrl SATA */
+				      <0x9b04000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x114 0x818 0xe0 0xec>;
 				#phy-cells = <1>;
 				st,osc-rdy;
 				reset-names = "miphy-sw-rst";
@@ -75,18 +70,13 @@ example:
 			phy_port1: port@9b2a000 {
 				reg = <0x9b2a000 0xff>,
 				      <0x9b19000 0xff>,
-				      <0x9b14000 0xff>,
-				      <0x118 0x4>,
-				      <0x81c 0x4>,
-				      <0xe4  0x4>,
-				      <0xf0  0x4>;
+				      <0x9b14000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x118 0x81c 0xe4 0xf0>;
+
 				#phy-cells = <1>;
 				st,osc-force-ext;
 				reset-names = "miphy-sw-rst";
@@ -95,13 +85,12 @@ example:
 
 			phy_port2: port@8f95000 {
 				reg = <0x8f95000 0xff>,
-				      <0x8f90000 0xff>,
-				      <0x11c 0x4>,
-				      <0x820 0x4>;
+				      <0x8f90000 0xff>;
 				reg-names = "pipew",
-				    "usb3-up",
-				    "miphy-ctrl-glue",
-				    "miphy-status-glue";
+					    "usb3-up";
+
+				st,syscfg = <0x11c 0x820>;
+
 				#phy-cells = <1>;
 				reset-names = "miphy-sw-rst";
 				resets = <&softreset STIH407_MIPHY2_SOFTRESET>;
@@ -125,4 +114,4 @@ example:
 
 Macro definitions for the supported miphy configuration can be found in:
 
-include/dt-bindings/phy/phy-miphy28lp.h
+include/dt-bindings/phy/phy.h
diff --git a/drivers/phy/phy-miphy28lp.c b/drivers/phy/phy-miphy28lp.c
index 27fa62c..9b2848e 100644
--- a/drivers/phy/phy-miphy28lp.c
+++ b/drivers/phy/phy-miphy28lp.c
@@ -194,6 +194,14 @@
 #define MIPHY_SATA_BANK_NB	3
 #define MIPHY_PCIE_BANK_NB	2
 
+enum {
+	SYSCFG_CTRL,
+	SYSCFG_STATUS,
+	SYSCFG_PCI,
+	SYSCFG_SATA,
+	SYSCFG_REG_MAX,
+};
+
 struct miphy28lp_phy {
 	struct phy *phy;
 	struct miphy28lp_dev *phydev;
@@ -211,10 +219,7 @@ struct miphy28lp_phy {
 	u32 sata_gen;
 
 	/* Sysconfig registers offsets needed to configure the device */
-	u32 syscfg_miphy_ctrl;
-	u32 syscfg_miphy_status;
-	u32 syscfg_pci;
-	u32 syscfg_sata;
+	u32 syscfg_reg[SYSCFG_REG_MAX];
 	u8 type;
 };
 
@@ -834,12 +839,12 @@ static int miphy_osc_is_ready(struct miphy28lp_phy *miphy_phy)
 	if (!miphy_phy->osc_rdy)
 		return 0;
 
-	if (!miphy_phy->syscfg_miphy_status)
+	if (!miphy_phy->syscfg_reg[SYSCFG_STATUS])
 		return -EINVAL;
 
 	do {
-		regmap_read(miphy_dev->regmap, miphy_phy->syscfg_miphy_status,
-			    &val);
+		regmap_read(miphy_dev->regmap,
+				miphy_phy->syscfg_reg[SYSCFG_STATUS], &val);
 
 		if ((val & MIPHY_OSC_RDY) != MIPHY_OSC_RDY)
 			cpu_relax();
@@ -888,7 +893,7 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	int err;
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 
-	if (!miphy_phy->syscfg_miphy_ctrl)
+	if (!miphy_phy->syscfg_reg[SYSCFG_CTRL])
 		return -EINVAL;
 
 	err = reset_control_assert(miphy_phy->miphy_rst);
@@ -900,7 +905,8 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	if (miphy_phy->osc_force_ext)
 		miphy_val |= MIPHY_OSC_FORCE_EXT;
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_miphy_ctrl,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_CTRL],
 			   MIPHY_CTRL_MASK, miphy_val);
 
 	err = reset_control_deassert(miphy_phy->miphy_rst);
@@ -917,8 +923,9 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err, sata_conf = SATA_CTRL_SELECT_SATA;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
-		|| (!miphy_phy->base))
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI]) ||
+			(!miphy_phy->base))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "sata-up mode, addr 0x%p\n", miphy_phy->base);
@@ -926,10 +933,11 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	/* Configure the glue-logic */
 	sata_conf |= ((miphy_phy->sata_gen - SATA_GEN1) << SATA_SPDMODE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, sata_conf);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
 	/* MiPHY path and clocking init */
@@ -951,17 +959,19 @@ static int miphy28lp_init_pcie(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI])
 		|| (!miphy_phy->base) || (!miphy_phy->pipebase))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "pcie-up mode, addr 0x%p\n", miphy_phy->base);
 
 	/* Configure the glue-logic */
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SYSCFG_PCIE_PCIE_VAL);
 
 	/* MiPHY path and clocking init */
@@ -1156,7 +1166,8 @@ static int miphy28lp_probe_resets(struct device_node *node,
 static int miphy28lp_of_probe(struct device_node *np,
 			      struct miphy28lp_phy *miphy_phy)
 {
-	struct resource res;
+	int i;
+	u32 ctrlreg;
 
 	miphy_phy->osc_force_ext =
 		of_property_read_bool(np, "st,osc-force-ext");
@@ -1175,18 +1186,10 @@ static int miphy28lp_of_probe(struct device_node *np,
 	if (!miphy_phy->sata_gen)
 		miphy_phy->sata_gen = SATA_GEN1;
 
-	if (!miphy28lp_get_resource_byname(np, "miphy-ctrl-glue", &res))
-		miphy_phy->syscfg_miphy_ctrl = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "miphy-status-glue", &res))
-		miphy_phy->syscfg_miphy_status = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "pcie-glue", &res))
-		miphy_phy->syscfg_pci = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "sata-glue", &res))
-		miphy_phy->syscfg_sata = res.start;
-
+	for (i = 0; i < SYSCFG_REG_MAX; i++) {
+		if (!of_property_read_u32_index(np, "st,syscfg", i, &ctrlreg))
+			miphy_phy->syscfg_reg[i] = ctrlreg;
+	}
 
 	return 0;
 }
-- 
1.9.1


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

* [PATCH 1/3] phy: miphy28lp: Pass sysconfig register offsets via syscfg dt property.
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, kernel, linux-kernel, linux-arm-kernel, Lee Jones,
	Gabriel Fernandez

Based on Arnds review comments here https://lkml.org/lkml/2014/11/13/161,
update the miphy28lp phy driver to access sysconfig register offsets via
syscfg dt property.

This is because the reg property should not be mixing address spaces like
it does currently for miphy28lp. This change then also aligns us to how other
platforms such as keystone and bcm7445 pass there syscon offsets via DT.

I have updated the miphy28lp phy driver same way as Peter's implementation.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 2 files changed, 48 insertions(+), 56 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
index 46a135d..89caa88 100644
--- a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
+++ b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
@@ -26,6 +26,7 @@ Required properties (port (child) node):
 		  filled in "reg". It can also contain the offset of the system configuration
 		  registers used as glue-logic to setup the device for SATA/PCIe or USB3
 		  devices.
+- st,syscfg	: Offset of the parent configuration register.
 - resets	: phandle to the parent reset controller.
 - reset-names	: Associated name must be "miphy-sw-rst".
 
@@ -54,18 +55,12 @@ example:
 			phy_port0: port@9b22000 {
 				reg = <0x9b22000 0xff>,
 				      <0x9b09000 0xff>,
-				      <0x9b04000 0xff>,
-				      <0x114 0x4>, /* sysctrl MiPHY cntrl */
-				      <0x818 0x4>, /* sysctrl MiPHY status*/
-				      <0xe0  0x4>, /* sysctrl PCIe */
-				      <0xec  0x4>; /* sysctrl SATA */
+				      <0x9b04000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x114 0x818 0xe0 0xec>;
 				#phy-cells = <1>;
 				st,osc-rdy;
 				reset-names = "miphy-sw-rst";
@@ -75,18 +70,13 @@ example:
 			phy_port1: port@9b2a000 {
 				reg = <0x9b2a000 0xff>,
 				      <0x9b19000 0xff>,
-				      <0x9b14000 0xff>,
-				      <0x118 0x4>,
-				      <0x81c 0x4>,
-				      <0xe4  0x4>,
-				      <0xf0  0x4>;
+				      <0x9b14000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x118 0x81c 0xe4 0xf0>;
+
 				#phy-cells = <1>;
 				st,osc-force-ext;
 				reset-names = "miphy-sw-rst";
@@ -95,13 +85,12 @@ example:
 
 			phy_port2: port@8f95000 {
 				reg = <0x8f95000 0xff>,
-				      <0x8f90000 0xff>,
-				      <0x11c 0x4>,
-				      <0x820 0x4>;
+				      <0x8f90000 0xff>;
 				reg-names = "pipew",
-				    "usb3-up",
-				    "miphy-ctrl-glue",
-				    "miphy-status-glue";
+					    "usb3-up";
+
+				st,syscfg = <0x11c 0x820>;
+
 				#phy-cells = <1>;
 				reset-names = "miphy-sw-rst";
 				resets = <&softreset STIH407_MIPHY2_SOFTRESET>;
@@ -125,4 +114,4 @@ example:
 
 Macro definitions for the supported miphy configuration can be found in:
 
-include/dt-bindings/phy/phy-miphy28lp.h
+include/dt-bindings/phy/phy.h
diff --git a/drivers/phy/phy-miphy28lp.c b/drivers/phy/phy-miphy28lp.c
index 27fa62c..9b2848e 100644
--- a/drivers/phy/phy-miphy28lp.c
+++ b/drivers/phy/phy-miphy28lp.c
@@ -194,6 +194,14 @@
 #define MIPHY_SATA_BANK_NB	3
 #define MIPHY_PCIE_BANK_NB	2
 
+enum {
+	SYSCFG_CTRL,
+	SYSCFG_STATUS,
+	SYSCFG_PCI,
+	SYSCFG_SATA,
+	SYSCFG_REG_MAX,
+};
+
 struct miphy28lp_phy {
 	struct phy *phy;
 	struct miphy28lp_dev *phydev;
@@ -211,10 +219,7 @@ struct miphy28lp_phy {
 	u32 sata_gen;
 
 	/* Sysconfig registers offsets needed to configure the device */
-	u32 syscfg_miphy_ctrl;
-	u32 syscfg_miphy_status;
-	u32 syscfg_pci;
-	u32 syscfg_sata;
+	u32 syscfg_reg[SYSCFG_REG_MAX];
 	u8 type;
 };
 
@@ -834,12 +839,12 @@ static int miphy_osc_is_ready(struct miphy28lp_phy *miphy_phy)
 	if (!miphy_phy->osc_rdy)
 		return 0;
 
-	if (!miphy_phy->syscfg_miphy_status)
+	if (!miphy_phy->syscfg_reg[SYSCFG_STATUS])
 		return -EINVAL;
 
 	do {
-		regmap_read(miphy_dev->regmap, miphy_phy->syscfg_miphy_status,
-			    &val);
+		regmap_read(miphy_dev->regmap,
+				miphy_phy->syscfg_reg[SYSCFG_STATUS], &val);
 
 		if ((val & MIPHY_OSC_RDY) != MIPHY_OSC_RDY)
 			cpu_relax();
@@ -888,7 +893,7 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	int err;
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 
-	if (!miphy_phy->syscfg_miphy_ctrl)
+	if (!miphy_phy->syscfg_reg[SYSCFG_CTRL])
 		return -EINVAL;
 
 	err = reset_control_assert(miphy_phy->miphy_rst);
@@ -900,7 +905,8 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	if (miphy_phy->osc_force_ext)
 		miphy_val |= MIPHY_OSC_FORCE_EXT;
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_miphy_ctrl,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_CTRL],
 			   MIPHY_CTRL_MASK, miphy_val);
 
 	err = reset_control_deassert(miphy_phy->miphy_rst);
@@ -917,8 +923,9 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err, sata_conf = SATA_CTRL_SELECT_SATA;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
-		|| (!miphy_phy->base))
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI]) ||
+			(!miphy_phy->base))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "sata-up mode, addr 0x%p\n", miphy_phy->base);
@@ -926,10 +933,11 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	/* Configure the glue-logic */
 	sata_conf |= ((miphy_phy->sata_gen - SATA_GEN1) << SATA_SPDMODE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, sata_conf);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
 	/* MiPHY path and clocking init */
@@ -951,17 +959,19 @@ static int miphy28lp_init_pcie(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI])
 		|| (!miphy_phy->base) || (!miphy_phy->pipebase))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "pcie-up mode, addr 0x%p\n", miphy_phy->base);
 
 	/* Configure the glue-logic */
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SYSCFG_PCIE_PCIE_VAL);
 
 	/* MiPHY path and clocking init */
@@ -1156,7 +1166,8 @@ static int miphy28lp_probe_resets(struct device_node *node,
 static int miphy28lp_of_probe(struct device_node *np,
 			      struct miphy28lp_phy *miphy_phy)
 {
-	struct resource res;
+	int i;
+	u32 ctrlreg;
 
 	miphy_phy->osc_force_ext =
 		of_property_read_bool(np, "st,osc-force-ext");
@@ -1175,18 +1186,10 @@ static int miphy28lp_of_probe(struct device_node *np,
 	if (!miphy_phy->sata_gen)
 		miphy_phy->sata_gen = SATA_GEN1;
 
-	if (!miphy28lp_get_resource_byname(np, "miphy-ctrl-glue", &res))
-		miphy_phy->syscfg_miphy_ctrl = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "miphy-status-glue", &res))
-		miphy_phy->syscfg_miphy_status = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "pcie-glue", &res))
-		miphy_phy->syscfg_pci = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "sata-glue", &res))
-		miphy_phy->syscfg_sata = res.start;
-
+	for (i = 0; i < SYSCFG_REG_MAX; i++) {
+		if (!of_property_read_u32_index(np, "st,syscfg", i, &ctrlreg))
+			miphy_phy->syscfg_reg[i] = ctrlreg;
+	}
 
 	return 0;
 }
-- 
1.9.1

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

* [PATCH 1/3] phy: miphy28lp: Pass sysconfig register offsets via syscfg dt property.
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: linux-arm-kernel

Based on Arnds review comments here https://lkml.org/lkml/2014/11/13/161,
update the miphy28lp phy driver to access sysconfig register offsets via
syscfg dt property.

This is because the reg property should not be mixing address spaces like
it does currently for miphy28lp. This change then also aligns us to how other
platforms such as keystone and bcm7445 pass there syscon offsets via DT.

I have updated the miphy28lp phy driver same way as Peter's implementation.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
 drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
 2 files changed, 48 insertions(+), 56 deletions(-)

diff --git a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
index 46a135d..89caa88 100644
--- a/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
+++ b/Documentation/devicetree/bindings/phy/phy-miphy28lp.txt
@@ -26,6 +26,7 @@ Required properties (port (child) node):
 		  filled in "reg". It can also contain the offset of the system configuration
 		  registers used as glue-logic to setup the device for SATA/PCIe or USB3
 		  devices.
+- st,syscfg	: Offset of the parent configuration register.
 - resets	: phandle to the parent reset controller.
 - reset-names	: Associated name must be "miphy-sw-rst".
 
@@ -54,18 +55,12 @@ example:
 			phy_port0: port at 9b22000 {
 				reg = <0x9b22000 0xff>,
 				      <0x9b09000 0xff>,
-				      <0x9b04000 0xff>,
-				      <0x114 0x4>, /* sysctrl MiPHY cntrl */
-				      <0x818 0x4>, /* sysctrl MiPHY status*/
-				      <0xe0  0x4>, /* sysctrl PCIe */
-				      <0xec  0x4>; /* sysctrl SATA */
+				      <0x9b04000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x114 0x818 0xe0 0xec>;
 				#phy-cells = <1>;
 				st,osc-rdy;
 				reset-names = "miphy-sw-rst";
@@ -75,18 +70,13 @@ example:
 			phy_port1: port at 9b2a000 {
 				reg = <0x9b2a000 0xff>,
 				      <0x9b19000 0xff>,
-				      <0x9b14000 0xff>,
-				      <0x118 0x4>,
-				      <0x81c 0x4>,
-				      <0xe4  0x4>,
-				      <0xf0  0x4>;
+				      <0x9b14000 0xff>;
 				reg-names = "sata-up",
 					    "pcie-up",
-					    "pipew",
-					    "miphy-ctrl-glue",
-					    "miphy-status-glue",
-					    "pcie-glue",
-					    "sata-glue";
+					    "pipew";
+
+				st,syscfg = <0x118 0x81c 0xe4 0xf0>;
+
 				#phy-cells = <1>;
 				st,osc-force-ext;
 				reset-names = "miphy-sw-rst";
@@ -95,13 +85,12 @@ example:
 
 			phy_port2: port at 8f95000 {
 				reg = <0x8f95000 0xff>,
-				      <0x8f90000 0xff>,
-				      <0x11c 0x4>,
-				      <0x820 0x4>;
+				      <0x8f90000 0xff>;
 				reg-names = "pipew",
-				    "usb3-up",
-				    "miphy-ctrl-glue",
-				    "miphy-status-glue";
+					    "usb3-up";
+
+				st,syscfg = <0x11c 0x820>;
+
 				#phy-cells = <1>;
 				reset-names = "miphy-sw-rst";
 				resets = <&softreset STIH407_MIPHY2_SOFTRESET>;
@@ -125,4 +114,4 @@ example:
 
 Macro definitions for the supported miphy configuration can be found in:
 
-include/dt-bindings/phy/phy-miphy28lp.h
+include/dt-bindings/phy/phy.h
diff --git a/drivers/phy/phy-miphy28lp.c b/drivers/phy/phy-miphy28lp.c
index 27fa62c..9b2848e 100644
--- a/drivers/phy/phy-miphy28lp.c
+++ b/drivers/phy/phy-miphy28lp.c
@@ -194,6 +194,14 @@
 #define MIPHY_SATA_BANK_NB	3
 #define MIPHY_PCIE_BANK_NB	2
 
+enum {
+	SYSCFG_CTRL,
+	SYSCFG_STATUS,
+	SYSCFG_PCI,
+	SYSCFG_SATA,
+	SYSCFG_REG_MAX,
+};
+
 struct miphy28lp_phy {
 	struct phy *phy;
 	struct miphy28lp_dev *phydev;
@@ -211,10 +219,7 @@ struct miphy28lp_phy {
 	u32 sata_gen;
 
 	/* Sysconfig registers offsets needed to configure the device */
-	u32 syscfg_miphy_ctrl;
-	u32 syscfg_miphy_status;
-	u32 syscfg_pci;
-	u32 syscfg_sata;
+	u32 syscfg_reg[SYSCFG_REG_MAX];
 	u8 type;
 };
 
@@ -834,12 +839,12 @@ static int miphy_osc_is_ready(struct miphy28lp_phy *miphy_phy)
 	if (!miphy_phy->osc_rdy)
 		return 0;
 
-	if (!miphy_phy->syscfg_miphy_status)
+	if (!miphy_phy->syscfg_reg[SYSCFG_STATUS])
 		return -EINVAL;
 
 	do {
-		regmap_read(miphy_dev->regmap, miphy_phy->syscfg_miphy_status,
-			    &val);
+		regmap_read(miphy_dev->regmap,
+				miphy_phy->syscfg_reg[SYSCFG_STATUS], &val);
 
 		if ((val & MIPHY_OSC_RDY) != MIPHY_OSC_RDY)
 			cpu_relax();
@@ -888,7 +893,7 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	int err;
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 
-	if (!miphy_phy->syscfg_miphy_ctrl)
+	if (!miphy_phy->syscfg_reg[SYSCFG_CTRL])
 		return -EINVAL;
 
 	err = reset_control_assert(miphy_phy->miphy_rst);
@@ -900,7 +905,8 @@ static int miphy28lp_setup(struct miphy28lp_phy *miphy_phy, u32 miphy_val)
 	if (miphy_phy->osc_force_ext)
 		miphy_val |= MIPHY_OSC_FORCE_EXT;
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_miphy_ctrl,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_CTRL],
 			   MIPHY_CTRL_MASK, miphy_val);
 
 	err = reset_control_deassert(miphy_phy->miphy_rst);
@@ -917,8 +923,9 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err, sata_conf = SATA_CTRL_SELECT_SATA;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
-		|| (!miphy_phy->base))
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI]) ||
+			(!miphy_phy->base))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "sata-up mode, addr 0x%p\n", miphy_phy->base);
@@ -926,10 +933,11 @@ static int miphy28lp_init_sata(struct miphy28lp_phy *miphy_phy)
 	/* Configure the glue-logic */
 	sata_conf |= ((miphy_phy->sata_gen - SATA_GEN1) << SATA_SPDMODE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, sata_conf);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
 	/* MiPHY path and clocking init */
@@ -951,17 +959,19 @@ static int miphy28lp_init_pcie(struct miphy28lp_phy *miphy_phy)
 	struct miphy28lp_dev *miphy_dev = miphy_phy->phydev;
 	int err;
 
-	if ((!miphy_phy->syscfg_sata) || (!miphy_phy->syscfg_pci)
+	if ((!miphy_phy->syscfg_reg[SYSCFG_SATA]) ||
+			(!miphy_phy->syscfg_reg[SYSCFG_PCI])
 		|| (!miphy_phy->base) || (!miphy_phy->pipebase))
 		return -EINVAL;
 
 	dev_info(miphy_dev->dev, "pcie-up mode, addr 0x%p\n", miphy_phy->base);
 
 	/* Configure the glue-logic */
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_sata,
+	regmap_update_bits(miphy_dev->regmap,
+			   miphy_phy->syscfg_reg[SYSCFG_SATA],
 			   SATA_CTRL_MASK, SATA_CTRL_SELECT_PCIE);
 
-	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_pci,
+	regmap_update_bits(miphy_dev->regmap, miphy_phy->syscfg_reg[SYSCFG_PCI],
 			   PCIE_CTRL_MASK, SYSCFG_PCIE_PCIE_VAL);
 
 	/* MiPHY path and clocking init */
@@ -1156,7 +1166,8 @@ static int miphy28lp_probe_resets(struct device_node *node,
 static int miphy28lp_of_probe(struct device_node *np,
 			      struct miphy28lp_phy *miphy_phy)
 {
-	struct resource res;
+	int i;
+	u32 ctrlreg;
 
 	miphy_phy->osc_force_ext =
 		of_property_read_bool(np, "st,osc-force-ext");
@@ -1175,18 +1186,10 @@ static int miphy28lp_of_probe(struct device_node *np,
 	if (!miphy_phy->sata_gen)
 		miphy_phy->sata_gen = SATA_GEN1;
 
-	if (!miphy28lp_get_resource_byname(np, "miphy-ctrl-glue", &res))
-		miphy_phy->syscfg_miphy_ctrl = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "miphy-status-glue", &res))
-		miphy_phy->syscfg_miphy_status = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "pcie-glue", &res))
-		miphy_phy->syscfg_pci = res.start;
-
-	if (!miphy28lp_get_resource_byname(np, "sata-glue", &res))
-		miphy_phy->syscfg_sata = res.start;
-
+	for (i = 0; i < SYSCFG_REG_MAX; i++) {
+		if (!of_property_read_u32_index(np, "st,syscfg", i, &ctrlreg))
+			miphy_phy->syscfg_reg[i] = ctrlreg;
+	}
 
 	return 0;
 }
-- 
1.9.1

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

* [PATCH 2/3] ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
  2015-01-14  9:54 ` Gabriel FERNANDEZ
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  -1 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

The MiPHY28lp is a Generic PHY which can serve various SATA, PCIe or
USB3 devices. The two first ports can be use for either; both SATA, both
PCIe or one of each in any configuration.
The Third port is only for USB3.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 arch/arm/boot/dts/stih407-family.dtsi | 53 +++++++++++++++++++++++++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi  | 11 ++++++++
 2 files changed, 64 insertions(+)

diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
index d4a8f84..c06a546 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -283,5 +283,58 @@
 				 <&picophyreset STIH407_PICOPHY0_RESET>;
 			reset-names = "global", "port";
 		};
+
+		miphy28lp_phy: miphy28lp@9b22000 {
+			compatible = "st,miphy28lp-phy";
+			st,syscfg = <&syscfg_core>;
+			#address-cells	= <1>;
+			#size-cells	= <1>;
+			ranges;
+
+			phy_port0: port@9b22000 {
+				reg = <0x9b22000 0xff>,
+				      <0x9b09000 0xff>,
+				      <0x9b04000 0xff>;
+				reg-names = "sata-up",
+					    "pcie-up",
+					    "pipew";
+
+				st,syscfg = <0x114 0x818 0xe0 0xec>;
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY0_SOFTRESET>;
+			};
+
+			phy_port1: port@9b2a000 {
+				reg = <0x9b2a000 0xff>,
+				      <0x9b19000 0xff>,
+				      <0x9b14000 0xff>;
+				reg-names = "sata-up",
+					    "pcie-up",
+					    "pipew";
+
+				st,syscfg = <0x118 0x81c 0xe4 0xf0>;
+
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY1_SOFTRESET>;
+			};
+
+			phy_port2: port@8f95000 {
+				reg = <0x8f95000 0xff>,
+				      <0x8f90000 0xff>;
+				reg-names = "pipew",
+					    "usb3-up";
+
+				st,syscfg = <0x11c 0x820>;
+
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY2_SOFTRESET>;
+			};
+		};
 	};
 };
diff --git a/arch/arm/boot/dts/stihxxx-b2120.dtsi b/arch/arm/boot/dts/stihxxx-b2120.dtsi
index 0074bd4..8af5282 100644
--- a/arch/arm/boot/dts/stihxxx-b2120.dtsi
+++ b/arch/arm/boot/dts/stihxxx-b2120.dtsi
@@ -55,5 +55,16 @@
 			st,i2c-min-scl-pulse-width-us = <0>;
 			st,i2c-min-sda-pulse-width-us = <5>;
 		};
+
+		miphy28lp_phy: miphy28lp@9b22000 {
+
+			phy_port0: port@9b22000 {
+				st,osc-rdy;
+			};
+
+			phy_port1: port@9b2a000 {
+				st,osc-force-ext;
+			};
+		};
 	};
 };
-- 
1.9.1


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

* [PATCH 2/3] ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: linux-arm-kernel

The MiPHY28lp is a Generic PHY which can serve various SATA, PCIe or
USB3 devices. The two first ports can be use for either; both SATA, both
PCIe or one of each in any configuration.
The Third port is only for USB3.

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 arch/arm/boot/dts/stih407-family.dtsi | 53 +++++++++++++++++++++++++++++++++++
 arch/arm/boot/dts/stihxxx-b2120.dtsi  | 11 ++++++++
 2 files changed, 64 insertions(+)

diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi
index d4a8f84..c06a546 100644
--- a/arch/arm/boot/dts/stih407-family.dtsi
+++ b/arch/arm/boot/dts/stih407-family.dtsi
@@ -283,5 +283,58 @@
 				 <&picophyreset STIH407_PICOPHY0_RESET>;
 			reset-names = "global", "port";
 		};
+
+		miphy28lp_phy: miphy28lp at 9b22000 {
+			compatible = "st,miphy28lp-phy";
+			st,syscfg = <&syscfg_core>;
+			#address-cells	= <1>;
+			#size-cells	= <1>;
+			ranges;
+
+			phy_port0: port at 9b22000 {
+				reg = <0x9b22000 0xff>,
+				      <0x9b09000 0xff>,
+				      <0x9b04000 0xff>;
+				reg-names = "sata-up",
+					    "pcie-up",
+					    "pipew";
+
+				st,syscfg = <0x114 0x818 0xe0 0xec>;
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY0_SOFTRESET>;
+			};
+
+			phy_port1: port at 9b2a000 {
+				reg = <0x9b2a000 0xff>,
+				      <0x9b19000 0xff>,
+				      <0x9b14000 0xff>;
+				reg-names = "sata-up",
+					    "pcie-up",
+					    "pipew";
+
+				st,syscfg = <0x118 0x81c 0xe4 0xf0>;
+
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY1_SOFTRESET>;
+			};
+
+			phy_port2: port at 8f95000 {
+				reg = <0x8f95000 0xff>,
+				      <0x8f90000 0xff>;
+				reg-names = "pipew",
+					    "usb3-up";
+
+				st,syscfg = <0x11c 0x820>;
+
+				#phy-cells = <1>;
+
+				reset-names = "miphy-sw-rst";
+				resets = <&softreset STIH407_MIPHY2_SOFTRESET>;
+			};
+		};
 	};
 };
diff --git a/arch/arm/boot/dts/stihxxx-b2120.dtsi b/arch/arm/boot/dts/stihxxx-b2120.dtsi
index 0074bd4..8af5282 100644
--- a/arch/arm/boot/dts/stihxxx-b2120.dtsi
+++ b/arch/arm/boot/dts/stihxxx-b2120.dtsi
@@ -55,5 +55,16 @@
 			st,i2c-min-scl-pulse-width-us = <0>;
 			st,i2c-min-sda-pulse-width-us = <5>;
 		};
+
+		miphy28lp_phy: miphy28lp at 9b22000 {
+
+			phy_port0: port at 9b22000 {
+				st,osc-rdy;
+			};
+
+			phy_port1: port at 9b2a000 {
+				st,osc-force-ext;
+			};
+		};
 	};
 };
-- 
1.9.1

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

* [PATCH 3/3] ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe & USB3) PHY
  2015-01-14  9:54 ` Gabriel FERNANDEZ
  (?)
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  -1 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 arch/arm/configs/multi_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 4b87fd1..88dfa7e 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -459,6 +459,7 @@ CONFIG_PWM_VT8500=y
 CONFIG_PHY_HIX5HD2_SATA=y
 CONFIG_OMAP_USB2=y
 CONFIG_TI_PIPE3=y
+CONFIG_PHY_MIPHY28LP=y
 CONFIG_PHY_MIPHY365X=y
 CONFIG_PHY_STIH41X_USB=y
 CONFIG_PHY_STIH407_USB=y
-- 
1.9.1


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

* [PATCH 3/3] ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe & USB3) PHY
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: Rob Herring, Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Maxime Coquelin, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, kernel, linux-kernel, linux-arm-kernel, Lee Jones,
	Gabriel Fernandez

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 arch/arm/configs/multi_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 4b87fd1..88dfa7e 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -459,6 +459,7 @@ CONFIG_PWM_VT8500=y
 CONFIG_PHY_HIX5HD2_SATA=y
 CONFIG_OMAP_USB2=y
 CONFIG_TI_PIPE3=y
+CONFIG_PHY_MIPHY28LP=y
 CONFIG_PHY_MIPHY365X=y
 CONFIG_PHY_STIH41X_USB=y
 CONFIG_PHY_STIH407_USB=y
-- 
1.9.1

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

* [PATCH 3/3] ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe & USB3) PHY
@ 2015-01-14  9:54   ` Gabriel FERNANDEZ
  0 siblings, 0 replies; 23+ messages in thread
From: Gabriel FERNANDEZ @ 2015-01-14  9:54 UTC (permalink / raw)
  To: linux-arm-kernel

Signed-off-by: Gabriel Fernandez <gabriel.fernandez@linaro.org>
---
 arch/arm/configs/multi_v7_defconfig | 1 +
 1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 4b87fd1..88dfa7e 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -459,6 +459,7 @@ CONFIG_PWM_VT8500=y
 CONFIG_PHY_HIX5HD2_SATA=y
 CONFIG_OMAP_USB2=y
 CONFIG_TI_PIPE3=y
+CONFIG_PHY_MIPHY28LP=y
 CONFIG_PHY_MIPHY365X=y
 CONFIG_PHY_STIH41X_USB=y
 CONFIG_PHY_STIH407_USB=y
-- 
1.9.1

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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 11:56   ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 11:56 UTC (permalink / raw)
  To: Gabriel FERNANDEZ, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Srinivas Kandagatla, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

Hi Gabriel,

     For the series:
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

     Kishon, could you take care of adding PHY driver patch for v3.20?

Thanks,
Maxime
On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>
> The first patch is to update miphy28lp phy driver to access sysconfig register
> offsets via syscfg dt property. It's based on Arnds review comments here
> https://lkml.org/lkml/2014/11/13/161
> I have updated the miphy28lp phy driver same way as Peter's implementation.
>
> Gabriel Fernandez (3):
>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>      property.
>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>      USB3) PHY
>
>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>   arch/arm/configs/multi_v7_defconfig                |  1 +
>   drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
>   5 files changed, 113 insertions(+), 56 deletions(-)
>


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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 11:56   ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 11:56 UTC (permalink / raw)
  To: Gabriel FERNANDEZ, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Srinivas Kandagatla, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ, Lee Jones, Gabriel Fernandez

Hi Gabriel,

     For the series:
Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>

     Kishon, could you take care of adding PHY driver patch for v3.20?

Thanks,
Maxime
On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>
> The first patch is to update miphy28lp phy driver to access sysconfig register
> offsets via syscfg dt property. It's based on Arnds review comments here
> https://lkml.org/lkml/2014/11/13/161
> I have updated the miphy28lp phy driver same way as Peter's implementation.
>
> Gabriel Fernandez (3):
>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>      property.
>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>      USB3) PHY
>
>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>   arch/arm/configs/multi_v7_defconfig                |  1 +
>   drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
>   5 files changed, 113 insertions(+), 56 deletions(-)
>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 11:56   ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 11:56 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Gabriel,

     For the series:
Acked-by: Maxime Coquelin <maxime.coquelin@st.com>

     Kishon, could you take care of adding PHY driver patch for v3.20?

Thanks,
Maxime
On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>
> The first patch is to update miphy28lp phy driver to access sysconfig register
> offsets via syscfg dt property. It's based on Arnds review comments here
> https://lkml.org/lkml/2014/11/13/161
> I have updated the miphy28lp phy driver same way as Peter's implementation.
>
> Gabriel Fernandez (3):
>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>      property.
>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>      USB3) PHY
>
>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>   arch/arm/configs/multi_v7_defconfig                |  1 +
>   drivers/phy/phy-miphy28lp.c                        | 61 ++++++++++++----------
>   5 files changed, 113 insertions(+), 56 deletions(-)
>

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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 12:32     ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 12:32 UTC (permalink / raw)
  To: Gabriel FERNANDEZ, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Srinivas Kandagatla, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez


On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
> Hi Gabriel,
>
>     For the series:
> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>
>     Kishon, could you take care of adding PHY driver patch for v3.20?
Hi Kishon,

     Don't apply the patch, since we have 2 other PHY driver patches, I 
will send you a pull request.

Best regards,
Maxime


>
> Thanks,
> Maxime
> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>> The goal of this series is to enable the support of MiPHY28lp Generic 
>> PHY.
>>
>> The first patch is to update miphy28lp phy driver to access sysconfig 
>> register
>> offsets via syscfg dt property. It's based on Arnds review comments here
>> https://lkml.org/lkml/2014/11/13/161
>> I have updated the miphy28lp phy driver same way as Peter's 
>> implementation.
>>
>> Gabriel Fernandez (3):
>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>      property.
>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, 
>> PCIe &
>>      USB3) PHY
>>
>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 
>> ++++++---------
>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 
>> +++++++++++++++++++
>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>   drivers/phy/phy-miphy28lp.c                        | 61 
>> ++++++++++++----------
>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>
>


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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 12:32     ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 12:32 UTC (permalink / raw)
  To: Gabriel FERNANDEZ, Rob Herring, Pawel Moll, Mark Rutland,
	Ian Campbell, Kumar Gala, Srinivas Kandagatla, Patrice Chotard,
	Russell King, Kishon Vijay Abraham I, Grant Likely
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ, Lee Jones, Gabriel Fernandez


On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
> Hi Gabriel,
>
>     For the series:
> Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>
>
>     Kishon, could you take care of adding PHY driver patch for v3.20?
Hi Kishon,

     Don't apply the patch, since we have 2 other PHY driver patches, I 
will send you a pull request.

Best regards,
Maxime


>
> Thanks,
> Maxime
> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>> The goal of this series is to enable the support of MiPHY28lp Generic 
>> PHY.
>>
>> The first patch is to update miphy28lp phy driver to access sysconfig 
>> register
>> offsets via syscfg dt property. It's based on Arnds review comments here
>> https://lkml.org/lkml/2014/11/13/161
>> I have updated the miphy28lp phy driver same way as Peter's 
>> implementation.
>>
>> Gabriel Fernandez (3):
>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>      property.
>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, 
>> PCIe &
>>      USB3) PHY
>>
>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 
>> ++++++---------
>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 
>> +++++++++++++++++++
>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>   drivers/phy/phy-miphy28lp.c                        | 61 
>> ++++++++++++----------
>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>
>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 12:32     ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 12:32 UTC (permalink / raw)
  To: linux-arm-kernel


On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
> Hi Gabriel,
>
>     For the series:
> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>
>     Kishon, could you take care of adding PHY driver patch for v3.20?
Hi Kishon,

     Don't apply the patch, since we have 2 other PHY driver patches, I 
will send you a pull request.

Best regards,
Maxime


>
> Thanks,
> Maxime
> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>> The goal of this series is to enable the support of MiPHY28lp Generic 
>> PHY.
>>
>> The first patch is to update miphy28lp phy driver to access sysconfig 
>> register
>> offsets via syscfg dt property. It's based on Arnds review comments here
>> https://lkml.org/lkml/2014/11/13/161
>> I have updated the miphy28lp phy driver same way as Peter's 
>> implementation.
>>
>> Gabriel Fernandez (3):
>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>      property.
>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, 
>> PCIe &
>>      USB3) PHY
>>
>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 
>> ++++++---------
>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 
>> +++++++++++++++++++
>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>   drivers/phy/phy-miphy28lp.c                        | 61 
>> ++++++++++++----------
>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>
>

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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 13:19       ` Kishon Vijay Abraham I
  0 siblings, 0 replies; 23+ messages in thread
From: Kishon Vijay Abraham I @ 2015-01-16 13:19 UTC (permalink / raw)
  To: Maxime Coquelin, Gabriel FERNANDEZ, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Srinivas Kandagatla,
	Patrice Chotard, Russell King, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

Hi,

On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
> 
> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>> Hi Gabriel,
>>
>>     For the series:
>> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>>
>>     Kishon, could you take care of adding PHY driver patch for v3.20?
> Hi Kishon,
> 
>     Don't apply the patch, since we have 2 other PHY driver patches, I will
> send you a pull request.

Send me as patches if that is okay.

Thanks
Kishon

> 
> Best regards,
> Maxime
> 
> 
>>
>> Thanks,
>> Maxime
>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>
>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>> https://lkml.org/lkml/2014/11/13/161
>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>
>>> Gabriel Fernandez (3):
>>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>      property.
>>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>      USB3) PHY
>>>
>>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>>   drivers/phy/phy-miphy28lp.c                        | 61
>>> ++++++++++++----------
>>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>>
>>
> 

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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 13:19       ` Kishon Vijay Abraham I
  0 siblings, 0 replies; 23+ messages in thread
From: Kishon Vijay Abraham I @ 2015-01-16 13:19 UTC (permalink / raw)
  To: Maxime Coquelin, Gabriel FERNANDEZ, Rob Herring, Pawel Moll,
	Mark Rutland, Ian Campbell, Kumar Gala, Srinivas Kandagatla,
	Patrice Chotard, Russell King, Grant Likely
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ, Lee Jones, Gabriel Fernandez

Hi,

On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
> 
> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>> Hi Gabriel,
>>
>>     For the series:
>> Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>
>>
>>     Kishon, could you take care of adding PHY driver patch for v3.20?
> Hi Kishon,
> 
>     Don't apply the patch, since we have 2 other PHY driver patches, I will
> send you a pull request.

Send me as patches if that is okay.

Thanks
Kishon

> 
> Best regards,
> Maxime
> 
> 
>>
>> Thanks,
>> Maxime
>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>
>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>> https://lkml.org/lkml/2014/11/13/161
>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>
>>> Gabriel Fernandez (3):
>>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>      property.
>>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>      USB3) PHY
>>>
>>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>>   drivers/phy/phy-miphy28lp.c                        | 61
>>> ++++++++++++----------
>>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>>
>>
> 
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 13:19       ` Kishon Vijay Abraham I
  0 siblings, 0 replies; 23+ messages in thread
From: Kishon Vijay Abraham I @ 2015-01-16 13:19 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
> 
> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>> Hi Gabriel,
>>
>>     For the series:
>> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>>
>>     Kishon, could you take care of adding PHY driver patch for v3.20?
> Hi Kishon,
> 
>     Don't apply the patch, since we have 2 other PHY driver patches, I will
> send you a pull request.

Send me as patches if that is okay.

Thanks
Kishon

> 
> Best regards,
> Maxime
> 
> 
>>
>> Thanks,
>> Maxime
>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>
>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>> https://lkml.org/lkml/2014/11/13/161
>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>
>>> Gabriel Fernandez (3):
>>>    phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>      property.
>>>    ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>    ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>      USB3) PHY
>>>
>>>   .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>   arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>   arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>   arch/arm/configs/multi_v7_defconfig                |  1 +
>>>   drivers/phy/phy-miphy28lp.c                        | 61
>>> ++++++++++++----------
>>>   5 files changed, 113 insertions(+), 56 deletions(-)
>>>
>>
> 

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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 15:04         ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 15:04 UTC (permalink / raw)
  To: Kishon Vijay Abraham I, Gabriel FERNANDEZ, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Patrice Chotard, Russell King, Grant Likely
  Cc: devicetree, linux-kernel, linux-arm-kernel, kernel, Lee Jones,
	Gabriel Fernandez

Hi,

On 01/16/2015 02:19 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
>> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>>> Hi Gabriel,
>>>
>>>      For the series:
>>> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>>>
>>>      Kishon, could you take care of adding PHY driver patch for v3.20?
>> Hi Kishon,
>>
>>      Don't apply the patch, since we have 2 other PHY driver patches, I will
>> send you a pull request.
> Send me as patches if that is okay.
Ok. Just sent the patches.

Thanks,
Maxime
>
> Thanks
> Kishon
>
>> Best regards,
>> Maxime
>>
>>
>>> Thanks,
>>> Maxime
>>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>>
>>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>>> https://lkml.org/lkml/2014/11/13/161
>>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>>
>>>> Gabriel Fernandez (3):
>>>>     phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>>       property.
>>>>     ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>>     ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>>       USB3) PHY
>>>>
>>>>    .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>>    arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>>    arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>>    arch/arm/configs/multi_v7_defconfig                |  1 +
>>>>    drivers/phy/phy-miphy28lp.c                        | 61
>>>> ++++++++++++----------
>>>>    5 files changed, 113 insertions(+), 56 deletions(-)
>>>>


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

* Re: [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 15:04         ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 15:04 UTC (permalink / raw)
  To: Kishon Vijay Abraham I, Gabriel FERNANDEZ, Rob Herring,
	Pawel Moll, Mark Rutland, Ian Campbell, Kumar Gala,
	Srinivas Kandagatla, Patrice Chotard, Russell King, Grant Likely
  Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	kernel-F5mvAk5X5gdBDgjK7y7TUQ, Lee Jones, Gabriel Fernandez

Hi,

On 01/16/2015 02:19 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
>> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>>> Hi Gabriel,
>>>
>>>      For the series:
>>> Acked-by: Maxime Coquelin <maxime.coquelin-qxv4g6HH51o@public.gmane.org>
>>>
>>>      Kishon, could you take care of adding PHY driver patch for v3.20?
>> Hi Kishon,
>>
>>      Don't apply the patch, since we have 2 other PHY driver patches, I will
>> send you a pull request.
> Send me as patches if that is okay.
Ok. Just sent the patches.

Thanks,
Maxime
>
> Thanks
> Kishon
>
>> Best regards,
>> Maxime
>>
>>
>>> Thanks,
>>> Maxime
>>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>>
>>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>>> https://lkml.org/lkml/2014/11/13/161
>>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>>
>>>> Gabriel Fernandez (3):
>>>>     phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>>       property.
>>>>     ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>>     ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>>       USB3) PHY
>>>>
>>>>    .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>>    arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>>    arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>>    arch/arm/configs/multi_v7_defconfig                |  1 +
>>>>    drivers/phy/phy-miphy28lp.c                        | 61
>>>> ++++++++++++----------
>>>>    5 files changed, 113 insertions(+), 56 deletions(-)
>>>>

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

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

* [PATCH 0/3] Enable myphy28lp support
@ 2015-01-16 15:04         ` Maxime Coquelin
  0 siblings, 0 replies; 23+ messages in thread
From: Maxime Coquelin @ 2015-01-16 15:04 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On 01/16/2015 02:19 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Friday 16 January 2015 06:02 PM, Maxime Coquelin wrote:
>> On 01/16/2015 12:56 PM, Maxime Coquelin wrote:
>>> Hi Gabriel,
>>>
>>>      For the series:
>>> Acked-by: Maxime Coquelin <maxime.coquelin@st.com>
>>>
>>>      Kishon, could you take care of adding PHY driver patch for v3.20?
>> Hi Kishon,
>>
>>      Don't apply the patch, since we have 2 other PHY driver patches, I will
>> send you a pull request.
> Send me as patches if that is okay.
Ok. Just sent the patches.

Thanks,
Maxime
>
> Thanks
> Kishon
>
>> Best regards,
>> Maxime
>>
>>
>>> Thanks,
>>> Maxime
>>> On 01/14/2015 10:54 AM, Gabriel FERNANDEZ wrote:
>>>> The goal of this series is to enable the support of MiPHY28lp Generic PHY.
>>>>
>>>> The first patch is to update miphy28lp phy driver to access sysconfig register
>>>> offsets via syscfg dt property. It's based on Arnds review comments here
>>>> https://lkml.org/lkml/2014/11/13/161
>>>> I have updated the miphy28lp phy driver same way as Peter's implementation.
>>>>
>>>> Gabriel Fernandez (3):
>>>>     phy: miphy28lp: Pass sysconfig register offsets via syscfg dt
>>>>       property.
>>>>     ARM: DT: STi: STiH407: Add DT node for MiPHY28lp
>>>>     ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe &
>>>>       USB3) PHY
>>>>
>>>>    .../devicetree/bindings/phy/phy-miphy28lp.txt      | 43 ++++++---------
>>>>    arch/arm/boot/dts/stih407-family.dtsi              | 53 +++++++++++++++++++
>>>>    arch/arm/boot/dts/stihxxx-b2120.dtsi               | 11 ++++
>>>>    arch/arm/configs/multi_v7_defconfig                |  1 +
>>>>    drivers/phy/phy-miphy28lp.c                        | 61
>>>> ++++++++++++----------
>>>>    5 files changed, 113 insertions(+), 56 deletions(-)
>>>>

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

end of thread, other threads:[~2015-01-16 15:05 UTC | newest]

Thread overview: 23+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-14  9:54 [PATCH 0/3] Enable myphy28lp support Gabriel FERNANDEZ
2015-01-14  9:54 ` Gabriel FERNANDEZ
2015-01-14  9:54 ` Gabriel FERNANDEZ
2015-01-14  9:54 ` [PATCH 1/3] phy: miphy28lp: Pass sysconfig register offsets via syscfg dt property Gabriel FERNANDEZ
2015-01-14  9:54   ` Gabriel FERNANDEZ
2015-01-14  9:54   ` Gabriel FERNANDEZ
2015-01-14  9:54 ` [PATCH 2/3] ARM: DT: STi: STiH407: Add DT node for MiPHY28lp Gabriel FERNANDEZ
2015-01-14  9:54   ` Gabriel FERNANDEZ
2015-01-14  9:54 ` [PATCH 3/3] ARM: multi_v7_defconfig: Enable MiPHY28lp - ST's Generic (SATA, PCIe & USB3) PHY Gabriel FERNANDEZ
2015-01-14  9:54   ` Gabriel FERNANDEZ
2015-01-14  9:54   ` Gabriel FERNANDEZ
2015-01-16 11:56 ` [PATCH 0/3] Enable myphy28lp support Maxime Coquelin
2015-01-16 11:56   ` Maxime Coquelin
2015-01-16 11:56   ` Maxime Coquelin
2015-01-16 12:32   ` Maxime Coquelin
2015-01-16 12:32     ` Maxime Coquelin
2015-01-16 12:32     ` Maxime Coquelin
2015-01-16 13:19     ` Kishon Vijay Abraham I
2015-01-16 13:19       ` Kishon Vijay Abraham I
2015-01-16 13:19       ` Kishon Vijay Abraham I
2015-01-16 15:04       ` Maxime Coquelin
2015-01-16 15:04         ` Maxime Coquelin
2015-01-16 15:04         ` Maxime Coquelin

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.