* [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS @ 2022-03-24 9:57 Robert Marko 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko ` (4 more replies) 0 siblings, 5 replies; 15+ messages in thread From: Robert Marko @ 2022-03-24 9:57 UTC (permalink / raw) To: u-boot, sr, pali, marek.behun; +Cc: Robert Marko Update the uDPU DTS to the version that is pending upstream [1][2]. [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-1-robert.marko@sartura.hr/ [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-2-robert.marko@sartura.hr/ Signed-off-by: Robert Marko <robert.marko@sartura.hr> --- arch/arm/dts/armada-3720-uDPU.dts | 21 +++++++++++++-------- 1 file changed, 13 insertions(+), 8 deletions(-) diff --git a/arch/arm/dts/armada-3720-uDPU.dts b/arch/arm/dts/armada-3720-uDPU.dts index 95d46e8d08..1f534c0c65 100644 --- a/arch/arm/dts/armada-3720-uDPU.dts +++ b/arch/arm/dts/armada-3720-uDPU.dts @@ -99,7 +99,7 @@ pinctrl-names = "default"; pinctrl-0 = <&spi_quad_pins>; - m25p80@0 { + spi-flash@0 { compatible = "jedec,spi-nor"; reg = <0>; spi-max-frequency = <54000000>; @@ -108,10 +108,15 @@ compatible = "fixed-partitions"; #address-cells = <1>; #size-cells = <1>; - /* only bootloader is located on the SPI */ + partition@0 { - label = "uboot"; - reg = <0 0x400000>; + label = "firmware"; + reg = <0x0 0x180000>; + }; + + partition@180000 { + label = "u-boot-env"; + reg = <0x180000 0x10000>; }; }; }; @@ -148,15 +153,15 @@ scl-gpios = <&gpionb 2 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; sda-gpios = <&gpionb 3 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; - lm75@48 { + nct375@48 { status = "okay"; - compatible = "lm75"; + compatible = "ti,tmp75c"; reg = <0x48>; }; - lm75@49 { + nct375@49 { status = "okay"; - compatible = "lm75"; + compatible = "ti,tmp75c"; reg = <0x49>; }; }; -- 2.35.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko @ 2022-03-24 9:57 ` Robert Marko 2022-03-24 10:12 ` Marek Behún ` (3 more replies) 2022-03-24 9:57 ` [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking Robert Marko ` (3 subsequent siblings) 4 siblings, 4 replies; 15+ messages in thread From: Robert Marko @ 2022-03-24 9:57 UTC (permalink / raw) To: u-boot, sr, pali, marek.behun; +Cc: Robert Marko Add support for handling SFP TX disable for MVNETA in the same fashion as to what MVPP2 is doing in order to enable using SFP-s. This allows using ethernet on SFP only boards. Signed-off-by: Robert Marko <robert.marko@sartura.hr> --- Changes in v3: * Check whether the SFP node is enabled Changes in v2: * Parse the standard SFP node for TX disable GPIO instead of using a custom property drivers/net/mvneta.c | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c index 4a4268c2b2..edd818338c 100644 --- a/drivers/net/mvneta.c +++ b/drivers/net/mvneta.c @@ -286,6 +286,7 @@ struct mvneta_port { struct phy_device *phydev; #if CONFIG_IS_ENABLED(DM_GPIO) struct gpio_desc phy_reset_gpio; + struct gpio_desc sfp_tx_disable_gpio; #endif struct mii_dev *bus; }; @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) { struct eth_pdata *pdata = dev_get_plat(dev); struct mvneta_port *pp = dev_get_priv(dev); +#if CONFIG_IS_ENABLED(DM_GPIO) + struct ofnode_phandle_args sfp_args; +#endif void *blob = (void *)gd->fdt_blob; int node = dev_of_offset(dev); struct mii_dev *bus; @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) return ret; #if CONFIG_IS_ENABLED(DM_GPIO) + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); + if (!ret && ofnode_is_enabled(sfp_args.node)) + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); + gpio_request_by_name(dev, "phy-reset-gpios", 0, &pp->phy_reset_gpio, GPIOD_IS_OUT); @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) mdelay(10); dm_gpio_set_value(&pp->phy_reset_gpio, 0); } + + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); #endif return board_network_enable(bus); -- 2.35.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko @ 2022-03-24 10:12 ` Marek Behún 2022-03-24 10:59 ` Robert Marko 2022-04-01 15:34 ` Ramon Fried ` (2 subsequent siblings) 3 siblings, 1 reply; 15+ messages in thread From: Marek Behún @ 2022-03-24 10:12 UTC (permalink / raw) To: Robert Marko; +Cc: u-boot, sr, pali On Thu, 24 Mar 2022 10:57:37 +0100 Robert Marko <robert.marko@sartura.hr> wrote: > Add support for handling SFP TX disable for MVNETA in the same fashion as > to what MVPP2 is doing in order to enable using SFP-s. > > This allows using ethernet on SFP only boards. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> > --- > Changes in v3: > * Check whether the SFP node is enabled > > Changes in v2: > * Parse the standard SFP node for TX disable GPIO instead of using a > custom property > > drivers/net/mvneta.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c > index 4a4268c2b2..edd818338c 100644 > --- a/drivers/net/mvneta.c > +++ b/drivers/net/mvneta.c > @@ -286,6 +286,7 @@ struct mvneta_port { > struct phy_device *phydev; > #if CONFIG_IS_ENABLED(DM_GPIO) > struct gpio_desc phy_reset_gpio; > + struct gpio_desc sfp_tx_disable_gpio; > #endif > struct mii_dev *bus; > }; > @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) > { > struct eth_pdata *pdata = dev_get_plat(dev); > struct mvneta_port *pp = dev_get_priv(dev); > +#if CONFIG_IS_ENABLED(DM_GPIO) > + struct ofnode_phandle_args sfp_args; > +#endif > void *blob = (void *)gd->fdt_blob; > int node = dev_of_offset(dev); > struct mii_dev *bus; > @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) > return ret; > > #if CONFIG_IS_ENABLED(DM_GPIO) > + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); > + if (!ret && ofnode_is_enabled(sfp_args.node)) > + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, > + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); > + > gpio_request_by_name(dev, "phy-reset-gpios", 0, > &pp->phy_reset_gpio, GPIOD_IS_OUT); > > @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) > mdelay(10); > dm_gpio_set_value(&pp->phy_reset_gpio, 0); > } > + > + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) > + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); > #endif > > return board_network_enable(bus); Reviewed-by: Marek Behún <marek.behun@nic.cz> Robert, I am wondering if it would make sense to add driver .remove() method (and flag DM_FLAG_OS_PREPARE so that the .remove() method is called before booting kernel), and disable TX again in this method... Marek ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 10:12 ` Marek Behún @ 2022-03-24 10:59 ` Robert Marko 0 siblings, 0 replies; 15+ messages in thread From: Robert Marko @ 2022-03-24 10:59 UTC (permalink / raw) To: Marek Behún; +Cc: U-Boot-Denx, Stefan Roese, Pali Rohár On Thu, Mar 24, 2022 at 11:12 AM Marek Behún <marek.behun@nic.cz> wrote: > > On Thu, 24 Mar 2022 10:57:37 +0100 > Robert Marko <robert.marko@sartura.hr> wrote: > > > Add support for handling SFP TX disable for MVNETA in the same fashion as > > to what MVPP2 is doing in order to enable using SFP-s. > > > > This allows using ethernet on SFP only boards. > > > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> > > --- > > Changes in v3: > > * Check whether the SFP node is enabled > > > > Changes in v2: > > * Parse the standard SFP node for TX disable GPIO instead of using a > > custom property > > > > drivers/net/mvneta.c | 12 ++++++++++++ > > 1 file changed, 12 insertions(+) > > > > diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c > > index 4a4268c2b2..edd818338c 100644 > > --- a/drivers/net/mvneta.c > > +++ b/drivers/net/mvneta.c > > @@ -286,6 +286,7 @@ struct mvneta_port { > > struct phy_device *phydev; > > #if CONFIG_IS_ENABLED(DM_GPIO) > > struct gpio_desc phy_reset_gpio; > > + struct gpio_desc sfp_tx_disable_gpio; > > #endif > > struct mii_dev *bus; > > }; > > @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) > > { > > struct eth_pdata *pdata = dev_get_plat(dev); > > struct mvneta_port *pp = dev_get_priv(dev); > > +#if CONFIG_IS_ENABLED(DM_GPIO) > > + struct ofnode_phandle_args sfp_args; > > +#endif > > void *blob = (void *)gd->fdt_blob; > > int node = dev_of_offset(dev); > > struct mii_dev *bus; > > @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) > > return ret; > > > > #if CONFIG_IS_ENABLED(DM_GPIO) > > + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); > > + if (!ret && ofnode_is_enabled(sfp_args.node)) > > + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, > > + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); > > + > > gpio_request_by_name(dev, "phy-reset-gpios", 0, > > &pp->phy_reset_gpio, GPIOD_IS_OUT); > > > > @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) > > mdelay(10); > > dm_gpio_set_value(&pp->phy_reset_gpio, 0); > > } > > + > > + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) > > + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); > > #endif > > > > return board_network_enable(bus); > > Reviewed-by: Marek Behún <marek.behun@nic.cz> > > Robert, I am wondering if it would make sense to add driver .remove() > method (and flag DM_FLAG_OS_PREPARE so that the .remove() method is > called before booting kernel), and disable TX again in this method... Hi, It could be added as a follow up improvement, however the kernels SFP driver will disable the TX as soon as it probes anyway and wait until the interface its attached to is up to enable it again. Regards, Robert > > Marek -- Robert Marko Staff Embedded Linux Engineer Sartura Ltd. Lendavska ulica 16a 10000 Zagreb, Croatia Email: robert.marko@sartura.hr Web: www.sartura.hr ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko 2022-03-24 10:12 ` Marek Behún @ 2022-04-01 15:34 ` Ramon Fried 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 3 siblings, 0 replies; 15+ messages in thread From: Ramon Fried @ 2022-04-01 15:34 UTC (permalink / raw) To: Robert Marko Cc: U-Boot Mailing List, Stefan Roese, Pali Rohár, Marek Behun On Thu, Mar 24, 2022 at 11:58 AM Robert Marko <robert.marko@sartura.hr> wrote: > > Add support for handling SFP TX disable for MVNETA in the same fashion as > to what MVPP2 is doing in order to enable using SFP-s. > > This allows using ethernet on SFP only boards. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> > --- > Changes in v3: > * Check whether the SFP node is enabled > > Changes in v2: > * Parse the standard SFP node for TX disable GPIO instead of using a > custom property > > drivers/net/mvneta.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c > index 4a4268c2b2..edd818338c 100644 > --- a/drivers/net/mvneta.c > +++ b/drivers/net/mvneta.c > @@ -286,6 +286,7 @@ struct mvneta_port { > struct phy_device *phydev; > #if CONFIG_IS_ENABLED(DM_GPIO) > struct gpio_desc phy_reset_gpio; > + struct gpio_desc sfp_tx_disable_gpio; > #endif > struct mii_dev *bus; > }; > @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) > { > struct eth_pdata *pdata = dev_get_plat(dev); > struct mvneta_port *pp = dev_get_priv(dev); > +#if CONFIG_IS_ENABLED(DM_GPIO) > + struct ofnode_phandle_args sfp_args; > +#endif > void *blob = (void *)gd->fdt_blob; > int node = dev_of_offset(dev); > struct mii_dev *bus; > @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) > return ret; > > #if CONFIG_IS_ENABLED(DM_GPIO) > + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); > + if (!ret && ofnode_is_enabled(sfp_args.node)) > + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, > + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); > + > gpio_request_by_name(dev, "phy-reset-gpios", 0, > &pp->phy_reset_gpio, GPIOD_IS_OUT); > > @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) > mdelay(10); > dm_gpio_set_value(&pp->phy_reset_gpio, 0); > } > + > + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) > + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); > #endif > > return board_network_enable(bus); > -- > 2.35.1 > Reviewed-by: Ramon Fried <rfried.dev@gmail.com> ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko 2022-03-24 10:12 ` Marek Behún 2022-04-01 15:34 ` Ramon Fried @ 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 3 siblings, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-07 6:58 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Add support for handling SFP TX disable for MVNETA in the same fashion as > to what MVPP2 is doing in order to enable using SFP-s. > > This allows using ethernet on SFP only boards. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > Changes in v3: > * Check whether the SFP node is enabled > > Changes in v2: > * Parse the standard SFP node for TX disable GPIO instead of using a > custom property > > drivers/net/mvneta.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c > index 4a4268c2b2..edd818338c 100644 > --- a/drivers/net/mvneta.c > +++ b/drivers/net/mvneta.c > @@ -286,6 +286,7 @@ struct mvneta_port { > struct phy_device *phydev; > #if CONFIG_IS_ENABLED(DM_GPIO) > struct gpio_desc phy_reset_gpio; > + struct gpio_desc sfp_tx_disable_gpio; > #endif > struct mii_dev *bus; > }; > @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) > { > struct eth_pdata *pdata = dev_get_plat(dev); > struct mvneta_port *pp = dev_get_priv(dev); > +#if CONFIG_IS_ENABLED(DM_GPIO) > + struct ofnode_phandle_args sfp_args; > +#endif > void *blob = (void *)gd->fdt_blob; > int node = dev_of_offset(dev); > struct mii_dev *bus; > @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) > return ret; > > #if CONFIG_IS_ENABLED(DM_GPIO) > + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); > + if (!ret && ofnode_is_enabled(sfp_args.node)) > + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, > + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); > + > gpio_request_by_name(dev, "phy-reset-gpios", 0, > &pp->phy_reset_gpio, GPIOD_IS_OUT); > > @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) > mdelay(10); > dm_gpio_set_value(&pp->phy_reset_gpio, 0); > } > + > + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) > + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); > #endif > > return board_network_enable(bus); Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 2/4] net: mvneta: add SFP TX disable handling 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko ` (2 preceding siblings ...) 2022-04-07 6:58 ` Stefan Roese @ 2022-04-21 14:05 ` Stefan Roese 3 siblings, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-21 14:05 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Add support for handling SFP TX disable for MVNETA in the same fashion as > to what MVPP2 is doing in order to enable using SFP-s. > > This allows using ethernet on SFP only boards. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Applied to u-boot-marvell/master Thanks, Stefan > --- > Changes in v3: > * Check whether the SFP node is enabled > > Changes in v2: > * Parse the standard SFP node for TX disable GPIO instead of using a > custom property > > drivers/net/mvneta.c | 12 ++++++++++++ > 1 file changed, 12 insertions(+) > > diff --git a/drivers/net/mvneta.c b/drivers/net/mvneta.c > index 4a4268c2b2..edd818338c 100644 > --- a/drivers/net/mvneta.c > +++ b/drivers/net/mvneta.c > @@ -286,6 +286,7 @@ struct mvneta_port { > struct phy_device *phydev; > #if CONFIG_IS_ENABLED(DM_GPIO) > struct gpio_desc phy_reset_gpio; > + struct gpio_desc sfp_tx_disable_gpio; > #endif > struct mii_dev *bus; > }; > @@ -1693,6 +1694,9 @@ static int mvneta_probe(struct udevice *dev) > { > struct eth_pdata *pdata = dev_get_plat(dev); > struct mvneta_port *pp = dev_get_priv(dev); > +#if CONFIG_IS_ENABLED(DM_GPIO) > + struct ofnode_phandle_args sfp_args; > +#endif > void *blob = (void *)gd->fdt_blob; > int node = dev_of_offset(dev); > struct mii_dev *bus; > @@ -1767,6 +1771,11 @@ static int mvneta_probe(struct udevice *dev) > return ret; > > #if CONFIG_IS_ENABLED(DM_GPIO) > + ret = dev_read_phandle_with_args(dev, "sfp", NULL, 0, 0, &sfp_args); > + if (!ret && ofnode_is_enabled(sfp_args.node)) > + gpio_request_by_name_nodev(sfp_args.node, "tx-disable-gpio", 0, > + &pp->sfp_tx_disable_gpio, GPIOD_IS_OUT); > + > gpio_request_by_name(dev, "phy-reset-gpios", 0, > &pp->phy_reset_gpio, GPIOD_IS_OUT); > > @@ -1775,6 +1784,9 @@ static int mvneta_probe(struct udevice *dev) > mdelay(10); > dm_gpio_set_value(&pp->phy_reset_gpio, 0); > } > + > + if (dm_gpio_is_valid(&pp->sfp_tx_disable_gpio)) > + dm_gpio_set_value(&pp->sfp_tx_disable_gpio, 0); > #endif > > return board_network_enable(bus); Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko @ 2022-03-24 9:57 ` Robert Marko 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 2022-03-24 9:57 ` [PATCH v3 4/4] mvebu: uDPU: update defconfig Robert Marko ` (2 subsequent siblings) 4 siblings, 2 replies; 15+ messages in thread From: Robert Marko @ 2022-03-24 9:57 UTC (permalink / raw) To: u-boot, sr, pali, marek.behun; +Cc: Robert Marko uDPU is a bit of a specific device in that it does not have any copper ports nor any ethernet PHY-s but 2 SFP ports. This is an issue since MVNETA requires a PHY phandle or a fixed-link to be defined under its node. Since U-boot has no SFP support this is reasonable in order to know how to configure the MAC. However this also means that networking does not work on uDPU at all currently, and fails with: uDPU>> dhcp Could not get PHY for neta@30000: addr 0 phy_connect failed Could not get PHY for neta@40000: addr 1 phy_connect failed So, to provide working networking using only SFP-s let add the fixed-link at 1G which is much more common than 2.5G SFP-s as well as disable the TX_DISABLE pins like done on Armada 7040 and 8040 platforms. Since uDPU is not using any of the GPIO-s on the SB controller for any purpose other than GPIO, a call to the pinctrl must be made in order for it to get probed and thus register the SB GPIO bank, otherwise SB GPIO-s are not registered at all. Signed-off-by: Robert Marko <robert.marko@sartura.hr> --- Changes in v2: * Drop the custom SFP TX GPIO properties as they are parsed from the SFP nodes now arch/arm/dts/armada-3720-uDPU-u-boot.dtsi | 24 +++++++++++++++++++++++ 1 file changed, 24 insertions(+) diff --git a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi index cf8ae4412b..47d87d4bd8 100644 --- a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi +++ b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi @@ -31,3 +31,27 @@ &sdhci0 { u-boot,dm-pre-reloc; }; + +&pinctrl_sb { + sfp_pin: sfp-pin { + groups = "pcie1_clkreq"; + function = "gpio"; + }; +}; + +ð0 { + pinctrl-names = "default"; + pinctrl-0 = <&sfp_pin>; + + fixed-link { + speed = <1000>; + full-duplex; + }; +}; + +ð1 { + fixed-link { + speed = <1000>; + full-duplex; + }; +}; -- 2.35.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking 2022-03-24 9:57 ` [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking Robert Marko @ 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 1 sibling, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-07 6:58 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > uDPU is a bit of a specific device in that it does not have any copper > ports nor any ethernet PHY-s but 2 SFP ports. > > This is an issue since MVNETA requires a PHY phandle or a fixed-link to > be defined under its node. > Since U-boot has no SFP support this is reasonable in order to know how > to configure the MAC. > > However this also means that networking does not work on uDPU at all > currently, and fails with: > > uDPU>> dhcp > Could not get PHY for neta@30000: addr 0 > phy_connect failed > Could not get PHY for neta@40000: addr 1 > phy_connect failed > > So, to provide working networking using only SFP-s let add the fixed-link > at 1G which is much more common than 2.5G SFP-s as well as disable the > TX_DISABLE pins like done on Armada 7040 and 8040 platforms. > > Since uDPU is not using any of the GPIO-s on the SB controller for any > purpose other than GPIO, a call to the pinctrl must be made in order for > it to get probed and thus register the SB GPIO bank, otherwise SB GPIO-s > are not registered at all. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > Changes in v2: > * Drop the custom SFP TX GPIO properties as they are parsed from the SFP > nodes now > > arch/arm/dts/armada-3720-uDPU-u-boot.dtsi | 24 +++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > index cf8ae4412b..47d87d4bd8 100644 > --- a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > +++ b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > @@ -31,3 +31,27 @@ > &sdhci0 { > u-boot,dm-pre-reloc; > }; > + > +&pinctrl_sb { > + sfp_pin: sfp-pin { > + groups = "pcie1_clkreq"; > + function = "gpio"; > + }; > +}; > + > +ð0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&sfp_pin>; > + > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > +}; > + > +ð1 { > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > +}; Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking 2022-03-24 9:57 ` [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking Robert Marko 2022-04-07 6:58 ` Stefan Roese @ 2022-04-21 14:05 ` Stefan Roese 1 sibling, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-21 14:05 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > uDPU is a bit of a specific device in that it does not have any copper > ports nor any ethernet PHY-s but 2 SFP ports. > > This is an issue since MVNETA requires a PHY phandle or a fixed-link to > be defined under its node. > Since U-boot has no SFP support this is reasonable in order to know how > to configure the MAC. > > However this also means that networking does not work on uDPU at all > currently, and fails with: > > uDPU>> dhcp > Could not get PHY for neta@30000: addr 0 > phy_connect failed > Could not get PHY for neta@40000: addr 1 > phy_connect failed > > So, to provide working networking using only SFP-s let add the fixed-link > at 1G which is much more common than 2.5G SFP-s as well as disable the > TX_DISABLE pins like done on Armada 7040 and 8040 platforms. > > Since uDPU is not using any of the GPIO-s on the SB controller for any > purpose other than GPIO, a call to the pinctrl must be made in order for > it to get probed and thus register the SB GPIO bank, otherwise SB GPIO-s > are not registered at all. > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Applied to u-boot-marvell/master Thanks, Stefan > --- > Changes in v2: > * Drop the custom SFP TX GPIO properties as they are parsed from the SFP > nodes now > > arch/arm/dts/armada-3720-uDPU-u-boot.dtsi | 24 +++++++++++++++++++++++ > 1 file changed, 24 insertions(+) > > diff --git a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > index cf8ae4412b..47d87d4bd8 100644 > --- a/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > +++ b/arch/arm/dts/armada-3720-uDPU-u-boot.dtsi > @@ -31,3 +31,27 @@ > &sdhci0 { > u-boot,dm-pre-reloc; > }; > + > +&pinctrl_sb { > + sfp_pin: sfp-pin { > + groups = "pcie1_clkreq"; > + function = "gpio"; > + }; > +}; > + > +ð0 { > + pinctrl-names = "default"; > + pinctrl-0 = <&sfp_pin>; > + > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > +}; > + > +ð1 { > + fixed-link { > + speed = <1000>; > + full-duplex; > + }; > +}; Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* [PATCH v3 4/4] mvebu: uDPU: update defconfig 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko 2022-03-24 9:57 ` [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking Robert Marko @ 2022-03-24 9:57 ` Robert Marko 2022-04-07 6:59 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 2022-04-07 6:57 ` [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Stefan Roese 2022-04-21 14:04 ` Stefan Roese 4 siblings, 2 replies; 15+ messages in thread From: Robert Marko @ 2022-03-24 9:57 UTC (permalink / raw) To: u-boot, sr, pali, marek.behun; +Cc: Robert Marko Update the current uDPU defconfig with following changes: * Disable CONFIG_SPI_BOOT, its not needed for booting and the device boots from eMMC anyway. * Disable CONFIG_SYS_CONSOLE_INFO_QUIET, there is no need to diverge from other boards by not priting the console device * Enable CONFIG_CMD_MTD in order to allow use of the MTD tool * Disable CONFIG_CMD_MTDPARTS, with MTD now being able to parse partitions from DTS there is no need for it, the default MTDPARTS were incorrect anyway * Enable CONFIG_MMC_HS200_SUPPORT, the eMMC used support both HS200 and HS400 modes, so enable at least HS200 because Xenon driver does not support HS400 currently * Replace CONFIG_SPI_FLASH_BAR with CONFIG_SPI_FLASH_SFDP_SUPPORT Utilize SFDP parsing instead of relying on the extended address registers Signed-off-by: Robert Marko <robert.marko@sartura.hr> --- configs/uDPU_defconfig | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig index b5f2115e63..1a2182b823 100644 --- a/configs/uDPU_defconfig +++ b/configs/uDPU_defconfig @@ -16,9 +16,8 @@ CONFIG_DISTRO_DEFAULTS=y # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set CONFIG_SYS_LOAD_ADDR=0x6000000 CONFIG_FIT=y -CONFIG_SPI_BOOT=y +CONFIG_FIT_VERBOSE=y CONFIG_USE_PREBOOT=y -CONFIG_SYS_CONSOLE_INFO_QUIET=y # CONFIG_DISPLAY_CPUINFO is not set # CONFIG_DISPLAY_BOARDINFO is not set CONFIG_DISPLAY_BOARDINFO_LATE=y @@ -32,6 +31,7 @@ CONFIG_SYS_PROMPT="uDPU>> " CONFIG_CMD_GPIO=y CONFIG_CMD_I2C=y CONFIG_CMD_MMC=y +CONFIG_CMD_MTD=y CONFIG_CMD_PCI=y CONFIG_CMD_SPI=y CONFIG_CMD_USB=y @@ -41,9 +41,6 @@ CONFIG_CMD_CACHE=y CONFIG_CMD_TIME=y CONFIG_CMD_MVEBU_BUBT=y CONFIG_CMD_EXT4_WRITE=y -CONFIG_CMD_MTDPARTS=y -CONFIG_MTDIDS_DEFAULT="nor0=spi0" -CONFIG_MTDPARTS_DEFAULT="mtdparts=spi0:4m(uboot),-(rootfs)" CONFIG_MAC_PARTITION=y CONFIG_ENV_OVERWRITE=y CONFIG_ENV_IS_IN_SPI_FLASH=y @@ -56,12 +53,13 @@ CONFIG_DM_I2C=y CONFIG_DM_I2C_GPIO=y CONFIG_SYS_I2C_MV=y CONFIG_MISC=y +CONFIG_MMC_HS200_SUPPORT=y CONFIG_MMC_SDHCI=y CONFIG_MMC_SDHCI_SDMA=y CONFIG_MMC_SDHCI_XENON=y CONFIG_MTD=y CONFIG_DM_MTD=y -CONFIG_SPI_FLASH_BAR=y +CONFIG_SPI_FLASH_SFDP_SUPPORT=y CONFIG_SPI_FLASH_MACRONIX=y CONFIG_SPI_FLASH_SPANSION=y CONFIG_SPI_FLASH_STMICRO=y -- 2.35.1 ^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [PATCH v3 4/4] mvebu: uDPU: update defconfig 2022-03-24 9:57 ` [PATCH v3 4/4] mvebu: uDPU: update defconfig Robert Marko @ 2022-04-07 6:59 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 1 sibling, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-07 6:59 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Update the current uDPU defconfig with following changes: > * Disable CONFIG_SPI_BOOT, its not needed for booting and the device boots > from eMMC anyway. > * Disable CONFIG_SYS_CONSOLE_INFO_QUIET, there is no need to diverge from > other boards by not priting the console device > * Enable CONFIG_CMD_MTD in order to allow use of the MTD tool > * Disable CONFIG_CMD_MTDPARTS, with MTD now being able to parse partitions > from DTS there is no need for it, the default MTDPARTS were incorrect > anyway > * Enable CONFIG_MMC_HS200_SUPPORT, the eMMC used support both HS200 and > HS400 modes, so enable at least HS200 because Xenon driver does not > support HS400 currently > * Replace CONFIG_SPI_FLASH_BAR with CONFIG_SPI_FLASH_SFDP_SUPPORT > Utilize SFDP parsing instead of relying on the extended address registers > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > configs/uDPU_defconfig | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig > index b5f2115e63..1a2182b823 100644 > --- a/configs/uDPU_defconfig > +++ b/configs/uDPU_defconfig > @@ -16,9 +16,8 @@ CONFIG_DISTRO_DEFAULTS=y > # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set > CONFIG_SYS_LOAD_ADDR=0x6000000 > CONFIG_FIT=y > -CONFIG_SPI_BOOT=y > +CONFIG_FIT_VERBOSE=y > CONFIG_USE_PREBOOT=y > -CONFIG_SYS_CONSOLE_INFO_QUIET=y > # CONFIG_DISPLAY_CPUINFO is not set > # CONFIG_DISPLAY_BOARDINFO is not set > CONFIG_DISPLAY_BOARDINFO_LATE=y > @@ -32,6 +31,7 @@ CONFIG_SYS_PROMPT="uDPU>> " > CONFIG_CMD_GPIO=y > CONFIG_CMD_I2C=y > CONFIG_CMD_MMC=y > +CONFIG_CMD_MTD=y > CONFIG_CMD_PCI=y > CONFIG_CMD_SPI=y > CONFIG_CMD_USB=y > @@ -41,9 +41,6 @@ CONFIG_CMD_CACHE=y > CONFIG_CMD_TIME=y > CONFIG_CMD_MVEBU_BUBT=y > CONFIG_CMD_EXT4_WRITE=y > -CONFIG_CMD_MTDPARTS=y > -CONFIG_MTDIDS_DEFAULT="nor0=spi0" > -CONFIG_MTDPARTS_DEFAULT="mtdparts=spi0:4m(uboot),-(rootfs)" > CONFIG_MAC_PARTITION=y > CONFIG_ENV_OVERWRITE=y > CONFIG_ENV_IS_IN_SPI_FLASH=y > @@ -56,12 +53,13 @@ CONFIG_DM_I2C=y > CONFIG_DM_I2C_GPIO=y > CONFIG_SYS_I2C_MV=y > CONFIG_MISC=y > +CONFIG_MMC_HS200_SUPPORT=y > CONFIG_MMC_SDHCI=y > CONFIG_MMC_SDHCI_SDMA=y > CONFIG_MMC_SDHCI_XENON=y > CONFIG_MTD=y > CONFIG_DM_MTD=y > -CONFIG_SPI_FLASH_BAR=y > +CONFIG_SPI_FLASH_SFDP_SUPPORT=y > CONFIG_SPI_FLASH_MACRONIX=y > CONFIG_SPI_FLASH_SPANSION=y > CONFIG_SPI_FLASH_STMICRO=y Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 4/4] mvebu: uDPU: update defconfig 2022-03-24 9:57 ` [PATCH v3 4/4] mvebu: uDPU: update defconfig Robert Marko 2022-04-07 6:59 ` Stefan Roese @ 2022-04-21 14:05 ` Stefan Roese 1 sibling, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-21 14:05 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Update the current uDPU defconfig with following changes: > * Disable CONFIG_SPI_BOOT, its not needed for booting and the device boots > from eMMC anyway. > * Disable CONFIG_SYS_CONSOLE_INFO_QUIET, there is no need to diverge from > other boards by not priting the console device > * Enable CONFIG_CMD_MTD in order to allow use of the MTD tool > * Disable CONFIG_CMD_MTDPARTS, with MTD now being able to parse partitions > from DTS there is no need for it, the default MTDPARTS were incorrect > anyway > * Enable CONFIG_MMC_HS200_SUPPORT, the eMMC used support both HS200 and > HS400 modes, so enable at least HS200 because Xenon driver does not > support HS400 currently > * Replace CONFIG_SPI_FLASH_BAR with CONFIG_SPI_FLASH_SFDP_SUPPORT > Utilize SFDP parsing instead of relying on the extended address registers > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Applied to u-boot-marvell/master Thanks, Stefan > --- > configs/uDPU_defconfig | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/configs/uDPU_defconfig b/configs/uDPU_defconfig > index b5f2115e63..1a2182b823 100644 > --- a/configs/uDPU_defconfig > +++ b/configs/uDPU_defconfig > @@ -16,9 +16,8 @@ CONFIG_DISTRO_DEFAULTS=y > # CONFIG_SYS_MALLOC_CLEAR_ON_INIT is not set > CONFIG_SYS_LOAD_ADDR=0x6000000 > CONFIG_FIT=y > -CONFIG_SPI_BOOT=y > +CONFIG_FIT_VERBOSE=y > CONFIG_USE_PREBOOT=y > -CONFIG_SYS_CONSOLE_INFO_QUIET=y > # CONFIG_DISPLAY_CPUINFO is not set > # CONFIG_DISPLAY_BOARDINFO is not set > CONFIG_DISPLAY_BOARDINFO_LATE=y > @@ -32,6 +31,7 @@ CONFIG_SYS_PROMPT="uDPU>> " > CONFIG_CMD_GPIO=y > CONFIG_CMD_I2C=y > CONFIG_CMD_MMC=y > +CONFIG_CMD_MTD=y > CONFIG_CMD_PCI=y > CONFIG_CMD_SPI=y > CONFIG_CMD_USB=y > @@ -41,9 +41,6 @@ CONFIG_CMD_CACHE=y > CONFIG_CMD_TIME=y > CONFIG_CMD_MVEBU_BUBT=y > CONFIG_CMD_EXT4_WRITE=y > -CONFIG_CMD_MTDPARTS=y > -CONFIG_MTDIDS_DEFAULT="nor0=spi0" > -CONFIG_MTDPARTS_DEFAULT="mtdparts=spi0:4m(uboot),-(rootfs)" > CONFIG_MAC_PARTITION=y > CONFIG_ENV_OVERWRITE=y > CONFIG_ENV_IS_IN_SPI_FLASH=y > @@ -56,12 +53,13 @@ CONFIG_DM_I2C=y > CONFIG_DM_I2C_GPIO=y > CONFIG_SYS_I2C_MV=y > CONFIG_MISC=y > +CONFIG_MMC_HS200_SUPPORT=y > CONFIG_MMC_SDHCI=y > CONFIG_MMC_SDHCI_SDMA=y > CONFIG_MMC_SDHCI_XENON=y > CONFIG_MTD=y > CONFIG_DM_MTD=y > -CONFIG_SPI_FLASH_BAR=y > +CONFIG_SPI_FLASH_SFDP_SUPPORT=y > CONFIG_SPI_FLASH_MACRONIX=y > CONFIG_SPI_FLASH_SPANSION=y > CONFIG_SPI_FLASH_STMICRO=y Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko ` (2 preceding siblings ...) 2022-03-24 9:57 ` [PATCH v3 4/4] mvebu: uDPU: update defconfig Robert Marko @ 2022-04-07 6:57 ` Stefan Roese 2022-04-21 14:04 ` Stefan Roese 4 siblings, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-07 6:57 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Update the uDPU DTS to the version that is pending upstream [1][2]. > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-1-robert.marko@sartura.hr/ > [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-2-robert.marko@sartura.hr/ > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Reviewed-by: Stefan Roese <sr@denx.de> Thanks, Stefan > --- > arch/arm/dts/armada-3720-uDPU.dts | 21 +++++++++++++-------- > 1 file changed, 13 insertions(+), 8 deletions(-) > > diff --git a/arch/arm/dts/armada-3720-uDPU.dts b/arch/arm/dts/armada-3720-uDPU.dts > index 95d46e8d08..1f534c0c65 100644 > --- a/arch/arm/dts/armada-3720-uDPU.dts > +++ b/arch/arm/dts/armada-3720-uDPU.dts > @@ -99,7 +99,7 @@ > pinctrl-names = "default"; > pinctrl-0 = <&spi_quad_pins>; > > - m25p80@0 { > + spi-flash@0 { > compatible = "jedec,spi-nor"; > reg = <0>; > spi-max-frequency = <54000000>; > @@ -108,10 +108,15 @@ > compatible = "fixed-partitions"; > #address-cells = <1>; > #size-cells = <1>; > - /* only bootloader is located on the SPI */ > + > partition@0 { > - label = "uboot"; > - reg = <0 0x400000>; > + label = "firmware"; > + reg = <0x0 0x180000>; > + }; > + > + partition@180000 { > + label = "u-boot-env"; > + reg = <0x180000 0x10000>; > }; > }; > }; > @@ -148,15 +153,15 @@ > scl-gpios = <&gpionb 2 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > sda-gpios = <&gpionb 3 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > > - lm75@48 { > + nct375@48 { > status = "okay"; > - compatible = "lm75"; > + compatible = "ti,tmp75c"; > reg = <0x48>; > }; > > - lm75@49 { > + nct375@49 { > status = "okay"; > - compatible = "lm75"; > + compatible = "ti,tmp75c"; > reg = <0x49>; > }; > }; Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko ` (3 preceding siblings ...) 2022-04-07 6:57 ` [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Stefan Roese @ 2022-04-21 14:04 ` Stefan Roese 4 siblings, 0 replies; 15+ messages in thread From: Stefan Roese @ 2022-04-21 14:04 UTC (permalink / raw) To: Robert Marko, u-boot, pali, marek.behun On 3/24/22 10:57, Robert Marko wrote: > Update the uDPU DTS to the version that is pending upstream [1][2]. > > [1] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-1-robert.marko@sartura.hr/ > [2] https://patchwork.kernel.org/project/linux-arm-kernel/patch/20220322105857.1107016-2-robert.marko@sartura.hr/ > > Signed-off-by: Robert Marko <robert.marko@sartura.hr> Applied to u-boot-marvell/master Thanks, Stefan > --- > arch/arm/dts/armada-3720-uDPU.dts | 21 +++++++++++++-------- > 1 file changed, 13 insertions(+), 8 deletions(-) > > diff --git a/arch/arm/dts/armada-3720-uDPU.dts b/arch/arm/dts/armada-3720-uDPU.dts > index 95d46e8d08..1f534c0c65 100644 > --- a/arch/arm/dts/armada-3720-uDPU.dts > +++ b/arch/arm/dts/armada-3720-uDPU.dts > @@ -99,7 +99,7 @@ > pinctrl-names = "default"; > pinctrl-0 = <&spi_quad_pins>; > > - m25p80@0 { > + spi-flash@0 { > compatible = "jedec,spi-nor"; > reg = <0>; > spi-max-frequency = <54000000>; > @@ -108,10 +108,15 @@ > compatible = "fixed-partitions"; > #address-cells = <1>; > #size-cells = <1>; > - /* only bootloader is located on the SPI */ > + > partition@0 { > - label = "uboot"; > - reg = <0 0x400000>; > + label = "firmware"; > + reg = <0x0 0x180000>; > + }; > + > + partition@180000 { > + label = "u-boot-env"; > + reg = <0x180000 0x10000>; > }; > }; > }; > @@ -148,15 +153,15 @@ > scl-gpios = <&gpionb 2 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > sda-gpios = <&gpionb 3 (GPIO_ACTIVE_HIGH | GPIO_OPEN_DRAIN)>; > > - lm75@48 { > + nct375@48 { > status = "okay"; > - compatible = "lm75"; > + compatible = "ti,tmp75c"; > reg = <0x48>; > }; > > - lm75@49 { > + nct375@49 { > status = "okay"; > - compatible = "lm75"; > + compatible = "ti,tmp75c"; > reg = <0x49>; > }; > }; Viele Grüße, Stefan Roese -- DENX Software Engineering GmbH, Managing Director: Wolfgang Denk HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: sr@denx.de ^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2022-04-21 14:06 UTC | newest] Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2022-03-24 9:57 [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Robert Marko 2022-03-24 9:57 ` [PATCH v3 2/4] net: mvneta: add SFP TX disable handling Robert Marko 2022-03-24 10:12 ` Marek Behún 2022-03-24 10:59 ` Robert Marko 2022-04-01 15:34 ` Ramon Fried 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 2022-03-24 9:57 ` [PATCH v3 3/4] arm: mvebu: dts: uDPU: fix non-working networking Robert Marko 2022-04-07 6:58 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 2022-03-24 9:57 ` [PATCH v3 4/4] mvebu: uDPU: update defconfig Robert Marko 2022-04-07 6:59 ` Stefan Roese 2022-04-21 14:05 ` Stefan Roese 2022-04-07 6:57 ` [PATCH v3 1/4] arm: mvebu: dts: uDPU: update DTS Stefan Roese 2022-04-21 14:04 ` Stefan Roese
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.