* [PATCH v2 net-next 1/2] net: ethernet: mediatek: Add MT7621 TRGMII mode support
2019-06-20 12:21 [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support René van Dorst
@ 2019-06-20 12:21 ` René van Dorst
2019-06-20 12:21 ` [PATCH v2 net-next 2/2] net: dsa: mt7530: " René van Dorst
` (2 subsequent siblings)
3 siblings, 0 replies; 7+ messages in thread
From: René van Dorst @ 2019-06-20 12:21 UTC (permalink / raw)
To: frank-w, sean.wang, f.fainelli, davem, matthias.bgg, andrew,
vivien.didelot
Cc: netdev, john, linux-mediatek, linux-mips, René van Dorst
MT7621 SOC also supports TRGMII.
TRGMII speed is 1200MBit.
Signed-off-by: René van Dorst <opensource@vdorst.com>
---
drivers/net/ethernet/mediatek/mtk_eth_soc.c | 38 ++++++++++++++++++---
drivers/net/ethernet/mediatek/mtk_eth_soc.h | 11 ++++++
2 files changed, 45 insertions(+), 4 deletions(-)
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.c b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
index f27efe4110cc..066712f2e985 100644
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.c
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.c
@@ -134,6 +134,28 @@ static int mtk_mdio_read(struct mii_bus *bus, int phy_addr, int phy_reg)
return _mtk_mdio_read(eth, phy_addr, phy_reg);
}
+static int mt7621_gmac0_rgmii_adjust(struct mtk_eth *eth,
+ phy_interface_t interface)
+{
+ u32 val;
+
+ /* Check DDR memory type. Currently DDR2 is not supported. */
+ regmap_read(eth->ethsys, ETHSYS_SYSCFG, &val);
+ if (val & SYSCFG_DRAM_TYPE_DDR2) {
+ dev_err(eth->dev,
+ "TRGMII mode with DDR2 memory is not supported!\n");
+ return -EOPNOTSUPP;
+ }
+
+ val = (interface == PHY_INTERFACE_MODE_TRGMII) ?
+ ETHSYS_TRGMII_MT7621_DDR_PLL : 0;
+
+ regmap_update_bits(eth->ethsys, ETHSYS_CLKCFG0,
+ ETHSYS_TRGMII_MT7621_MASK, val);
+
+ return 0;
+}
+
static void mtk_gmac0_rgmii_adjust(struct mtk_eth *eth, int speed)
{
u32 val;
@@ -183,9 +205,17 @@ static void mtk_phy_link_adjust(struct net_device *dev)
break;
}
- if (MTK_HAS_CAPS(mac->hw->soc->caps, MTK_GMAC1_TRGMII) &&
- !mac->id && !mac->trgmii)
- mtk_gmac0_rgmii_adjust(mac->hw, dev->phydev->speed);
+ if (MTK_HAS_CAPS(mac->hw->soc->caps, MTK_GMAC1_TRGMII) && !mac->id) {
+ if (MTK_HAS_CAPS(mac->hw->soc->caps, MTK_TRGMII_MT7621_CLK)) {
+ if (mt7621_gmac0_rgmii_adjust(mac->hw,
+ dev->phydev->interface))
+ return;
+ } else {
+ if (!mac->trgmii)
+ mtk_gmac0_rgmii_adjust(mac->hw,
+ dev->phydev->speed);
+ }
+ }
if (dev->phydev->link)
mcr |= MAC_MCR_FORCE_LINK;
@@ -2607,7 +2637,7 @@ static const struct mtk_soc_data mt2701_data = {
};
static const struct mtk_soc_data mt7621_data = {
- .caps = MTK_SHARED_INT,
+ .caps = MT7621_CAPS,
.required_clks = MT7621_CLKS_BITMAP,
.required_pctl = false,
};
diff --git a/drivers/net/ethernet/mediatek/mtk_eth_soc.h b/drivers/net/ethernet/mediatek/mtk_eth_soc.h
index 85e3144f1af5..876ce6798709 100644
--- a/drivers/net/ethernet/mediatek/mtk_eth_soc.h
+++ b/drivers/net/ethernet/mediatek/mtk_eth_soc.h
@@ -363,6 +363,10 @@
#define MT7622_ETH 7622
#define MT7621_ETH 7621
+/* ethernet system control register */
+#define ETHSYS_SYSCFG 0x10
+#define SYSCFG_DRAM_TYPE_DDR2 BIT(4)
+
/* ethernet subsystem config register */
#define ETHSYS_SYSCFG0 0x14
#define SYSCFG0_GE_MASK 0x3
@@ -377,6 +381,9 @@
/* ethernet subsystem clock register */
#define ETHSYS_CLKCFG0 0x2c
#define ETHSYS_TRGMII_CLK_SEL362_5 BIT(11)
+#define ETHSYS_TRGMII_MT7621_MASK (BIT(5) | BIT(6))
+#define ETHSYS_TRGMII_MT7621_APLL BIT(6)
+#define ETHSYS_TRGMII_MT7621_DDR_PLL BIT(5)
/* ethernet reset control register */
#define ETHSYS_RSTCTRL 0x34
@@ -616,6 +623,7 @@ enum mtk_eth_path {
#define MTK_SHARED_SGMII BIT(7)
#define MTK_HWLRO BIT(8)
#define MTK_SHARED_INT BIT(9)
+#define MTK_TRGMII_MT7621_CLK BIT(10)
/* Supported path present on SoCs */
#define MTK_PATH_BIT(x) BIT((x) + 10)
@@ -667,6 +675,9 @@ enum mtk_eth_path {
#define MTK_HAS_CAPS(caps, _x) (((caps) & (_x)) == (_x))
+#define MT7621_CAPS (MTK_GMAC1_RGMII | MTK_GMAC1_TRGMII | \
+ MTK_GMAC2_RGMII | MTK_SHARED_INT | MTK_TRGMII_MT7621_CLK)
+
#define MT7622_CAPS (MTK_GMAC1_RGMII | MTK_GMAC1_SGMII | MTK_GMAC2_RGMII | \
MTK_GMAC2_SGMII | MTK_GDM1_ESW | \
MTK_MUX_GDM1_TO_GMAC1_ESW | \
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH v2 net-next 2/2] net: dsa: mt7530: Add MT7621 TRGMII mode support
2019-06-20 12:21 [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support René van Dorst
2019-06-20 12:21 ` [PATCH v2 net-next 1/2] net: ethernet: " René van Dorst
@ 2019-06-20 12:21 ` René van Dorst
2019-06-20 17:32 ` Frank Wunderlich
2019-06-20 13:02 ` [PATCH v2 net-next 0/2] net: mediatek: " Frank Wunderlich
2019-06-22 23:58 ` David Miller
3 siblings, 1 reply; 7+ messages in thread
From: René van Dorst @ 2019-06-20 12:21 UTC (permalink / raw)
To: frank-w, sean.wang, f.fainelli, davem, matthias.bgg, andrew,
vivien.didelot
Cc: netdev, john, linux-mediatek, linux-mips, René van Dorst
This patch add support TRGMII mode for MT7621 internal MT7530 switch.
MT7621 TRGMII has only one fix speed mode of 1200MBit.
Also adding support for mt7530 25MHz and 40MHz crystal clocksource.
Values are based on Banana Pi R2 bsp [1].
Don't change MT7623 registers on a MT7621 device.
[1] https://github.com/BPI-SINOVOIP/BPI-R2-bsp/blob/master/linux-mt/drivers/net/ethernet/mediatek/gsw_mt7623.c#L769
Signed-off-by: René van Dorst <opensource@vdorst.com>
---
drivers/net/dsa/mt7530.c | 46 +++++++++++++++++++++++++++++++---------
drivers/net/dsa/mt7530.h | 4 ++++
2 files changed, 40 insertions(+), 10 deletions(-)
diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c
index c7d352da5448..3181e95586d6 100644
--- a/drivers/net/dsa/mt7530.c
+++ b/drivers/net/dsa/mt7530.c
@@ -428,24 +428,48 @@ static int
mt7530_pad_clk_setup(struct dsa_switch *ds, int mode)
{
struct mt7530_priv *priv = ds->priv;
- u32 ncpo1, ssc_delta, trgint, i;
+ u32 ncpo1, ssc_delta, trgint, i, xtal;
+
+ xtal = mt7530_read(priv, MT7530_MHWTRAP) & HWTRAP_XTAL_MASK;
+
+ if (xtal == HWTRAP_XTAL_20MHZ) {
+ dev_err(priv->dev,
+ "%s: MT7530 with a 20MHz XTAL is not supported!\n",
+ __func__);
+ return -EINVAL;
+ }
switch (mode) {
case PHY_INTERFACE_MODE_RGMII:
trgint = 0;
+ /* PLL frequency: 125MHz */
ncpo1 = 0x0c80;
- ssc_delta = 0x87;
break;
case PHY_INTERFACE_MODE_TRGMII:
trgint = 1;
- ncpo1 = 0x1400;
- ssc_delta = 0x57;
+ if (priv->id == ID_MT7621) {
+ /* PLL frequency: 150MHz: 1.2GBit */
+ if (xtal == HWTRAP_XTAL_40MHZ)
+ ncpo1 = 0x0780;
+ if (xtal == HWTRAP_XTAL_25MHZ)
+ ncpo1 = 0x0a00;
+ } else { /* PLL frequency: 250MHz: 2.0Gbit */
+ if (xtal == HWTRAP_XTAL_40MHZ)
+ ncpo1 = 0x0c80;
+ if (xtal == HWTRAP_XTAL_25MHZ)
+ ncpo1 = 0x1400;
+ }
break;
default:
dev_err(priv->dev, "xMII mode %d not supported\n", mode);
return -EINVAL;
}
+ if (xtal == HWTRAP_XTAL_25MHZ)
+ ssc_delta = 0x57;
+ else
+ ssc_delta = 0x87;
+
mt7530_rmw(priv, MT7530_P6ECR, P6_INTF_MODE_MASK,
P6_INTF_MODE(trgint));
@@ -507,7 +531,9 @@ mt7530_pad_clk_setup(struct dsa_switch *ds, int mode)
mt7530_rmw(priv, MT7530_TRGMII_RD(i),
RD_TAP_MASK, RD_TAP(16));
else
- mt7623_trgmii_set(priv, GSW_INTF_MODE, INTF_MODE_TRGMII);
+ if (priv->id != ID_MT7621)
+ mt7623_trgmii_set(priv, GSW_INTF_MODE,
+ INTF_MODE_TRGMII);
return 0;
}
@@ -613,13 +639,13 @@ static void mt7530_adjust_link(struct dsa_switch *ds, int port,
struct mt7530_priv *priv = ds->priv;
if (phy_is_pseudo_fixed_link(phydev)) {
- if (priv->id == ID_MT7530) {
- dev_dbg(priv->dev, "phy-mode for master device = %x\n",
- phydev->interface);
+ dev_dbg(priv->dev, "phy-mode for master device = %x\n",
+ phydev->interface);
- /* Setup TX circuit incluing relevant PAD and driving */
- mt7530_pad_clk_setup(ds, phydev->interface);
+ /* Setup TX circuit incluing relevant PAD and driving */
+ mt7530_pad_clk_setup(ds, phydev->interface);
+ if (priv->id == ID_MT7530) {
/* Setup RX circuit, relevant PAD and driving on the
* host which must be placed after the setup on the
* device side is all finished.
diff --git a/drivers/net/dsa/mt7530.h b/drivers/net/dsa/mt7530.h
index 4331429969fa..bfac90f48102 100644
--- a/drivers/net/dsa/mt7530.h
+++ b/drivers/net/dsa/mt7530.h
@@ -244,6 +244,10 @@ enum mt7530_vlan_port_attr {
/* Register for hw trap status */
#define MT7530_HWTRAP 0x7800
+#define HWTRAP_XTAL_MASK (BIT(10) | BIT(9))
+#define HWTRAP_XTAL_25MHZ (BIT(10) | BIT(9))
+#define HWTRAP_XTAL_40MHZ (BIT(10))
+#define HWTRAP_XTAL_20MHZ (BIT(9))
/* Register for hw trap modification */
#define MT7530_MHWTRAP 0x7804
--
2.20.1
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2 net-next 2/2] net: dsa: mt7530: Add MT7621 TRGMII mode support
2019-06-20 12:21 ` [PATCH v2 net-next 2/2] net: dsa: mt7530: " René van Dorst
@ 2019-06-20 17:32 ` Frank Wunderlich
0 siblings, 0 replies; 7+ messages in thread
From: Frank Wunderlich @ 2019-06-20 17:32 UTC (permalink / raw)
To: René van Dorst, sean.wang, f.fainelli, davem, matthias.bgg,
andrew, vivien.didelot
Cc: netdev, john, linux-mediatek, linux-mips
Tested on Bananapi R2 (mt7623) with 5.2-rc5 + net-next
Tested-by: Frank Wunderlich <frank-w@public-files.de>
Am 20. Juni 2019 14:21:55 MESZ schrieb "René van Dorst" <opensource@vdorst.com>:
>This patch add support TRGMII mode for MT7621 internal MT7530 switch.
>MT7621 TRGMII has only one fix speed mode of 1200MBit.
>
>Also adding support for mt7530 25MHz and 40MHz crystal clocksource.
>Values are based on Banana Pi R2 bsp [1].
>
>Don't change MT7623 registers on a MT7621 device.
>
>[1]
>https://github.com/BPI-SINOVOIP/BPI-R2-bsp/blob/master/linux-mt/drivers/net/ethernet/mediatek/gsw_mt7623.c#L769
>
>Signed-off-by: René van Dorst <opensource@vdorst.com>
>---
> drivers/net/dsa/mt7530.c | 46 +++++++++++++++++++++++++++++++---------
> drivers/net/dsa/mt7530.h | 4 ++++
> 2 files changed, 40 insertions(+), 10 deletions(-)
>
>diff --git a/drivers/net/dsa/mt7530.c b/drivers/net/dsa/mt7530.c
>index c7d352da5448..3181e95586d6 100644
>--- a/drivers/net/dsa/mt7530.c
>+++ b/drivers/net/dsa/mt7530.c
>@@ -428,24 +428,48 @@ static int
> mt7530_pad_clk_setup(struct dsa_switch *ds, int mode)
> {
> struct mt7530_priv *priv = ds->priv;
>- u32 ncpo1, ssc_delta, trgint, i;
>+ u32 ncpo1, ssc_delta, trgint, i, xtal;
>+
>+ xtal = mt7530_read(priv, MT7530_MHWTRAP) & HWTRAP_XTAL_MASK;
>+
>+ if (xtal == HWTRAP_XTAL_20MHZ) {
>+ dev_err(priv->dev,
>+ "%s: MT7530 with a 20MHz XTAL is not supported!\n",
>+ __func__);
>+ return -EINVAL;
>+ }
>
> switch (mode) {
> case PHY_INTERFACE_MODE_RGMII:
> trgint = 0;
>+ /* PLL frequency: 125MHz */
> ncpo1 = 0x0c80;
>- ssc_delta = 0x87;
> break;
> case PHY_INTERFACE_MODE_TRGMII:
> trgint = 1;
>- ncpo1 = 0x1400;
>- ssc_delta = 0x57;
>+ if (priv->id == ID_MT7621) {
>+ /* PLL frequency: 150MHz: 1.2GBit */
>+ if (xtal == HWTRAP_XTAL_40MHZ)
>+ ncpo1 = 0x0780;
>+ if (xtal == HWTRAP_XTAL_25MHZ)
>+ ncpo1 = 0x0a00;
>+ } else { /* PLL frequency: 250MHz: 2.0Gbit */
>+ if (xtal == HWTRAP_XTAL_40MHZ)
>+ ncpo1 = 0x0c80;
>+ if (xtal == HWTRAP_XTAL_25MHZ)
>+ ncpo1 = 0x1400;
>+ }
> break;
> default:
> dev_err(priv->dev, "xMII mode %d not supported\n", mode);
> return -EINVAL;
> }
>
>+ if (xtal == HWTRAP_XTAL_25MHZ)
>+ ssc_delta = 0x57;
>+ else
>+ ssc_delta = 0x87;
>+
> mt7530_rmw(priv, MT7530_P6ECR, P6_INTF_MODE_MASK,
> P6_INTF_MODE(trgint));
>
>@@ -507,7 +531,9 @@ mt7530_pad_clk_setup(struct dsa_switch *ds, int
>mode)
> mt7530_rmw(priv, MT7530_TRGMII_RD(i),
> RD_TAP_MASK, RD_TAP(16));
> else
>- mt7623_trgmii_set(priv, GSW_INTF_MODE, INTF_MODE_TRGMII);
>+ if (priv->id != ID_MT7621)
>+ mt7623_trgmii_set(priv, GSW_INTF_MODE,
>+ INTF_MODE_TRGMII);
>
> return 0;
> }
>@@ -613,13 +639,13 @@ static void mt7530_adjust_link(struct dsa_switch
>*ds, int port,
> struct mt7530_priv *priv = ds->priv;
>
> if (phy_is_pseudo_fixed_link(phydev)) {
>- if (priv->id == ID_MT7530) {
>- dev_dbg(priv->dev, "phy-mode for master device = %x\n",
>- phydev->interface);
>+ dev_dbg(priv->dev, "phy-mode for master device = %x\n",
>+ phydev->interface);
>
>- /* Setup TX circuit incluing relevant PAD and driving */
>- mt7530_pad_clk_setup(ds, phydev->interface);
>+ /* Setup TX circuit incluing relevant PAD and driving */
>+ mt7530_pad_clk_setup(ds, phydev->interface);
>
>+ if (priv->id == ID_MT7530) {
> /* Setup RX circuit, relevant PAD and driving on the
> * host which must be placed after the setup on the
> * device side is all finished.
>diff --git a/drivers/net/dsa/mt7530.h b/drivers/net/dsa/mt7530.h
>index 4331429969fa..bfac90f48102 100644
>--- a/drivers/net/dsa/mt7530.h
>+++ b/drivers/net/dsa/mt7530.h
>@@ -244,6 +244,10 @@ enum mt7530_vlan_port_attr {
>
> /* Register for hw trap status */
> #define MT7530_HWTRAP 0x7800
>+#define HWTRAP_XTAL_MASK (BIT(10) | BIT(9))
>+#define HWTRAP_XTAL_25MHZ (BIT(10) | BIT(9))
>+#define HWTRAP_XTAL_40MHZ (BIT(10))
>+#define HWTRAP_XTAL_20MHZ (BIT(9))
>
> /* Register for hw trap modification */
> #define MT7530_MHWTRAP 0x7804
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support
2019-06-20 12:21 [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support René van Dorst
2019-06-20 12:21 ` [PATCH v2 net-next 1/2] net: ethernet: " René van Dorst
2019-06-20 12:21 ` [PATCH v2 net-next 2/2] net: dsa: mt7530: " René van Dorst
@ 2019-06-20 13:02 ` Frank Wunderlich
2019-06-21 3:55 ` Sean Wang
2019-06-22 23:58 ` David Miller
3 siblings, 1 reply; 7+ messages in thread
From: Frank Wunderlich @ 2019-06-20 13:02 UTC (permalink / raw)
To: René van Dorst, sean.wang, f.fainelli, davem, matthias.bgg,
andrew, vivien.didelot
Cc: netdev, john, linux-mediatek, linux-mips
Tested on Bananapi R2 (mt7623)
Tested-by: "Frank Wunderlich" <frank-w@public-files.de>
Am 20. Juni 2019 14:21:53 MESZ schrieb "René van Dorst" <opensource@vdorst.com>:
>Like many other mediatek SOCs, the MT7621 SOC and the internal MT7530
>switch both supports TRGMII mode. MT7621 TRGMII speed is fix 1200MBit.
>
>v1->v2:
> - Fix breakage on non MT7621 SOC
> - Support 25MHz and 40MHz XTAL as MT7530 clocksource
>
>René van Dorst (2):
> net: ethernet: mediatek: Add MT7621 TRGMII mode support
> net: dsa: mt7530: Add MT7621 TRGMII mode support
>
> drivers/net/dsa/mt7530.c | 46 ++++++++++++++++-----
> drivers/net/dsa/mt7530.h | 4 ++
> drivers/net/ethernet/mediatek/mtk_eth_soc.c | 38 +++++++++++++++--
> drivers/net/ethernet/mediatek/mtk_eth_soc.h | 11 +++++
> 4 files changed, 85 insertions(+), 14 deletions(-)
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support
2019-06-20 13:02 ` [PATCH v2 net-next 0/2] net: mediatek: " Frank Wunderlich
@ 2019-06-21 3:55 ` Sean Wang
0 siblings, 0 replies; 7+ messages in thread
From: Sean Wang @ 2019-06-21 3:55 UTC (permalink / raw)
To: Frank Wunderlich
Cc: René van Dorst, Sean Wang (王志亘),
Florian Fainelli, David S. Miller, Matthias Brugger, Andrew Lunn,
vivien.didelot, netdev, moderated list:ARM/Mediatek SoC support,
linux-mips, John Crispin
On Thu, Jun 20, 2019 at 6:02 AM Frank Wunderlich
<frank-w@public-files.de> wrote:
>
> Tested on Bananapi R2 (mt7623)
>
> Tested-by: "Frank Wunderlich" <frank-w@public-files.de>
These changes also look good to me, thanks for add the patch to enrich
different application variants.
Acked-by: Sean Wang <sean.wang@kernel.org>
>
> Am 20. Juni 2019 14:21:53 MESZ schrieb "René van Dorst" <opensource@vdorst.com>:
> >Like many other mediatek SOCs, the MT7621 SOC and the internal MT7530
> >switch both supports TRGMII mode. MT7621 TRGMII speed is fix 1200MBit.
> >
> >v1->v2:
> > - Fix breakage on non MT7621 SOC
> > - Support 25MHz and 40MHz XTAL as MT7530 clocksource
> >
> >René van Dorst (2):
> > net: ethernet: mediatek: Add MT7621 TRGMII mode support
> > net: dsa: mt7530: Add MT7621 TRGMII mode support
> >
> > drivers/net/dsa/mt7530.c | 46 ++++++++++++++++-----
> > drivers/net/dsa/mt7530.h | 4 ++
> > drivers/net/ethernet/mediatek/mtk_eth_soc.c | 38 +++++++++++++++--
> > drivers/net/ethernet/mediatek/mtk_eth_soc.h | 11 +++++
> > 4 files changed, 85 insertions(+), 14 deletions(-)
>
> _______________________________________________
> Linux-mediatek mailing list
> Linux-mediatek@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-mediatek
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support
2019-06-20 12:21 [PATCH v2 net-next 0/2] net: mediatek: Add MT7621 TRGMII mode support René van Dorst
` (2 preceding siblings ...)
2019-06-20 13:02 ` [PATCH v2 net-next 0/2] net: mediatek: " Frank Wunderlich
@ 2019-06-22 23:58 ` David Miller
3 siblings, 0 replies; 7+ messages in thread
From: David Miller @ 2019-06-22 23:58 UTC (permalink / raw)
To: opensource
Cc: frank-w, sean.wang, f.fainelli, matthias.bgg, andrew,
vivien.didelot, netdev, john, linux-mediatek, linux-mips
From: René van Dorst <opensource@vdorst.com>
Date: Thu, 20 Jun 2019 14:21:53 +0200
> Like many other mediatek SOCs, the MT7621 SOC and the internal MT7530
> switch both supports TRGMII mode. MT7621 TRGMII speed is fix 1200MBit.
>
> v1->v2:
> - Fix breakage on non MT7621 SOC
> - Support 25MHz and 40MHz XTAL as MT7530 clocksource
Series applied, thanks.
^ permalink raw reply [flat|nested] 7+ messages in thread