Re: [PATCH v2 3/3] board: toradex: add verdin am62 support

2023-08-25 Thread Sverdlin, Alexander
Hello Marcel, On Mon, 2023-06-12 at 22:54 +0200, Marcel Ziswiler wrote: > +Build U-Boot for R5 > +--- > + > +.. code-block:: bash > + > +    $ export ARCH=arm > +    $ export CROSS_COMPILE=arm-none-linux-gnueabihf- > +    $ make verdin-am62_r5_defconfig O=/tmp/r5 > +    $ make O=/t

Re: [PATCH v2 3/3] board: toradex: add verdin am62 support

2023-08-25 Thread Sverdlin, Alexander
Hello Marcel, On Fri, 2023-08-25 at 15:20 +0200, Alexander Sverdlin wrote: > > +Build U-Boot for R5 > > +--- > > + > > +.. code-block:: bash > > + > > +    $ export ARCH=arm > > +    $ export CROSS_COMPILE=arm-none-linux-gnueabihf- > > +    $ make verdin-am62_r5_defconfig O=/tmp/r5

Re: [PATCH v2 4/8] arm: dts: k3-am625-sk: Enable emmc in SPL

2023-09-19 Thread Sverdlin, Alexander
Hello Sjoerd, On Thu, 2023-04-06 at 20:55 +0200, Sjoerd Simons wrote: > sdhci0 on the k3-am625-sk is the emmc, enable this in SPL as well to > allow booting from that media. > > Signed-off-by: Sjoerd Simons > Reviewed-by: Tom Rini thanks for the patch! I've tested the eMMC boot on AM625 SK and

Re: [PATCH 6/6] usb: dwc3: gadget: Convert epautoconf workaround to match_ep callback

2024-06-10 Thread Sverdlin, Alexander
Hello Marek! On Sun, 2024-06-09 at 23:32 +0200, Marek Vasut wrote: > Use the .match_ep() callback instead of workaround in core code. > Replace descriptor parsing with ch9 macros with the same effect. > Drop the SPL specific behavior, it is unclear why SPL should even > be special. > > Signed-off

Re: [PATCH 00/11] usb: gadget: Introduce handle_interrupts ops to USB_GADGET_GENERIC uclass

2024-06-14 Thread Sverdlin, Alexander
dget: dwc3: Convert interrupt handling to >     usb_gadget_generic_ops >   usb: gadget: Mark dm_usb_gadget_handle_interrupts as non-weak for >     DM_USB_GADGET For these, Tested-by: Alexander Sverdlin -- Alexander Sverdlin Siemens AG www.siemens.com

Re: [PATCH v4 2/6] siemens: eeprom: simplify setup & read

2024-01-08 Thread Sverdlin, Alexander
Hello Enrico et al.! On Wed, 2024-01-03 at 14:31 +0100, Enrico Leto wrote: > Since we have boards using the driver model or not for i2c, use abstraction > function to probe the i2c, check the EEPROM and read from EEPROM. > > Signed-off-by: Enrico Leto Reviewed-by: Alexander Sverdlin > --- >  

Re: [PATCH 4/6] siemens: board: clean up draco products vs common

2023-11-29 Thread Sverdlin, Alexander
Hi Enrico and all, On Tue, 2023-11-21 at 15:21 +0100, Enrico Leto wrote: > The common folder was initialially created for the common parts of the > products based on draco-am355x board family. These are the product > lines 'pxm2', 'rut' and the base line named 'draco'! > > Adding the new capricor

Re: [PATCH 3/6] siemens draco: i2c: use driver model for u-boot

2023-11-29 Thread Sverdlin, Alexander
Hi Enrico and all, On Tue, 2023-11-21 at 15:21 +0100, Enrico Leto wrote: > Add support for driver model where EEPROM data are read in draco board. > > Signed-off-by: Enrico Leto Reviewed-by: Alexander Sverdlin > --- >  configs/draco-etamin_defconfig   | 4 +++- >  configs/draco-rastaban_defcon

Re: [PATCH 6/6] siemens: factoryset: use correct config for soc specific implementation

2023-11-29 Thread Sverdlin, Alexander
Hi Enrico, On Tue, 2023-11-21 at 15:21 +0100, Enrico Leto wrote: > Adding the capricorn board family some parts diverge from draco family. > The switches used were not pertinent and need to be enhanced for each new > board of the capricorn family. Replace them through the SOC name 'AM33XX' > and '

