Re: [PATCH v4 0/6] Enable ICSSG Ethernet Driver for AM65x

2024-04-25 Thread Ravi Gunasekaran
igs/am65x_evm_a53_defconfig | 2 + > 6 files changed, 221 insertions(+), 8 deletions(-) > create mode 100644 arch/arm/dts/k3-am654-icssg2.dtso > > > base-commit: 6e316e3f397b5e01e98c5dd56cdbaab961daeedf For the series, Acked-by: Ravi Gunasekaran Regards, Ravi

[PATCH v3] usb: cdns3: gadget.c: Set fast access bit

2024-04-24 Thread Ravi Gunasekaran
reference. Signed-off-by: Aswath Govindraju Signed-off-by: Ravi Gunasekaran Reviewed-by: Roger Quadros Reviewed-by: Marek Vasut --- Changes since v2: * Updated the commit description with reference to upstream kernel commit * Picked up Marek's Reviewed-by tag from v2 Change

[PATCH v2] usb: cdns3: gadget.c: Set fast access bit

2024-04-23 Thread Ravi Gunasekaran
register access bit ensures that the PHY clock is keeping up in active state. Therefore, set fast access bit to ensure the accesses to device registers are quick even in low power states. Signed-off-by: Aswath Govindraju Signed-off-by: Ravi Gunasekaran Reviewed-by: Roger Quadros --- v1

Re: [PATCH v5 4/5] net: ti: icssg: Add ICSSG ethernet driver

2024-04-05 Thread Ravi Gunasekaran
e | 1 + > drivers/net/ti/icssg_prueth.c | 685 ++ > drivers/net/ti/icssg_prueth.h | 3 + > 5 files changed, 713 insertions(+) > create mode 100644 drivers/net/ti/icssg_prueth.c > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v5 2/5] net: ti: icssg: Add Firmware config and classification APIs.

2024-04-05 Thread Ravi Gunasekaran
lassifier.c > create mode 100644 drivers/net/ti/icssg_config.c > create mode 100644 drivers/net/ti/icssg_config.h > create mode 100644 drivers/net/ti/icssg_prueth.h > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v4 5/5] net: ti: icssg: Add support sending FDB command to update rx_flow_id

2024-04-03 Thread Ravi Gunasekaran
MD Danish Anwar > --- > drivers/net/ti/icssg_config.c | 68 +++ > drivers/net/ti/icssg_config.h | 18 ++ > drivers/net/ti/icssg_prueth.c | 6 > drivers/net/ti/icssg_prueth.h | 6 > 4 files changed, 98 insertions(+) > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v4 4/5] net: ti: icssg: Add ICSSG ethernet driver

2024-04-03 Thread Ravi Gunasekaran
On 3/27/24 10:58 AM, MD Danish Anwar wrote: > This is the PRUSS Ethernet driver for TI AM654 Sr2.0 and laterSoCs with s/Sr2.0/SR2.0 s/laterSoCs/later SoCs > the ICSSG PRU Sub-system running EMAC firmware. ICSSG Subsystem supports > two slices per instance. This driver caters to both slices /

Re: [PATCH v4 3/5] net: ti: icssg: Add icssg queues APIs and macros

2024-04-02 Thread Ravi Gunasekaran
> drivers/net/ti/icssg_queues.c | 51 +++ > 2 files changed, 56 insertions(+) > create mode 100644 drivers/net/ti/icssg_queues.c > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v4 2/5] net: ti: icssg: Add Firmware config and classification APIs.

2024-04-02 Thread Ravi Gunasekaran
On 3/27/24 10:58 AM, MD Danish Anwar wrote: > Add icssg_config.h / .c and icssg_classifier.c files. These are firmware > configuration and classification related files. Add MII helper APIs and > MACROs. These APIs and MACROs will be later used by ICSSG Ethernet driver. > Also introduce

Re: [PATCH v4 1/5] net: ti: icssg: Add Firmware Interface for ICSSG Ethernet driver.

2024-04-02 Thread Ravi Gunasekaran
switch_map.h | 209 ++ > 1 file changed, 209 insertions(+) > create mode 100644 drivers/net/ti/icssg_switch_map.h > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH] arm: mach-k3: Fix config check for FS_LOADER

2024-03-18 Thread Ravi Gunasekaran
ot-common.c > @@ -190,7 +190,7 @@ int load_firmware(char *name_fw, u32 *loadaddr) > struct udevice *fsdev; > int size = 0; > > - if (!IS_ENABLED(CONFIG_FS_LOADER)) > + if (!CONFIG_IS_ENABLED(FS_LOADER)) > return 0; > > if (!*loadaddr) Acked-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v2 2/2] net: am65-cpsw: cpsw_mdio: Switch to proper DM_MDIO framework

