On 22/08/23 17:43, Roger Quadros wrote:
> Some boards (e.g. Beagleplay) need multiple attempts to detect the PHY
> and the multiple "PHY not found" prints are not nice.
I tried grepping for calls to "phy_connect" across the drivers present in
drivers/net. Most of them simply return -ENODEV on
On 23/08/23 13:22, Roger Quadros wrote:
> Hi Siddharth,
>
> On 23/08/2023 07:35, Siddharth Vadapalli wrote:
>> Roger,
>>
>> On 22/08/23 17:43, Roger Quadros wrote:
>>> Beagleplay has a buggy Ethernet PHY implementation for the Gigabit
>>> PHY in
Roger,
On 22/08/23 17:43, Roger Quadros wrote:
> Beagleplay has a buggy Ethernet PHY implementation for the Gigabit
> PHY in the sense that it is non responsive over MDIO immediately
> after power-up/reset.
>
> We need to either try multiple times or wait sufficiently long enough
> (couple of
Hello Roger,
On 22-08-2023 17:43, Roger Quadros wrote:
> Beagleplay has a buggy Ethernet PHY implementation for the Gigabit
> PHY in the sense that it is non responsive over MDIO immediately
> after power-up/reset.
>
> We need to either try multiple times or wait sufficiently long enough
>
Hello Tom,
On 11/08/23 21:45, Tom Rini wrote:
> On Fri, Aug 11, 2023 at 10:49:23AM +0530, Siddharth Vadapalli wrote:
>> Ravi,
>>
>> On 10/08/23 17:00, Ravi Gunasekaran wrote:
>>> Siddharth,
>>>
>>> On 8/10/23 2:45 PM, Siddharth Vadapalli wrote:
e the documentation for printf format
specifiers, changing the format specifier from "%d" to "%u" for the
"unsigned int" variable, as reported by Tom Rini.
Siddharth Vadapalli (2):
net: Fix the displayed value of bytes transferred
doc: printf() codes: Fix format
ed.
Fixes: 1411157d8578 ("net: cosmetic: Fixup var names related to boot file")
Signed-off-by: Siddharth Vadapalli
---
net/net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/net.c b/net/net.c
index 43abbac7c3..e6f61f0f8f 100644
--- a/net/net.c
+++
The format specifier for the "unsigned int" variable is documented as
"%d". However, it should be "%u". Thus, fix it.
Fixes: f5e9035043fb ("doc: printf() codes")
Reported-by: Tom Rini
Signed-off-by: Siddharth Vadapalli
---
doc/develop/printf.rst |
On 14/08/23 10:06, Siddharth Vadapalli wrote:
> Hello Tom,
>
> On 11/08/23 21:45, Tom Rini wrote:
>> On Fri, Aug 11, 2023 at 10:49:23AM +0530, Siddharth Vadapalli wrote:
>>> Ravi,
>>>
>>> On 10/08/23 17:00, Ravi Gunasekaran wrote:
...
>>
>
Roger,
On 25/08/23 00:39, Roger Quadros wrote:
>
>
> On 24/08/2023 21:24, Tom Rini wrote:
>> On Thu, Aug 24, 2023 at 11:34:29PM +0530, Siddharth Vadapalli wrote:
>>> Hello Roger,
>>>
>>> On 22-08-2023 17:43, Roger Quadros wrote:
...
>
> Even a
On 25/08/23 13:22, Roger Quadros wrote:
> Siddharth,
>
> On 25/08/2023 08:42, Siddharth Vadapalli wrote:
>> Roger,
>>
>> On 25/08/23 00:39, Roger Quadros wrote:
>>>
>>>
>>> On 24/08/2023 21:24, Tom Rini wrote:
>>>> O
Hari,
On 21/03/23 19:40, Hari Nagalla wrote:
> Introduce the base dts files needed for u-boot or to augment the linux
> dtbs for use in the u-boot-spl and u-boot binaries.
>
> Signed-off-by: Apurva Nandan
> Signed-off-by: Hari Nagalla
> ---
> arch/arm/dts/Makefile|
Hello Tom,
On 10/04/23 19:00, Tom Rini wrote:
> On Mon, Apr 10, 2023 at 11:40:15AM +0530, Siddharth Vadapalli wrote:
>
>> From: Kishon Vijay Abraham I
>>
>> Add support to detect daughtercards (GESI Ethernet card) in-order
>> to set the MAC address of the main CPSW
From: Kishon Vijay Abraham I
Add support to detect daughtercards (GESI Ethernet card) in-order
to set the MAC address of the main CPSW2G interface.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
board/ti/j721s2/evm.c | 130
From: Kishon Vijay Abraham I
Add configs for enabling ETHBOOT in R5SPL. Adding a separate config
minimizes the risk of going past the R5-SPL size limit for any future
config additions.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Andreas Dannenberg
Signed-off-by: Siddharth Vadapalli
From: Kishon Vijay Abraham I
Update the default BOOTCOMMAND to provide an automatic and easier way
to configure ethernet PHY before loading the firmware.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
configs/j7200_evm_a72_defconfig | 2 +-
1 file changed, 1
Hello,
Kindly ignore this patch.
On 11-04-2023 15:03, Siddharth Vadapalli wrote:
> From: Kishon Vijay Abraham I
>
> Add configs for enabling ETHBOOT in R5SPL. Adding a separate config
> minimizes the risk of going past the R5-SPL size limit for any future
> config additions.
Hello Hari,
On 21/03/23 19:40, Hari Nagalla wrote:
> Introduce the basic j784s4 SoC dtbs from the linux kernel along with
> the new j784s4 specific pinmux definitions that we will use to generate
> the dtbs for the u-boot-spl and u-boot binaries.
>
> Signed-off-by: Apurva Nandan
>
Hello Ulf,
On 07/02/23 13:55, Ulf Samuelsson wrote:
> cpsw_mdio_get_alive reads the wrong register.
> See page 2316 in SPRUH73Q AM335x TRM
>
> Signed-off-by: Ulf Samuelsson
> Cc: Joe Hershberger
> Cc: Ramon Fried
> ---
> drivers/net/ti/cpsw_mdio.c | 6 +++---
> 1 file changed, 3
On 09/02/23 14:56, Ulf Samuelsson wrote:
>
> Den 2023-02-09 kl. 08:29, skrev Siddharth Vadapalli:
>> Hello Ulf,
>>
>> On 07/02/23 13:55, Ulf Samuelsson wrote:
>>> cpsw_mdio_get_alive reads the wrong register.
>>> See page 2316 in SPRUH73Q AM335x
the different
> ports are spaced exactly 0x4 bytes apart.
>
> Fixes: 9d0dca1199d1 ("net: ethernet: ti: Introduce am654 gigabit eth switch
> subsystem driver")
> Signed-off-by: Andreas Dannenberg
> Reviewed-by: Siddharth Vadapalli
> ---
> Driver changes bench-te
EVM, powering on the Ethernet PHY and
resetting it requires GPIO Hogging capability. Enable it as well.
Signed-off-by: Siddharth Vadapalli
---
configs/j7200_evm_a72_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/j7200_evm_a72_defconfig b/configs/j7200_evm_a72_defconfig
kernel. The
reset is done once when the CPSW MDIO bus is being initialized.
Signed-off-by: Suman Anna
Signed-off-by: Siddharth Vadapalli
---
drivers/net/ti/am65-cpsw-nuss.c | 34 -
1 file changed, 33 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ti/am65-cpsw
Enable full rate divider configuration support for J721E_WIZ_16G for SGMII.
Signed-off-by: Siddharth Vadapalli
---
drivers/phy/ti/phy-j721e-wiz.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/phy/ti/phy-j721e-wiz.c b/drivers/phy/ti/phy-j721e-wiz.c
index 12421f90b0..ecb2f684ef
Select the same mac divider for SGMII too as the one being used for
QSGMII.
Enable full rate divider configuration support for J721E_WIZ_10G for
SGMII.
Signed-off-by: Siddharth Vadapalli
---
drivers/phy/ti/phy-j721e-wiz.c | 19 +--
1 file changed, 13 insertions(+), 6 deletions
is applicable to J7200
SoC as well, the MAIN CPSW0 instance on J7200 will also be usable now.
Signed-off-by: Suman Anna
[s-vadapa...@ti.com: Update commit message indicating support for J7200]
Signed-off-by: Siddharth Vadapalli
---
drivers/dma/ti/k3-psil-j721e.c | 17 ++---
1 file changed, 14
Add support for configuring the CPSW Ethernet Switch in SGMII mode.
Signed-off-by: Siddharth Vadapalli
---
drivers/net/ti/am65-cpsw-nuss.c | 23 +++
1 file changed, 23 insertions(+)
diff --git a/drivers/net/ti/am65-cpsw-nuss.c b/drivers/net/ti/am65-cpsw-nuss.c
index
: correct at91 tree link
and depends on the following patch being merged as well:
Link:
https://patchwork.ozlabs.org/project/uboot/patch/20230614222853.574427-1-dannenb...@ti.com/
Regards,
Siddharth.
Siddharth Vadapalli (4):
net: ti: am65-cpsw-nuss: Add support for SGMII mode
phy: ti: phy-j721e-wiz
On 14/07/23 01:06, Nishanth Menon wrote:
> Instead of hard-coding the count of entries manually, use ARRAY_SIZE
> to keep the count updates appropriately.
>
> Cc: Bryan Brattlof
> Suggested-by: Ravi Gunasekaran
> Signed-off-by: Nishanth Menon
Reviewed-by: S
Tom,
On 22/07/23 01:06, Tom Rini wrote:
> On Sat, Jul 08, 2023 at 04:15:20PM +0530, Siddharth Vadapalli wrote:
>
>> From: Suman Anna
>>
>> Enhance the AM65 CPSW NUSS driver to perform a MDIO reset using a GPIO
>> line. Logic is also added to perform a pre and po
On 31/07/23 10:13, Siddharth Vadapalli wrote:
> Tom,
>
> On 22/07/23 01:06, Tom Rini wrote:
>> On Sat, Jul 08, 2023 at 04:15:20PM +0530, Siddharth Vadapalli wrote:
>>
>>> From: Suman Anna
>>>
>>> Enhance the AM65 CPSW NUSS driver to perform a
> Tree does.
>
> Let's rework the driver a bit to create a dummy MDIO driver that we will
> then get during our initialization to force the core to select the right
> muxing.
>
> Signed-off-by: Maxime Ripard
LGTM.
Reviewed-by: Siddharth Vadapalli
> ---
> drivers/net/ti/K
Ravi,
On 10/08/23 17:00, Ravi Gunasekaran wrote:
> Siddharth,
>
> On 8/10/23 2:45 PM, Siddharth Vadapalli wrote:
>> In the case of NETLOOP_SUCCESS, the decimal value of the u32 variable
>> "net_boot_file_size" is printed using "%d", resulting in negativ
ed.
Fixes: 1411157d8578 ("net: cosmetic: Fixup var names related to boot file")
Signed-off-by: Siddharth Vadapalli
---
net/net.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/net/net.c b/net/net.c
index 43abbac7c3..7aaeafc247 100644
--- a/net/net.c
+++
**: This series is based on top of commit:
7755b22007 Merge tag 'x86-pull-20230801' of
https://source.denx.de/u-boot/custodians/u-boot-x86
Siddharth Vadapalli (4):
net: ti: am65-cpsw-nuss: Add support for SGMII mode
phy: ti: phy-j721e-wiz: Add SGMII support in wiz driver for J7200
phy: ti: j721e-wiz
Enable full rate divider configuration support for J721E_WIZ_16G for SGMII.
Signed-off-by: Siddharth Vadapalli
---
drivers/phy/ti/phy-j721e-wiz.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/phy/ti/phy-j721e-wiz.c b/drivers/phy/ti/phy-j721e-wiz.c
index cba87f5093..7261339907
kernel. The
reset is done once when the CPSW MDIO bus is being initialized.
Signed-off-by: Suman Anna
Signed-off-by: Siddharth Vadapalli
---
drivers/net/ti/am65-cpsw-nuss.c | 38 -
1 file changed, 37 insertions(+), 1 deletion(-)
diff --git a/drivers/net/ti/am65-cpsw
Select the same mac divider for SGMII too as the one being used for
QSGMII.
Enable full rate divider configuration support for J721E_WIZ_10G for
SGMII.
Signed-off-by: Siddharth Vadapalli
---
drivers/phy/ti/phy-j721e-wiz.c | 19 +--
1 file changed, 13 insertions(+), 6 deletions
Add support for configuring the CPSW Ethernet Switch in SGMII mode.
Signed-off-by: Siddharth Vadapalli
---
drivers/net/ti/am65-cpsw-nuss.c | 23 +++
1 file changed, 23 insertions(+)
diff --git a/drivers/net/ti/am65-cpsw-nuss.c b/drivers/net/ti/am65-cpsw-nuss.c
index
is applicable to J7200
SoC as well, the MAIN CPSW0 instance on J7200 will also be usable now.
Signed-off-by: Suman Anna
[s-vadapa...@ti.com: Update commit message indicating support for J7200]
Signed-off-by: Siddharth Vadapalli
---
drivers/dma/ti/k3-psil-j721e.c | 17 ++---
1 file changed, 14
EVM, powering on the Ethernet PHY and
resetting it requires GPIO Hogging capability. Enable it as well.
Signed-off-by: Siddharth Vadapalli
---
configs/j7200_evm_a72_defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/configs/j7200_evm_a72_defconfig b/configs/j7200_evm_a72_defconfig
es: a8837cf43839 ("dma: ti: k3-udma: Query DMA channels allocated from
Resource Manager")
Signed-off-by: Siddharth Vadapalli
---
Hello,
This patch is based on commit
9e00b6993f Merge tag 'u-boot-dfu-20240215' of
https://source.denx.de/u-boot/custodians/u-boot-dfu
of the master branch of
On 24/02/20 12:51PM, Dan Carpenter wrote:
> On Fri, Feb 16, 2024 at 04:11:05PM +0530, Siddharth Vadapalli wrote:
> > diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
> > index eea9ec9659..8a6625f034 100644
> > --- a/drivers/dma/ti/k3-udma.c
> > +++
On 24/02/20 03:24PM, Siddharth Vadapalli wrote:
> On 24/02/20 12:51PM, Dan Carpenter wrote:
> > On Fri, Feb 16, 2024 at 04:11:05PM +0530, Siddharth Vadapalli wrote:
> > > diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
> > > index eea9ec9659..8a6625f034
es: a8837cf43839 ("dma: ti: k3-udma: Query DMA channels allocated from
Resource Manager")
Signed-off-by: Siddharth Vadapalli
---
Hello,
This patch is based on commit
3e6f2a94bf Merge tag 'u-boot-imx-master-20240219' of
https://gitlab.denx.de/u-boot/custodians/u-boot-imx
of th
-by: Siddharth Vadapalli
---
Hello,
This patch is based on commit
65eed68772 test/py: Disable error E0611 in two cases for pylint
Regards,
Siddharth.
include/env/ti/ti_common.env | 5 -
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/include/env/ti/ti_common.env b/include/env/ti
From: Kishon Vijay Abraham I
RX_FL_CFG message should not be forwarded to TIFS
and should be handled within R5 SPL (when DM services
are not available). Add a no-op function to not handle
RX_FL_CFG messages.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
drivers
: Init DRAM size in R5/A53 SPL
firmware: ti_sci: Add No-OP for "RX_FL_CFG"
soc: ti: k3-navss-ringacc: Initialize base address of ring cfg
registers
dma: ti: k3-udma: Add support for native configuration of chan/flow
arm: mach-k3: am625_init: Probe AM65 CPSW NUSS
configs: am
t Ethernet boot on AM62x.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
board/ti/am62x/evm.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/board/ti/am62x/evm.c b/board/ti/am62x/evm.c
index ad93908840..35f291d83a 100644
--- a/board/ti/am62x/evm.c
+++ b/boar
Enable DM services for main_pktdma during R5 SPL stage.
Signed-off-by: Siddharth Vadapalli
---
arch/arm/dts/k3-am625-r5-sk.dts | 5 +
1 file changed, 5 insertions(+)
diff --git a/arch/arm/dts/k3-am625-r5-sk.dts b/arch/arm/dts/k3-am625-r5-sk.dts
index 6b9f40e555..0912b953db 100644
From: Andreas Dannenberg
In order to enable Ethernet Boot using CPSW, update the clock data.
Signed-off-by: Andreas Dannenberg
Signed-off-by: Siddharth Vadapalli
---
arch/arm/mach-k3/r5/am62x/clk-data.c | 79 ++--
1 file changed, 39 insertions(+), 40 deletions
From: Kishon Vijay Abraham I
Enable config options needed to support Ethernet boot on AM62x SK.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
configs/am62x_evm_a53_defconfig | 8
1 file changed, 8 insertions(+)
diff --git a/configs
From: Kishon Vijay Abraham I
Initialize base address of ring config registers required to natively
setup ring cfg registers in the absence of Device Manager (DM) services
at R5 SPL stage.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
drivers/soc/ti/k3-navss
-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
drivers/dma/ti/k3-udma.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/drivers/dma/ti/k3-udma.c b/drivers/dma/ti/k3-udma.c
index 8a62d63dfe..3bd1e7668b 100644
--- a/drivers/dma/ti/k3-udma.c
+++ b/drivers/dma/ti/k3-udma.c
From: Vignesh Raghavendra
Expectation of k3_ringacc_ring_reset_raw() is to reset the ring to
requested size and not to 0. Fix this.
Signed-off-by: Vignesh Raghavendra
Signed-off-by: Siddharth Vadapalli
---
drivers/soc/ti/k3-navss-ringacc-u-boot.c | 9 -
1 file changed, 8 insertions
From: Kishon Vijay Abraham I
Add configs for enabling ETHBOOT in R5SPL. Adding a separate config
minimizes the risk of going past the R5-SPL size limit for any future
config additions.
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Andreas Dannenberg
Signed-off-by: Siddharth Vadapalli
From: Kishon Vijay Abraham I
In order to support Ethernet boot on AM62x, probe AM65 CPSW NUSS driver
in board_init_f().
Signed-off-by: Kishon Vijay Abraham I
Signed-off-by: Siddharth Vadapalli
---
arch/arm/mach-k3/am625_init.c | 10 ++
1 file changed, 10 insertions(+)
diff --git
Hello Nishanth,
On 12/01/24 17:56, Nishanth Menon wrote:
> On 12:17-20240112, Siddharth Vadapalli wrote:
>> From: Kishon Vijay Abraham I
>>
>> Call dram_init_banksize() from spl_board_init() otherwise TFTP download
>> fails with error "TFTP error: trying to over
On 12/01/24 18:01, Nishanth Menon wrote:
> On 12:17-20240112, Siddharth Vadapalli wrote:
>> From: Kishon Vijay Abraham I
>>
>> Add configs for enabling ETHBOOT in R5SPL. Adding a separate config
>> minimizes the risk of going past the R5-SPL size limit for any
On 12/01/24 18:12, Nishanth Menon wrote:
> On 18:06-20240112, Siddharth Vadapalli wrote:
>>
>>
>> On 12/01/24 18:02, Nishanth Menon wrote:
>>> On 12:17-20240112, Siddharth Vadapalli wrote:
>>>> Hello,
>>>>
>>>> This series
On 12/01/24 18:03, Nishanth Menon wrote:
> On 12:17-20240112, Siddharth Vadapalli wrote:
>> From: Kishon Vijay Abraham I
>>
>> Enable config options needed to support Ethernet boot on AM62x SK.
>>
>> Signed-off-by: Kishon Vijay Abraham I
>
On 12/01/24 18:02, Nishanth Menon wrote:
> On 12:17-20240112, Siddharth Vadapalli wrote:
>> Hello,
>>
>> This series enables Ethernet Boot on SK-AM62 device.
>> Product Link: https://www.ti.com/tool/SK-AM62
>> User Guide: https://www.ti.com/lit/pdf/spruj40
>&
Hello Tom,
On 12/01/24 18:56, Tom Rini wrote:
> On Fri, Jan 12, 2024 at 12:17:50PM +0530, Siddharth Vadapalli wrote:
>
>> From: Kishon Vijay Abraham I
>>
>> Call dram_init_banksize() from spl_board_init() otherwise TFTP download
>> fails with error "TFTP
On 12/01/24 18:31, Nishanth Menon wrote:
> On 18:17-20240112, Siddharth Vadapalli wrote:
>>
>>
>> On 12/01/24 18:12, Nishanth Menon wrote:
>>> On 18:06-20240112, Siddharth Vadapalli wrote:
>>>>
>>>>
>>>> On 12/01/24 18:02, Nishan
On 20/01/24 22:11, Tom Rini wrote:
> On Mon, Jan 15, 2024 at 01:42:51PM +0530, Siddharth Vadapalli wrote:
>> Hello Tom,
>>
>> On 12/01/24 18:56, Tom Rini wrote:
...
>>> The list of conditionals in common/spl/spl.c::board_init_r() should be
>>> updated
abit eth switch
> subsystem driver")
> Signed-off-by: Alexander Sverdlin
Reviewed-by: Siddharth Vadapalli
Regards,
Siddharth.
to emphasize this instead.
Something like:
net: ti: am65-cpsw-nuss: Update am65_cpsw_phy_init() to handle PHY absence
The rest of the patch looks good to me. With the subject reworded,
Reviewed-by: Siddharth Vadapalli
Regards,
Siddharth.
>
> am65_cpsw_ofdata_parse_phy() tries to handl
On Fri, Apr 26, 2024 at 10:02:24AM +0200, Matthias Schiffer wrote:
> With netconsole, any log message can result in an eth_init(), possibly
> causing an reentrant call into eth_init() if a driver's ops print
> anything:
>
> eth_init() -> driver.start() -> printf() -> netconsole -> eth_init()
68 matches
Mail list logo