Re: [PATCH 1/6] siemens: eeprom: clean up definitions

2023-11-29 Thread Sverdlin, Alexander
Hi Enrico and all, On Tue, 2023-11-21 at 15:21 +0100, Enrico Leto wrote: > Move the I2C and EEPROM address definitions in common board header. > > Signed-off-by: Enrico Leto Reviewed-by: Alexander Sverdlin > --- >  board/siemens/capricorn/board.c   |  5 ++--- >  board/siemens/common/board.c  

Re: [PATCH v2] net: ti: am65-cpsw-nuss: handle missing PHY in am65_cpsw_phy_init() gracefully

2024-05-01 Thread Sverdlin, Alexander
Hello Nishanth! On Thu, 2024-03-28 at 07:04 -0500, Nishanth Menon wrote: > On 11:53-20240328, Sverdlin, Alexander wrote: > [..] > > > Btw, this no longer applies on next. only applies on master. Depending > > > on where Tom would like to apply this change, might need a reb

Re: [PATCH v2] mmc: sdhci: introduce adma_write_desc() hook to struct sdhci_ops

2024-05-01 Thread Sverdlin, Alexander
Hello Greg, Ian, Jaehoon! On Mon, 2024-04-22 at 15:00 -0400, Greg Malysa wrote: > From: Ian Roberts > > Add this hook so that it can be overridden with driver specific > implementations. We also let the original sdhci_adma_write_desc() > accept &desc so that the function can set its new value. T

Re: [PATCH] mmc: sdhci: Correct ADMA_DESC_LEN to 12

2024-05-01 Thread Sverdlin, Alexander
Hi Greg, On Wed, 2024-05-01 at 22:38 -0400, Greg Malysa wrote: > Thanks for fixing this for me. > > > Confusion probably originates from Linux commit 685e444bbaa0 > > ("mmc: sdhci: Add ADMA2 64-bit addressing support for V4 mode"), but > > the latter "V4 mode" was never ported to U-Boot. > > I h

Re: [PATCH] configs: am62x_evm_r5: Increase size of malloc_simple heap after relocation