2024-03-01 Thread Ravi Gunasekaran
> drivers/net/ti/cpsw_mdio.c | 198 > +++- > drivers/net/ti/cpsw_mdio.h | 2 + > 5 files changed, 196 insertions(+), 205 deletions(-) > Tested dhcp and ping in u-boot on J721S2-EVM, SK-AM62B Tested-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v5] remoteproc: uclass: Add methods to load firmware to rproc and boot rproc

2024-02-27 Thread Ravi Gunasekaran
s/remoteproc/Kconfig > +++ b/drivers/remoteproc/Kconfig [...] > + > +free_buffer: > + free(addr); Thanks for taking care of freeing the memory. Acked-by: Ravi Gunasekaran [...] -- Regards, Ravi

Re: [PATCH 1/2] net: mdio: Handle bus level GPIO Reset

2024-02-21 Thread Ravi Gunasekaran
On 2/19/24 9:03 PM, Roger Quadros wrote: > Some platforms have bus level Reset controlled > by a GPIO line. If available then handle bus reset > via GPIO. > > Signed-off-by: Roger Quadros > --- > include/phy.h | 7 +++ > net/mdio-uclass.c | 37 - >

Re: [PATCH v4] remoteproc: uclass: Add methods to load firmware to rproc and boot rproc

2024-02-01 Thread Ravi Gunasekaran
On 1/30/2024 12:03 PM, MD Danish Anwar wrote: > Add APIs to set a firmware_name to a rproc and boot the rproc with the > same firmware. > > Clients can call rproc_set_firmware() API to set firmware_name for a rproc > whereas rproc_boot() will load the firmware set by rproc_set_firmware() to > a

Re: [PATCH v4] dma: ti: k3-udma: Use ring_idx to pair k3 nav rings

2024-02-01 Thread Ravi Gunasekaran
d; > + > + ret = k3_nav_ringacc_request_rings_pair(ud->ringacc, ring_idx, -1, > >tchan->t_ring, > >tchan->tc_ring); > if (ret) { I see this change present in upstream linux kernel as well. You could consider providing the link to kernel if such changes are already merged there. Reviewed-by: Ravi Gunasekaran

Re: [PATCH v4] misc: fs-loader: Use fw_storage_interface instead of storage_interface

2024-02-01 Thread Ravi Gunasekaran
series as suggested >by Nishant. > *) Droppped the RFC tag. > > v3: https://lore.kernel.org/all/20240124064930.1787929-3-danishan...@ti.com/ Acked-by: Ravi Gunasekaran

Re: [PATCH] include: env: ti: mmc: Add 'ti' directory prefix for overlays

2023-09-18 Thread Ravi Gunasekaran
On 9/15/23 6:12 PM, Nishanth Menon wrote: > On 15:42-20230915, Ravi Gunasekaran wrote: >> DTSOs are stored in vendor specific directories in Linux. > > what is dtso? kernel generates dtbo I'm sorry, my bad. I will be careful in future. > >> Add the vendor pr

[PATCH] include: env: ti: mmc: Add 'ti' directory prefix for overlays

2023-09-15 Thread Ravi Gunasekaran
DTSOs are stored in vendor specific directories in Linux. Add the vendor prefix to the path to load the overlay correctly. Signed-off-by: Ravi Gunasekaran --- include/env/ti/mmc.env | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/env/ti/mmc.env b/include/env/ti

[PATCH] configs: am65x_evm_r5_usbmsc_defconfig: Enable DWC3 wrapper for SPL

2023-09-08 Thread Ravi Gunasekaran
ld warnings due to SPL_USB_DWC3_GENERIC") Signed-off-by: Ravi Gunasekaran --- configs/am65x_evm_r5_usbmsc_defconfig | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configs/am65x_evm_r5_usbmsc_defconfig b/configs/am65x_evm_r5_usbmsc_defconfig index 8da49c78c8..733a1c1874 100644 --- a/configs/am65x_evm

Re: [PATCH] net: Fix the displayed value of bytes transferred

2023-08-10 Thread Ravi Gunasekaran
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 negative values > being reported for large file sizes. Fix this by using "%lu" to print > the decimal value

Re: [PATCH 0/4] net: ti: am65-cpsw-nuss: Fix DT binding handling of pinctrl

2023-07-26 Thread Ravi Gunasekaran
On 7/26/23 6:22 PM, Nishanth Menon wrote: > On 14:44-20230726, Ravi Gunasekaran wrote: > [...] >> If it is the former, then would a duplicate mdio node help keep the changes >> to minimum? > > That is worse hack. How does it make sense to have two mdio nodes to >

Re: [PATCH 0/4] net: ti: am65-cpsw-nuss: Fix DT binding handling of pinctrl