2024-05-01 Thread Sverdlin, Alexander
Hi Judith! On Mon, 2024-04-29 at 16:49 -0500, Judith Mendez wrote: > On AM62x SK we can see a boot failure with signature "alloc space > exhausted", so fix by increasing size of SPL_STACK_R_MALLOC_SIMPLE_LEN. > > Fixes: 128f81290b ("arm: dts: k3: binman: am625: add support for signing > TIFSSTUB

Re: [PATCH 5/5] dma: ti: k3-udma: invalidate prepared buffers before pushing to recv ring

2024-05-03 Thread Sverdlin, Alexander
Hello Matthias, On Fri, 2024-04-26 at 10:02 +0200, Matthias Schiffer wrote: > Buffers must not have an unclean cache before being used for DMA - a > pending write-back may corrupt the next dev-to-mem transfer otherwise. > > This was consistently noticeable during long TFTP transfers, when an ARP

Re: [PATCH] net: ti: am65-cpsw-nuss: don't touch DMA after stop

2024-05-10 Thread Sverdlin, Alexander
Hi Roger! On Fri, 2024-05-10 at 12:11 +0300, Roger Quadros wrote: > > Contrary to doc/develop/driver-model/ethernet.rst contract, eth_ops > > .free_pkt can be called after .stop, there are several error paths in TFTP, > > for instance: > > Doesn't this mean we need to fix TFTP instead of patching

Re: [PATCH v2] net: ti: am65-cpsw-nuss: handle missing PHY in am65_cpsw_phy_init() gracefully

2024-03-28 Thread Sverdlin, Alexander
Hi Nishanth! On Thu, 2024-03-28 at 06:45 -0500, Nishanth Menon wrote: > On 07:29-20240328, A. Sverdlin wrote: > > From: Alexander Sverdlin > > > > am65_cpsw_ofdata_parse_phy() tries to handle the case when PHY is not > > specified in DT gracefully: > > > > am65_cpsw_nuss_port ethernet@800po

Re: [PATCH v4 4/7] arm: dts: k3-am625-sk: Enable usb port in u-boot

2024-04-12 Thread Sverdlin, Alexander
Hi Sjoerd! On Fri, 2024-01-12 at 09:52 +0100, Sjoerd Simons wrote: > Enable usb0 in all boot phases for use with DFU > > Signed-off-by: Sjoerd Simons Reviewed-by: Alexander Sverdlin > --- > > Changes in v4: > - Don't force usb0 into peripheral mode > > Changes in v3: > - Enable usb nodes in

Re: [PATCH v4 3/7] board: ti: am62x: am62x: include env for DFU

2024-04-12 Thread Sverdlin, Alexander
Hi Sjoerd! On Fri, 2024-01-12 at 09:52 +0100, Sjoerd Simons wrote: > Include standard TI K3 dfu environment > > Signed-off-by: Sjoerd Simons > Reviewed-by: Mattijs Korpershoek Reviewed-by: Alexander Sverdlin > --- > > (no changes since v3) > > Changes in v3: > - Add dfu via environment rat

Re: [PATCH v4 2/7] usb: dwc3: Switch to device mode on gadget start

2024-04-12 Thread Sverdlin, Alexander
Hi Sjoerd! On Fri, 2024-01-12 at 09:52 +0100, Sjoerd Simons wrote: > When dr_mode is "otg" the dwc3 is initially configured in _OTG mode; > However in this mode the gadget functionality doesn't work without > further configuration. To resolve that on gadget start switch to _DEVICE > mode globally

Re: [PATCH v4 1/7] usb: dwc3: Add dwc3 glue driver for am62

2024-04-12 Thread Sverdlin, Alexander
Hi Sjoerd! Thank you for your efforts on the topic! On Fri, 2024-01-12 at 09:52 +0100, Sjoerd Simons wrote: > Add glue code for TI AM62 to the dwc3 driver; Most code adopted from > TI vendor u-boot code. > > Signed-off-by: Sjoerd Simons > Reviewed-by: Mattijs Korpershoek Works for me on AM623

Re: [PATCH v3 04/10] soc: ti: k3-navss-ringacc: Fix reset ring API

2024-08-16 Thread Sverdlin, Alexander
Hi Chintan, Vignesh, On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > 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

Re: [PATCH v3 02/10] firmware: ti_sci: Add No-OP for "RX_FL_CFG"

2024-08-16 Thread Sverdlin, Alexander
On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > 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: Ki

Re: [PATCH v3 03/10] soc: ti: k3-navss-ringacc: Initialize base address of ring cfg registers

2024-08-16 Thread Sverdlin, Alexander
On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > 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. Since register property is defined as "rin

Re: [PATCH v3 05/10] dma: ti: k3-udma: Add support for native configuration of chan/flow

2024-08-16 Thread Sverdlin, Alexander
Hi Chintan, On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > From: Kishon Vijay Abraham I > > In absence of Device Manager (DM) services such as at R5 SPL stage, > driver will have to natively setup TCHAN/RCHAN/RFLOW cfg registers. > Existing UDMA driver performed the above mentioned c

Re: [PATCH v3 06/10] arm: mach-k3: am62x: am625_init: Probe AM65 CPSW NUSS

2024-08-16 Thread Sverdlin, Alexander
Hi Chintan, On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > 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 > Signed-off-by:

Re: [PATCH v3 09/10] arm: dts: k3-am625-r5-sk: Enable DM services for main_pktdma

2024-08-16 Thread Sverdlin, Alexander
Hi Chintan, On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > From: Siddharth Vadapalli > > Enable DM services for main_pktdma during R5 SPL stage. > > Signed-off-by: Siddharth Vadapalli > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > Link to v2: > https

Re: [PATCH 12/35] global_data: Convert have_console into a flag

2024-08-16 Thread Sverdlin, Alexander
Hi Simon, On Wed, 2024-07-24 at 09:08 -0600, Simon Glass wrote: > We don't need a full word for this boolean value. Convert it into a flag > to save space in global_data. > > Signed-off-by: Simon Glass > --- > >  arch/arm/cpu/arm926ejs/mxs/spl_boot.c |  2 +- >  arch/arm/mach-exynos/spl_boot.c  

Re: [PATCH v3 04/10] soc: ti: k3-navss-ringacc: Fix reset ring API

2024-08-23 Thread Sverdlin, Alexander
Hi Chintan, On Fri, 2024-08-23 at 15:22 +0530, Chintan Vankar wrote: > On 16/08/24 17:58, Sverdlin, Alexander wrote: > > Hi Chintan, Vignesh, > > > > On Fri, 2024-07-05 at 10:20 +0530, Chintan Vankar wrote: > > > From: Vignesh Raghavendra > > > > &g

Re: [PATCH v3 04/10] soc: ti: k3-navss-ringacc: Fix reset ring API

2024-08-25 Thread Sverdlin, Alexander
Hi Chintan, On Mon, 2024-08-26 at 09:24 +0530, Chintan Vankar wrote: > > > > > 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

Re: [PATCH v6 04/11] soc: ti: k3-navss-ringacc: Fix reset ring API

2024-08-26 Thread Sverdlin, Alexander
Hi Chintan! On Mon, 2024-08-26 at 15:55 +0530, Chintan Vankar wrote: > 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 > Signed

Re: [PATCH v6 05/11] soc: ti: k3-navss-ringacc: Fix reconfiguration of qmode API

2024-08-30 Thread Sverdlin, Alexander
Hi! On Fri, 2024-08-30 at 20:29 +0530, Vignesh Raghavendra wrote: > On 26/08/24 15:55, Chintan Vankar wrote: > > Function "k3_ringacc_ring_reconfig_qmode_raw()" should reset qmode to > > requested value and should not update other fields in ring configuration > > register. > > > > Signed-off-by:

Re: [PATCH v1 13/22] siemens: configs/capricorn_cxg3_defconfig: updates

2024-11-11 Thread Sverdlin, Alexander
On Fri, 2024-11-08 at 06:21 +0100, Heiko Schocher wrote: > make savedefconfig and add SCU_WDT and fix environment > offsets, as since silicium c0 the boot container takes place > at offset 0 and so the u-boot-env must be moved outside of > the boot container area. > > Signed-off-by: Heiko Schocher

Re: [PATCH v1 15/22] siemens: imx8-capricorn.dtsi: small adaptions

2024-11-11 Thread Sverdlin, Alexander
Hi Heiko! On Mon, 2024-11-11 at 09:47 +0100, Heiko Schocher wrote: > > Here the commit "feat(scu_wdt): add option to control external wdt via IMX8 > > SCU" > > https://code.siemens.com/ccp/u-boot/-/commit/a5b99d1c406fdf92cd1d7d50d566a4e1e66c447b > > > > @Heiko: Check if we have a chance to put

Re: [PATCH v1 15/22] siemens: imx8-capricorn.dtsi: small adaptions

2024-11-11 Thread Sverdlin, Alexander
Hello Heiko, Fabio, On Mon, 2024-11-11 at 06:52 +0100, Heiko Schocher wrote: > > >    arch/arm/dts/imx8-capricorn.dtsi | 16 > > > > Are there any plans to upstream this dtsi and switch to OF_UPSTREAM? > > Hmm... good question! Currently we work on U-Boot to get mainline ready >

Re: [PATCH v1 07/22] siemens: imx8qxp-capricorn-u-boot.dtsi: fix boot

2024-11-11 Thread Sverdlin, Alexander
On Fri, 2024-11-08 at 06:21 +0100, Heiko Schocher wrote: > current generated flash.bin image does not longer > boot on cxg3 board. > > Rename bootph-pre-ram to bootph-all so flash.bin > boots again! > > Signed-off-by: Heiko Schocher Reviewed-by: Alexander Sverdlin > --- > >  arch/arm/dts/imx

Re: [PATCH v1 09/22] siemens: imximage.cfg: correct comment

2024-11-11 Thread Sverdlin, Alexander
On Fri, 2024-11-08 at 06:21 +0100, Heiko Schocher wrote: > fix wrong comment. > > Signed-off-by: Heiko Schocher Reviewed-by: Alexander Sverdlin > --- > >  board/siemens/capricorn/imximage.cfg | 3 ++- >  1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/board/siemens/capricorn/i

Re: [PATCH v1 03/22] tools: imx8image: Improve error message

2024-11-11 Thread Sverdlin, Alexander
On Fri, 2024-11-08 at 06:21 +0100, Heiko Schocher wrote: > Improve error message "header tag mismatched" > Add filename to error message to see, which file > is wrong. > > Signed-off-by: Heiko Schocher Reviewed-by: Alexander Sverdlin > --- > >  tools/imx8image.c | 2 +- >  1 file changed, 1 in

Re: [PATCH v1 01/22] wdt: imx8qxp: add option to control external PMIC wdt via IMX8 SCU

2024-11-08 Thread Sverdlin, Alexander
Hi Stefan, On Fri, 2024-11-08 at 08:19 +0100, Stefan Roese wrote: > > control the watchdog in the PMIC through SCU. > > A few more details would be good here (please see below). > > > Signed-off-by: Andrej Valek > > Signed-off-by: Heiko Schocher > > --- > > > >    drivers/misc/imx8/scu_api.c 

Re: [PATCH v1 22/22] siemens: capricorn: update maintainers

2024-11-08 Thread Sverdlin, Alexander
On Fri, 2024-11-08 at 06:21 +0100, Heiko Schocher wrote: > From: Enrico Leto > > update MAINTAINERS file, add some more board maintainers. > > Signed-off-by: Enrico Leto > Signed-off-by: Heiko Schocher Reviewed-by: Alexander Sverdlin > --- > >  board/siemens/capricorn/MAINTAINERS | 12

Re: [PATCH] console: Use printascii for lowlevel debug string

2024-11-08 Thread Sverdlin, Alexander
Hi Kever, On Fri, 2024-11-08 at 16:36 +0800, Kever Yang wrote: > The lowlevel debug with CONFIG_DEBUG_UART has printascii for string > print, and no need to convert to use printch now. > > Signed-off-by: Kever Yang LGTM, Reviewed-by: Alexander Sverdlin > --- > >  common/console.c | 6 +-

Re: [PATCH v2 01/20] wdt: imx8qxp: add option to control external PMIC wdt via IMX8 SCU

2024-11-18 Thread Sverdlin, Alexander
On Mon, 2024-11-18 at 07:26 +0100, Heiko Schocher wrote: > Driver for a PMIC watchdog timer controlled via Siemens SCU firmware > extensions. Only useful on some Siemens i.MX8-based platforms as > special SCFW is needed which provides the needed SCU API. > > Signed-off-by: Andrej Valek > Signed-o

Re: [PATCH] watchdog: rti: drop hack manipulating WDT clock rate

2024-11-20 Thread Sverdlin, Alexander
Thanks for quick feedback, Jan! On Thu, 2024-11-21 at 07:16 +0100, Jan Kiszka wrote: > On 20.11.24 23:24, A. Sverdlin wrote: > > From: Alexander Sverdlin > > > > The hack itself seems to be copied from Linux rti_wdt.c, but the WDT reset > > principle is different in U-Boot. While Linux relies on

Re: [PATCH] watchdog: rti: support SPL (or re-start)

2024-12-10 Thread Sverdlin, Alexander
Hi Stefan! On Fri, 2024-11-08 at 22:15 +0100, A. Sverdlin wrote: > From: Alexander Sverdlin > > If the RTI watchdog has been enabled in SPL, enabling it in U-Boot proper > fails because it can only be enabled once in HW and never stopped. This > however leads to a situation that wdt_cyclic() wat

Re: [PATCH] watchdog: rti: support SPL (or re-start)

2024-12-15 Thread Sverdlin, Alexander
Hi Stefan! On Sat, 2024-12-14 at 14:36 +0100, Stefan Roese wrote: > > > If the RTI watchdog has been enabled in SPL, enabling it in U-Boot proper > > > fails because it can only be enabled once in HW and never stopped. This > > > however leads to a situation that wdt_cyclic() watchdog trigger is n

Re: [PATCH v3 14/20] siemens: capricorn: small board updates

2024-11-21 Thread Sverdlin, Alexander
Hi Heiko! On Thu, 2024-11-21 at 06:25 +0100, Heiko Schocher wrote: > with newest SCFW build_info() works now, so call it > from checkboard() now. > > As we only use uart2 as console, do not init uart0. > > Signed-off-by: Heiko Schocher Reviewed-by: Alexander Sverdlin > --- > > (no changes s

Re: [PATCH v3 07/20] siemens: imx8qxp-capricorn-u-boot.dtsi: fix boot

2024-11-21 Thread Sverdlin, Alexander
Hi Heiko! On Thu, 2024-11-21 at 06:25 +0100, Heiko Schocher wrote: > current generated flash.bin image does not longer > boot on cxg3 board. > > Rename bootph-pre-ram to bootph-all so flash.bin > boots again! > > Add u-boot specific change (add bootph-all property) > in A35_0 node to imx8qxp-cap

Re: [PATCH v2 13/60] console: Support a format string for stderr output

2025-05-02 Thread Sverdlin, Alexander
Hi Simon! On Fri, 2025-05-02 at 08:46 -0600, Simon Glass wrote: > Add a console_printf_select_stderr() function so that it is not > necessary for the caller to process the format string. > > Signed-off-by: Simon Glass Reviewed-by: Alexander Sverdlin > --- > > (no changes since v1) > >  comm

Re: [PATCH v2 10/13] board: ti: am62px: evm: Enable cache for AM62p

2025-02-19 Thread Sverdlin, Alexander
On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > From: Andreas Dannenberg > > Enable cache for AM62p to optimize performance of CPU to access data from > memory. > > Signed-off-by: Andreas Dannenberg > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > Link t

Re: [PATCH v2 01/13] net: tftp: Increase TFTP pkt string length to include null character

2025-02-19 Thread Sverdlin, Alexander
On Thu, 2025-02-20 at 01:29 +0530, Vankar, Chintan wrote: > > On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > > > To append a string to a tftp pkt, "tftp_send()" API invokes "sprintf()" > > > function which copies a string excluding a null character causing TFTP > > > not-null terminated

Re: [PATCH v2 13/13] arch: arm: dts: k3-am62p5-sk-u-boot: Add bootph-all property to necessary nodes

2025-02-19 Thread Sverdlin, Alexander
On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > Add "bootph-all" property to necessary nodes required for Ethernet boot > starting from R5 SPL stage. > > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > Link to v1: > https://lore.kernel.org/r/20250107093840.2

Re: [PATCH v2 02/13] net: ti: am65-cpsw-nuss: Define and switch to bind method for CPSW driver

2025-02-19 Thread Sverdlin, Alexander
Hi Chintan! On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > CPSW driver is defined as UCLASS_MISC driver which needs to be probed > explicitly. Define bind method for CPSW driver to scan and bind > ethernet-ports with UCLASS_ETH driver which will eventually probe CPSW > driver and avoid

Re: [PATCH v2 01/13] net: tftp: Increase TFTP pkt string length to include null character

2025-02-19 Thread Sverdlin, Alexander
Hi Chintan! On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > To append a string to a tftp pkt, "tftp_send()" API invokes "sprintf()" > function which copies a string excluding a null character causing TFTP > not-null terminated string error. Increase TFTP pkt string by 1 to avoid Is thi

Re: [PATCH v2 08/13] arm: dts: k3-am68-sk-base-board-u-boot: Add bootph-all property to necessary nodes

2025-02-19 Thread Sverdlin, Alexander
On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > Add "bootph-all" property to necessary nodes required for Ethernet boot > starting from R5 SPL stage. > > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > Link to v1: > https://lore.kernel.org/r/20250107093840.2

Re: [PATCH v2 01/13] net: tftp: Increase TFTP pkt string length to include null character

2025-02-20 Thread Sverdlin, Alexander
Hi Chintan! On Thu, 2025-02-20 at 01:29 +0530, Vankar, Chintan wrote: > > On Wed, 2025-02-19 at 16:18 +0530, Chintan Vankar wrote: > > > To append a string to a tftp pkt, "tftp_send()" API invokes "sprintf()" > > > function which copies a string excluding a null character causing TFTP > > > not-nu

Re: [PATCH v3 13/15] board: ti: j722s: evm: Enable cache for J722s

2025-02-25 Thread Sverdlin, Alexander
On Tue, 2025-02-25 at 17:19 +0530, Chintan Vankar wrote: > Enable cache for J722s to optimize performance of CPU to access data from > memory. > > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > This patch is newly introduced in this series. > >  board/ti/j722s/evm.c

Re: [PATCH v3 05/15] net: ti: Kconfig: Enable SPL_SYSCON config for CPSW

2025-02-25 Thread Sverdlin, Alexander
On Tue, 2025-02-25 at 17:18 +0530, Chintan Vankar wrote: > TI's Ethernet switch needs system controllers enabled in R5SPL stage while > booting via Ethernet. Enable SPL_SYSCON config for > CONFIG_TI_AM65_CPSW_NUSS. > > Signed-off-by: Chintan Vankar Reviewed-by: Alexander Sverdlin > --- > > Th