2023-07-26 Thread Ravi Gunasekaran
Maxime, On 7/26/23 6:19 PM, Maxime Ripard wrote: > Hi Ravi, > > On Wed, Jul 26, 2023 at 02:44:00PM +0530, Ravi Gunasekaran wrote: >> On 7/20/23 7:42 PM, Maxime Ripard wrote: >>> Hi >>> >>> Sorry, I didn't receive Roger mail so I'll reply here >&g

Re: [PATCH 0/4] net: ti: am65-cpsw-nuss: Fix DT binding handling of pinctrl

2023-07-26 Thread Ravi Gunasekaran
Maxime, On 7/20/23 7:42 PM, Maxime Ripard wrote: > Hi > > Sorry, I didn't receive Roger mail so I'll reply here > > On Thu, Jul 20, 2023 at 09:00:19AM -0500, Nishanth Menon wrote: >> On 16:56-20230720, Roger Quadros wrote: >>> Hi, >>> >>

Re: [PATCH 3/3] arm: dts: k3-am62: Bump dtsi from linux v6.5-rc1

2023-07-25 Thread Ravi Gunasekaran
On 7/25/23 11:47 PM, Tom Rini wrote: > On Tue, Jul 25, 2023 at 09:09:34AM -0500, Nishanth Menon wrote: >> On 15:56-20230725, Maxime Ripard wrote: >>> Hi, >>> >>> On Tue, Jul 25, 2023 at 07:58:56AM -0500, Nishanth Menon wrote: Update the am62 and am625 device-trees from linux v6.3-rc5 This

Re: [PATCH 0/4] net: ti: am65-cpsw-nuss: Fix DT binding handling of pinctrl

2023-07-20 Thread Ravi Gunasekaran
On 7/20/2023 3:25 PM, Maxime Ripard wrote: > Hi, > > This series is based on: > https://lore.kernel.org/all/20230713072019.3153871-1...@ti.com/ > > It fixes the issue of Linux booting from the DT embedded by U-boot. The > main issue there is that U-Boot doesn't handle the MDIO child node that >

Re: [PATCH] usb: cdns3: gadget: Configure speed in udc_start

2023-07-19 Thread Ravi Gunasekaran
On 7/19/23 4:03 PM, Marek Vasut wrote: > On 7/19/23 10:59, Ravi Gunasekaran wrote: >> When one of the functions does not support super speed, the composite >> driver forces the gadget to high speed. But the speed is never >> configured in the cdns3 gadget driver. S

[PATCH] usb: cdns3: gadget: Configure speed in udc_start

2023-07-19 Thread Ravi Gunasekaran
When one of the functions does not support super speed, the composite driver forces the gadget to high speed. But the speed is never configured in the cdns3 gadget driver. So configure the speed in cdns3_gadget_udc_start just like in the kernel. Signed-off-by: Ravi Gunasekaran --- drivers/usb

Re: [RFC PATCH 2/4] arm: mach-k3: am62: Add timer0 id to the dev list

2023-07-13 Thread Ravi Gunasekaran
On 7/13/23 12:50 PM, Nishanth Menon wrote: > From: Sjoerd Simons > > Timer0 is used by u-boot as the tick timer; Add it to the soc devices > list so it can be enabled via the k3 power controller. > > Signed-off-by: Sjoerd Simons > Reviewed-by: Tom Rini > Tes

[PATCH] common: dfu: Remove leading space characters

2023-05-18 Thread Ravi Gunasekaran
One option is to add dfu_alt_info_* variable in .env in single line. But there is possiblity for it to exceed the line length limit. So update the parsing logic to remove leading space characters before adding to the dfu list. [1]: https://u-boot.readthedocs.io/en/latest/usage/dfu.html Signed-of

Re: [PATCH] phy: ti: phy-j721e-wiz: Add j721s2-wiz-10g module support

2023-05-15 Thread Ravi Gunasekaran
On 15/05/23 4:20 pm, Ravi Gunasekaran wrote: > Add support for j721s2-wiz-10g device to use clock-names interface > instead of explicitly defining clock nodes within device tree node. > > Signed-off-by: Ravi Gunasekaran > --- > drivers/phy/ti/phy-j721e-wiz.c | 21

[PATCH] phy: ti: phy-j721e-wiz: Add j721s2-wiz-10g module support

2023-05-15 Thread Ravi Gunasekaran
Add support for j721s2-wiz-10g device to use clock-names interface instead of explicitly defining clock nodes within device tree node. Signed-off-by: Ravi Gunasekaran --- drivers/phy/ti/phy-j721e-wiz.c | 21 - 1 file changed, 20 insertions(+), 1 deletion(-) diff --git

Re: [EXTERNAL] Re: [PATCH] usb: cdns3: gadget.c: Set fast access bit

2023-05-11 Thread Ravi Gunasekaran
Pawel, On 11/05/23 11:00 am, Pawel Laszczak wrote: >> >> +Pawel & Peter >> >> On 09/05/2023 08:58, Ravi Gunasekaran wrote: >>> Hi Roger, >>> >>> On 05/05/23 6:02 pm, Roger Quadros wrote: >>>> Hi Ravi, >>>>

Re: [EXTERNAL] Re: [PATCH] usb: cdns3: gadget.c: Set fast access bit

2023-05-09 Thread Ravi Gunasekaran
Hi Roger, On 05/05/23 6:02 pm, Roger Quadros wrote: > Hi Ravi, > > On 05/05/2023 15:13, Ravi Gunasekaran wrote: >> From: Aswath Govindraju >> >> When the device port is in a low power state [U3/L2/Not Connected], >> accesses to usb device registers may

[PATCH] usb: cdns3: gadget.c: Set fast access bit

2023-05-05 Thread Ravi Gunasekaran
From: Aswath Govindraju When the device port is in a low power state [U3/L2/Not Connected], accesses to usb device registers may take a long time. This could lead to potential core hang when the controller registers are accessed after the port is disabled by setting DEVDS field. Setting the fast

Re: [PATCH v2 0/8] Add DFU, emmc and usb boot for TI am62x

2023-04-11 Thread Ravi Gunasekaran
> drivers/usb/dwc3/dwc3-am62.c | 127 + > include/configs/am62x_evm.h | 10 +- > 15 files changed, 701 insertions(+), 92 deletions(-) > create mode 100644 configs/am62x_evm_r5_usbdfu_defconfig > create mode 100644 drivers/usb/dwc3/dwc3-am62.c I tested USB DFU boot on AM62 and it works fine. Tested-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH v2 0/5] enable DMA and CPSW nodes for am62ax SoC family

2023-04-03 Thread Ravi Gunasekaran
il-am62a.c| 196 ++ > drivers/dma/ti/k3-psil-priv.h | 1 + > drivers/dma/ti/k3-psil.c | 2 + > 12 files changed, 727 insertions(+), 3 deletions(-) > create mode 100644 drivers/dma/ti/k3-psil-am62a.c > > > base-commit: 942ac73afc37fb98695af4489ea1549c21615a5e Acked-by: Ravi Gunasekaran -- Regards, Ravi

Re: [PATCH 3/5] arm: dts: add DMA and CPSW nodes for uboot

2023-03-31 Thread Ravi Gunasekaran
Hi Bryan, On 28/03/23 12:10 am, Bryan Brattlof wrote: > Update the uboot dtsi to enable DMA and CPSW at the uboot level > > Signed-off-by: Bryan Brattlof > --- > arch/arm/dts/k3-am62a7-r5-sk.dts | 8 > arch/arm/dts/k3-am62a7-sk-u-boot.dtsi | 26 +- > 2

Re: [EXTERNAL] [PATCH 0/2] phy: ti: j721e-wiz: Add support to manage type-C swap on Lane2 and lane3

2023-03-15 Thread Ravi Gunasekaran
phy: ti: j721e-wiz: Manage TypeC lane swap if typec-dir-gpios not > specified > phy: ti: j721e-wiz: Add support to enable LN23 Type-C swap > > drivers/phy/ti/phy-j721e-wiz.c | 58 -- > 1 file changed, 49 insertions(+), 9 deletions(-) > Reviewed-by: Ravi Gunasekaran -- Regards, Ravi

[PATCH 1/2] net: ti: cpsw-mdio: Add workaround for errata i2329

2022-09-22 Thread Ravi Gunasekaran
and the workaround is available in: https://www.ti.com/lit/er/sprz487a/sprz487a.pdf Add implementation to disable MDIO state machine, configure MDIO in manual mode and provide software MDIO read and writes via MDIO bitbanging. Allow the MDIO to be initialized based on the need for manual mode. Signed-off-by: Ravi

[PATCH 2/2] net: ti: am65-cpsw-nuss: Enable MDIO manual mode

2022-09-22 Thread Ravi Gunasekaran
For the TI SoCs affected by errata i2329, enable MDIO manual mode by default Signed-off-by: Ravi Gunasekaran --- drivers/net/ti/am65-cpsw-nuss.c | 23 ++- 1 file changed, 22 insertions(+), 1 deletion(-) diff --git a/drivers/net/ti/am65-cpsw-nuss.c b/drivers/net/ti/am65-cpsw

[PATCH 0/2] Implement workaround for errata i2329

2022-09-22 Thread Ravi Gunasekaran
pins. Ravi Gunasekaran (2): net: ti: cpsw-mdio: Add workaround for errata i2329 net: ti: am65-cpsw-nuss: Enable MDIO manual mode drivers/net/ti/am65-cpsw-nuss.c | 24 ++- drivers/net/ti/cpsw.c | 3 +- drivers/net/ti/cpsw_mdio.c | 255 +++- drivers