Hi John,
thanks for your patch.
albeit it is obvious we tend to have a small commit message here. The
code change lgtm.
Regards,
Marco
On 23-01-22, John Watts wrote:
> Signed-off-by: John Watts
> ---
> drivers/i2c/busses/i2c-imx-early.c | 14 ++
> include/pbl/i2c.h
Move the dts file into a dtsi file. This is in preparation for the
imx8mm-evkb support which will reuse the config. No functional change.
Signed-off-by: Marco Felsch
---
v3:
- new in this series
arch/arm/dts/imx8mm-evk.dts | 54 +
arch/arm/dts/imx8mm-evk.dtsi
...@pengutronix.de: Fix pmic configuration settings to align with u-boot]
[m.fel...@pengutronix.de: Add dedicated dts support]
[m.fel...@pengutronix.de: Include Ahmad's review feedback]
Signed-off-by: Marco Felsch
---
Hi,
I pocked Johannes v2 [1] and added the review feedback from Ahmad.
[1
On 23-01-11, Ahmad Fatoum wrote:
> Hi,
>
> How do you generate your patches? Easiest is:
>
> git config sendemail.to barebox@lists.infradead.org
> git send-email -3 --annotate
>
> This will take care to number the patches correctly.
>
> On 11.01.23 11:01, Johannes Schneider wrote:
> >
On 22-12-22, Ahmad Fatoum wrote:
> In preparation for a fix in a follow-up commit, refactor the code, so
> each fixup is done in its own function. This enables easy early exists
> without increasing indentation level. No functional change
>
> Signed-off-by: Ahmad Fatoum
Rev
On 22-12-22, Ahmad Fatoum wrote:
> The Linux i.MX clk drivers instantiate all muxes with
> CLK_SET_RATE_NO_REPARENT. Do likewise in barebox.
>
> Signed-off-by: Ahmad Fatoum
Reviewed-by: Marco Felsch
> ---
> drivers/clk/imx/clk.h | 30 +++---
>
Hi Ahmad,
On 22-12-22, Ahmad Fatoum wrote:
> Give the defconfig some color by enabling console colors during startup,
> i.MX6 parallel LCD driver, backlight and some more options that were
> noticed missing during debugging a skov-imx6 issue.
I'm okay with enabling a few command and the colored
with:
>
> mode_name: invalid:0 (type: enum)
>
> Fixes: 31d2289da2f3 ("ARM: boards: skov-imx6: start using deep-probe")
> Signed-off-by: Ahmad Fatoum
Reviewed-by: Marco Felsch
> ---
> arch/arm/boards/skov-imx6/board.c | 7 ---
> 1 file changed, 4 insertions(+), 3
Hi Ahmad,
On 22-12-22, Ahmad Fatoum wrote:
> EDID readout errors happen often, e.g. because the HDMI port doesn't
> have a display connected. However, when a monitor is connected, but some
> other error occurs, barebox is silent. Add a debug message with an error
> code for this.
>
>
On 22-12-22, Ahmad Fatoum wrote:
> Hello Macro,
>
> On 22.12.22 16:47, Marco Felsch wrote:
> > Hi Ahamd,
> >
> > thanks for your patch.
> >
> > On 22-12-22, Ahmad Fatoum wrote:
> >> The "Deep probe supported due to" was not written t
Hi Ahamd,
thanks for your patch.
On 22-12-22, Ahmad Fatoum wrote:
> The "Deep probe supported due to" was not written to log and thus only
> shown on startup and not in dmesg output. Make it available in both.
>
> Signed-off-by: Ahmad Fatoum
> ---
> common/deep-probe.c | 4 +++-
> 1 file
gt; Rename struct fs_driver_d to fs_driver
Reviewed-by: Marco Felsch
> Documentation/devel/porting.rst | 3 +-
> arch/arm/boards/animeo_ip/init.c | 4 +-
> arch/arm/boards/at91sam9m10ihd/hw_version.c | 2 +-
> arch/arm/boards/at91sam9x5ek/hw_version.c
s use the Linux naming, we could drop the union
> again.
Nice :)
> Signed-off-by: Ahmad Fatoum
Reviewed-by: Marco Felsch
> ---
> include/driver.h | 10 --
> 1 file changed, 8 insertions(+), 2 deletions(-)
>
> diff --git a/include/driver.h b/include/driver.h
>
://lore.kernel.org/all/20220524152401.1663317-9-ross.bur...@arm.com/
[2]
https://lore.barebox.org/barebox/20220713100922.1880282-1-a.fat...@pengutronix.de/
[3] https://lkml.org/lkml/2020/7/1/848
Signed-off-by: Marco Felsch
---
pbl/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
> + if (dev->device_node && supply) {
> ri = of_regulator_get(dev, supply);
> if (IS_ERR(ri))
> return ERR_CAST(ri);
Nevertheless the change is correct, so:
Reviewed-by: Marco Felsch
Regards,
Marco
> --
> 2.30.2
>
>
>
ware
files. As result a firmware binary file update is not recognized by make
and and won't make into the final binary. This commit fixes this and so
an update will be recognized automatically without require a 'make
clean' before.
Fixes: 663894d99c64 ("firmware: refactor firmware/Makefile")
Signed-off-by: Ahmad Fatoum
Reviewed-by: Marco Felsch
> ---
> arch/arm/boards/lxa-mc1/board.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/boards/lxa-mc1/board.c b/arch/arm/boards/lxa-mc1/board.c
> index 9126973dcbdc..f8a7a6bd1f72 100644
> --- a/arch/ar
On 22-11-02, Sascha Hauer wrote:
> On Wed, Nov 02, 2022 at 09:44:41AM +0100, Marco Felsch wrote:
> > > +static void dump_fcb_n(struct fcb_block **fcbs, int n)
> > > +{
> > > + int i;
> > > +
> > > + if (!n || !fcbs[n])
> > > +
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> The FCB on NAND has a special page layout and thus can't be read with
> the normal MTD driver. Add a fcb command for printing information about
> the installed FCB on the console.
>
> Signed-off-by: Sascha Hauer
> ---
> commands/Kconfig |
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> i.MX7 xload NAND support works like on i.MX6, but the FCB is in a
> different format. The FCB page uses BCH62 ECC, has 8 ECC chunks with
> 128 bytes each with a resulting total page size of 1862 bytes. Also
> the page data is written with a pseudo
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> mxs_nand_mode_fcb_62bit() can be shared between the regular MTD NAND
> driver and the upcoming i.MX7 xload driver. Move to header file.
>
> Signed-off-by: Sascha Hauer
> ---
> drivers/mtd/nand/nand_mxs.c | 34 ++
>
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> The read page code can be reused by upcoming i.MX7 support, but the ECC
> checking will be different. Pull ECC status checking out of the read
> page code to make that reusable on i.MX7.
>
> Signed-off-by: Sascha Hauer
> ---
>
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> mxs_nand_check_onfi() and mxs_nand_read_id() setup the same DMA
> descriptors with only a small difference. factor out a common
> read_id function from it and share the code.
>
> Signed-off-by: Sascha Hauer
> ---
>
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> We have a set of GPMI register defines in include/, so use them for the
> xload driver as well.
>
> Signed-off-by: Sascha Hauer
> ---
> arch/arm/mach-imx/xload-gpmi-nand.c | 66 -
> 1 file changed, 17 insertions(+), 49
Hi Sascha,
On 22-11-01, Sascha Hauer wrote:
> The GPMI registers are needed in the MTD GPMI driver and also in the
> xload-gpmi driver. So far both use their own set of register defines.
> Move the MTD GPMI register defines to include/ so that we can use them
> in the xload driver as well.
Nit:
On 22-11-01, Ahmad Fatoum wrote:
> We have a couple of log messages in the file, so ensure they are
> appropriately prefixed.
>
> Signed-off-by: Ahmad Fatoum
> ---
> arch/arm/mach-imx/atf.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/arch/arm/mach-imx/atf.c
lator") targeted Renesas SOCs.
|
| Signed-off-by: Andrey Zhizhikin
| Cc: Stefano Babic
Signed-off-by: Marco Felsch
---
arch/arm/dts/imx8mp-evk.dts | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/dts/imx8mp-evk.dts b/arch/arm/dts/imx8mp-evk.dts
index 3264ade4b8..
Hi Sascha,
On 22-10-24, Sascha Hauer wrote:
> Signed-off-by: Sascha Hauer
> ---
> arch/arm/boards/nxp-imx8mm-evk/board.c | 24 +++-
> 1 file changed, 19 insertions(+), 5 deletions(-)
>
> diff --git a/arch/arm/boards/nxp-imx8mm-evk/board.c
>
Hi Antony,
On 22-10-22, Antony Pavlov wrote:
> On Wed, 5 Oct 2022 13:12:13 +0200
> Marco Felsch wrote:
>
> Hi Marco!
>
> It looks like this commit breaks emulate.pl, e.g.
>
> ARCH=riscv
> ARCH=${ARCH} ./test/emulate.pl --kconfig-full --test --
> --junitxml
On 22-10-19, Sascha Hauer wrote:
> On Fri, Oct 14, 2022 at 06:35:30PM +0200, Marco Felsch wrote:
> > No functional change, just apply the common code standards.
>
> I didn't know these are common code standards for barebox ;)
I thought that we are following the Linu
On 22-10-19, Sascha Hauer wrote:
> On Fri, Oct 14, 2022 at 06:35:28PM +0200, Marco Felsch wrote:
> > As written in the state.rst documentation, state will silently fail if
> > the STATE_DRV is not selected. So enabling state without the state
> > driver is useless.
set this
flag to tell barebox that everything is okay.
Signed-off-by: Marco Felsch
---
arch/arm/dts/imx8mp.dtsi | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/dts/imx8mp.dtsi b/arch/arm/dts/imx8mp.dtsi
index b251ebeada..7e46a71546 100644
--- a/arch/arm/dts/imx8mp.dtsi
+++ b/arch
Align the define with the name used by the usptream and downstream tf-a.
While on change the pr_info behaviour to only print the version if a0
contain something useful.
Signed-off-by: Marco Felsch
---
arch/arm/mach-imx/imx8m.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions
Convert the driver to the board driver mechanism. While on it enable the
deep-probe support and add a comment about the ENET1_RGMII_EN bit
setting.
Signed-off-by: Marco Felsch
---
arch/arm/boards/nxp-imx8mp-evk/board.c | 21 -
1 file changed, 16 insertions(+), 5 deletions
On 22-10-14, Marco Felsch wrote:
> Hi,
>
> this small series adds the support to store the barebox-state on a
> on-disk partition like mbr/gpt. Note for testing you need a very recent
> dt-utils package. I will send the patches soon as well and will add a
> link to the
Hi,
this small series adds the support to store the barebox-state on a
on-disk partition like mbr/gpt. Note for testing you need a very recent
dt-utils package. I will send the patches soon as well and will add a
link to the patches here later.
Regards,
Marco
Marco Felsch (8
Rename the local variable in preparation of addding support for backends
stored within a partition table. This also aligns the name with the name
used by of_state_fixup().
No functional change.
Signed-off-by: Marco Felsch
---
common/state/state.c | 14 +++---
1 file changed, 7
This helper checks if the given device_node is a of-partition. The check
is based on the node name which should something like partition@xxx. If
that fails the we try to check if the parent node contains a
"fixed-partitions" compatible.
Signed-off-by: Marco Felsch
---
drivers/of/p
Rename backend members to drop the assumption that the backend always
points to a partition. This is preperation work for the upcoming
state partition table support which allows us to store the state within
a MBR/GPT partition.
Signed-off-by: Marco Felsch
---
common/state/state.c | 17
s table are gone.
For this feature a recent dt-utils package must be used, else you can't
manipulate the state from user-space anymore.
Signed-off-by: Marco Felsch
---
.../bindings/barebox/barebox,state.rst| 30 ++-
Documentation/user/state.rst | 12 ++-
comm
As written in the state.rst documentation, state will silently fail if
the STATE_DRV is not selected. So enabling state without the state
driver is useless. Fix this by selecting the STATE_DRV if STATE is
selected.
Signed-off-by: Marco Felsch
---
common/Kconfig | 1 +
1 file changed, 1
makes the code easier to read. Also remove the unnecessary
ret initialization.
Signed-off-by: Marco Felsch
---
common/state/state.c | 55 ++--
1 file changed, 38 insertions(+), 17 deletions(-)
diff --git a/common/state/state.c b/common/state/state.c
index
No functional change, just apply the common code standards.
Signed-off-by: Marco Felsch
---
common/state/state.c | 32
1 file changed, 16 insertions(+), 16 deletions(-)
diff --git a/common/state/state.c b/common/state/state.c
index d954f0d453..65e47524a3 100644
Just a cleanup nothing special.
Signed-off-by: Marco Felsch
---
drivers/of/base.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/of/base.c b/drivers/of/base.c
index ea2a88764b..2eee1279a9 100644
--- a/drivers/of/base.c
+++ b/drivers/of/base.c
@@ -32,7 +32,7 @@ bool
Hi Ahmad,
On 22-10-13, Ahmad Fatoum wrote:
> TL;DR: GCC can prove that variables aren't supposed to overlap and as
> such it generated code than readded get_runtime_offset() on top of an
> already relocated linker-defined variable's address.
> See PATCH 2/3 for a disassembly of the affected code.
9 ("RISC-V: support incoherent I-Cache")
Signed-off-by: Marco Felsch
---
arch/riscv/include/asm/cache.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/riscv/include/asm/cache.h b/arch/riscv/include/asm/cache.h
index 9a0b9326b2..6d69ed49bd 100644
--- a/arch/riscv
Add the support to query the vendorid which is stored within the
mvendorid register. This register is only accessible from M-Mode so we
need to use the sbi interface if we are running from S-Mode.
Signed-off-by: Marco Felsch
---
v2:
- adapt switch-case to fix compile error.
arch/riscv/include
the generic Zicbom extension.
While on it replace the 'asm' statement by '__asm__' so we are not
relying on GNU extension.
Signed-off-by: Marco Felsch
---
Hi,
please note that I'm aware of the fact that not all RISC-V cores
implementing the vendorid register, which is quirky according
This commit squashes all 64bit risc-v defconfigs into one like armv8.
The 32bit defconfigs are not changed since those systems have tight
resource constraints and a generic defconfig may break some systems.
Signed-off-by: Marco Felsch
---
v2:
- new commit
Documentation/boards/emulated.rst
Add Allwinner sun20i SoC and D1-Nezha board support.
Signed-off-by: Marco Felsch
---
v2:
- drop own defconfig instead use the new rv64i_defconfig
Documentation/boards/riscv.rst| 102 ++
arch/riscv/Kconfig.socs | 16
arch/riscv/boards
by the relocation code.
This commit prepares barebox to add support for the special cache ops
used by several T-Head CPUs.
Signed-off-by: Marco Felsch
---
v2:
- adapt switch-case to fix compile error
arch/riscv/boot/entry.c | 3 +-
arch/riscv/boot/entry.h | 6 ++--
arch/riscv
Import the vendor id list from upstream linux.
Signed-off-by: Marco Felsch
---
arch/riscv/include/asm/vendorid_list.h | 11 +++
1 file changed, 11 insertions(+)
create mode 100644 arch/riscv/include/asm/vendorid_list.h
diff --git a/arch/riscv/include/asm/vendorid_list.h
b/arch/riscv
Hi,
On 22-09-30, Johannes Zink wrote:
> From: Ahmad Fatoum
>
> BootROM on the i.MX7 doesn't set the SMP bit when booted
> over serial download. This leads to vastly worse performance
> when doing memory-heavy operations in a USB-booted system,
> as the caches are not utilized. Example running
On 22-09-22, Sascha Hauer wrote:
> On Wed, Sep 21, 2022 at 07:20:51PM +0200, Marco Felsch wrote:
> > Currently the fsp_table must be set manually within the 'struct
> > dram_timing_info'. Since the 'struct fsp_msg' already has all
> > information needed for the fsp_table
Currently the fsp_table must be set manually within the 'struct
dram_timing_info'. Since the 'struct fsp_msg' already has all
information needed for the fsp_table we can use it to set it
automatically. This approach is less error-prone and avoids information
duplication.
Signed-off-by: Marco
Barebox now have the support to write the fsp_table entries
automatically based on fsp_msg.drata information. Remove the fsp_table
entry and let the common code do the rest for us.
Signed-off-by: Marco Felsch
---
arch/arm/boards/mnt-reform/lpddr4-timing.c | 1 -
arch/arm/boards/nxp-imx8mn
Hi Sascha,
On 22-09-16, Sascha Hauer wrote:
> So far we only bring up network interfaces when we actually need them.
> This means we could be idling in the shell for long and once the user
> decides to do networking he has to wait for the link to be established.
> We can do better: Before going
On 22-09-14, Sascha Hauer wrote:
> On Tue, Sep 13, 2022 at 02:49:52PM +0200, Marco Felsch wrote:
> > Use the dedicated scratch register for setting the pbl flags. Each mode
> > has it's own scratch register so we are not conflicting with M-mode
> > running firmware e.g. OpenS
On 22-09-14, Sascha Hauer wrote:
> On Wed, Sep 14, 2022 at 09:52:37AM +0200, Marco Felsch wrote:
> > On 22-09-14, Sascha Hauer wrote:
> > > On Tue, Sep 13, 2022 at 02:49:54PM +0200, Marco Felsch wrote:
> > > > Add Allwinner sun20i SoC and D1-Nezha board suppor
On 22-09-14, Sascha Hauer wrote:
> On Tue, Sep 13, 2022 at 02:49:54PM +0200, Marco Felsch wrote:
> > Add Allwinner sun20i SoC and D1-Nezha board support.
> >
> > Signed-off-by: Marco Felsch
> > ---
> > Documentation/boards/riscv.rst| 102
Add the support to query the vendorid which is stored within the
mvendorid register. This register is only accessible from M-Mode so we
need to use the sbi interface if we are running from S-Mode.
Signed-off-by: Marco Felsch
---
arch/riscv/include/asm/system.h | 31
the vendorid register. Please see the
patch notes on this patch.
Patch 6 adds the support for the D1 board and a detailed description how
to build and flash a bootable image.
Marco Felsch (6):
RISC-V: cache: fix local_flush_icache_all enabling
RISC-V: add riscv_vendor_id() support
RISC-V: import
the generic Zicbom extension.
While on it replace the 'asm' statement by '__asm__' so we are not
relying on GNU extension.
Signed-off-by: Marco Felsch
---
Hi,
please note that I'm aware of the fact that not all RISC-V cores
implementing the vendorid register, which is quirky according
Add Allwinner sun20i SoC and D1-Nezha board support.
Signed-off-by: Marco Felsch
---
Documentation/boards/riscv.rst| 102 +
arch/riscv/Kconfig.socs | 16 +++
arch/riscv/boards/Makefile| 1 +
arch/riscv/boards/allwinner-d1/Makefile
by the relocation code.
This commit prepares barebox to add support for the special cache ops
used by several T-Head CPUs.
Signed-off-by: Marco Felsch
---
arch/riscv/boot/entry.c | 3 +-
arch/riscv/boot/entry.h | 6 ++--
arch/riscv/boot/start.c | 13 +++
arch/riscv
9 ("RISC-V: support incoherent I-Cache")
Signed-off-by: Marco Felsch
---
arch/riscv/include/asm/cache.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/riscv/include/asm/cache.h b/arch/riscv/include/asm/cache.h
index 9a0b9326b2..6d69ed49bd 100644
--- a/arch/riscv
Import the vendor id list from upstream linux.
Signed-off-by: Marco Felsch
---
arch/riscv/include/asm/vendorid_list.h | 11 +++
1 file changed, 11 insertions(+)
create mode 100644 arch/riscv/include/asm/vendorid_list.h
diff --git a/arch/riscv/include/asm/vendorid_list.h
b/arch/riscv
On 22-08-30, Ahmad Fatoum wrote:
> Hello Marco,
>
> On 30.08.22 10:10, Marco Felsch wrote:
> > Hi Ahamd,
> >
> > On 22-08-30, Ahmad Fatoum wrote:
> >> Now with i.MX8M feature controller driver support available, have the
> >> OCOTP provide feature c
Hi Ahamd,
On 22-08-30, Ahmad Fatoum wrote:
> Now with i.MX8M feature controller driver support available, have the
> OCOTP provide feature control on the i.MX8MM to ensure the kernel DT
> does not cause Linux to access the VPU and its power domains,
> when barebox knows them to be unavailable.
>
Hi Ahmad,
On 22-08-15, Ahmad Fatoum wrote:
> For most users imx8mX_load_and_start_tfa saves a good deal of
> boilerplate. It's not always sufficient though:
>
> - board code may need a pointer to bl33 to install other firmware
> besides barebox proper from there before jumping to bl31
Hi Ahmad,
On 22-08-09, Ahmad Fatoum wrote:
> board code fix up the device tree for OS consumption. Sometimes it's
> useful for the fixup to append onto an existing property.
> Add a helper that simplifies this.
do you have a particular use-case for this?
Regards,
Marco
> Signed-off-by: Ahmad
> i2c: add for PBL use
> ARM: i.MX8M: use new pbl/pmic.h API
Hi Ahmad,
whole series LGTM.
Reviewed-by: Marco Felsch
>
> arch/arm/boards/ls1046ardb/lowlevel.c | 6 +-
> arch/arm/boards/mnt-reform/lowlevel.c | 28 +---
> arch/arm/boards/mnt-reform/
Hi Sascha,
please see below.
On 22-07-14, Sascha Hauer wrote:
> For the i.MX8MP NXP dropped the SDP protocol used on other SoCs.
> Instead the image is just sent straight to the device.
>
> Signed-off-by: Sascha Hauer
> ---
> scripts/imx/imx-usb-loader.c | 37
Hi Ahmad,
On 22-07-20, Ahmad Fatoum wrote:
> In remote labs co-located with other hardware, we've observed card
> detect levers of different boards to sporadically fail to detect
> the card, e.g. because the cable on the usbsdmux was yanked around
> by accident. When this happens, barebox usually
-by: Marco Felsch
Signed-off-by: Marco Felsch
---
arch/arm/boards/nxp-imx8mn-evk/board.c| 21 --
arch/arm/boards/nxp-imx8mn-evk/lowlevel.c | 13 +++-
arch/arm/dts/Makefile | 2 +-
arch/arm/dts/imx8mn-ddr4-evk.dts | 6 ++
arch/arm/dts/imx8mn-evk.dts
ources are available. So the probability to leave the system in
a partly initialized state is lesser.
Drop the !IS_ERR() since NULL and errors are handled in clk_enable() as
well.
Signed-off-by: Marco Felsch
---
drivers/usb/imx/chipidea-imx.c | 10 --
1 file changed, 8 insertions(+), 2 del
Don't return -ENODEV if of_phy_provider_lookup() fails instead propagate
the error. So errors like -EPROBE_DEFER are propagated correctly.
Signed-off-by: Marco Felsch
---
drivers/phy/phy-core.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/phy/phy-core.c b/drivers
: pass along correct DT depending on variant
Marco Felsch (3):
phy: fix deep probe support
phy: propagate error in of_phy_get_by_phandle
usb: chipidea: imx: call enable_clk after all resources are requested
arch/arm/boards/nxp-imx8mn-evk/board.c| 21 --
arch/arm/boards/nxp-imx8mn-evk
ets called by both functions.
Signed-off-by: Marco Felsch
---
drivers/phy/phy-core.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index b298da..ef7bd76648 100644
--- a/drivers/phy/phy-core.c
+++ b/drivers/phy/
Make the early printk buffer size adjustable since I noticed truncated
log buffers if I enable DEBUG_INITCALLS and DEBUG_PROBES.
Signed-off-by: Marco Felsch
---
common/Kconfig | 15 +++
common/console.c | 2 +-
2 files changed, 16 insertions(+), 1 deletion(-)
diff --git a/common
Hi Uwe,
thanks for the patch.
On 22-07-06, Uwe Kleine-König wrote:
...
> @@ -96,11 +92,6 @@ extern struct dram_timing_info imx8mm_evk_dram_timing;
>
> static void start_atf(void)
> {
> - size_t bl31_size;
> - const u8 *bl31;
> - enum bootsource src;
> - int instance;
> -
>
adapts the documentation for the i.MX8MP-EVK to
reference the upstream TF-A and to point out our favorite version.
Signed-off-by: Marco Felsch
---
Changelog:
v3:
- rephrase the complete commit message. Sry. again for the previouse
noise.
v2:
- fix commit message ger/eng mixup and mention tapeout
Drop the Barebox local 'phy-reset-gpios' property since the
upstream/linux device-tree is handling it now within the ethernet-phy
node and we get an error:
gpiolib: _gpio_request: gpio-118 (ethernet-phy@0) status -16
miibus0: failed to request reset gpio for: ethernet-phy@0
Signed-off-by: Marco
On 22-07-04, Lucas Stach wrote:
> Am Montag, dem 04.07.2022 um 16:27 +0200 schrieb Marco Felsch:
> > The usptream TF-A was shifting around the base address from version to
> > version. But finally with upstream TF-A v2.7 it is correctly set to
> > 0x97. This change was
the upstream TF-A. The new warning should point out that we
strongly recommended to use versions from v2.7 onwards, due to the
version mess explained above.
Signed-off-by: Marco Felsch
---
Changelog:
v2:
- fix commit message ger/eng mixup and mention tapeout version A1
Documentation/boards/imx
On 22-07-04, Lucas Stach wrote:
> Am Montag, dem 04.07.2022 um 14:33 +0200 schrieb Marco Felsch:
> > The usptream TF-A was shifting around the base address from version to
> > version. But finally with upstream TF-A v2.7 it is correctly set to
> > 0x97. This change was
Inform the user that appendroot failed instead of silently dropping the
request. This makes the debugging easier e.g. if nt-signature wasn't set
correctly.
Signed-off-by: Marco Felsch
---
common/bootm.c | 2 ++
1 file changed, 2 insertions(+)
diff --git a/common/bootm.c b/common/bootm.c
index
warning should point out that we
strongly recommended to use versions from v2.7 onwards, due to the
version mess explained above.
Signed-off-by: Marco Felsch
---
Documentation/boards/imx/nxp-imx8mp-evk.rst | 11 +++
arch/arm/mach-imx/include/mach/atf.h| 2 +-
2 files changed, 8
On 22-06-17, Sascha Hauer wrote:
> On Thu, Jun 16, 2022 at 06:38:45PM +0200, Oleksij Rempel wrote:
> > Am 16.06.22 um 18:28 schrieb Oleksij Rempel:
> > > Hi Robin,
> > >
> > > On Thu, Jun 16, 2022 at 03:11:06PM +0200, Robin van der Gracht wrote:
> > > > The usb check needs to be skipped unless
Just a cosmetic commit, no functional changes. Use a local variable for
the device_node instead of dereference the pointer multiple times.
Signed-off-by: Marco Felsch
---
drivers/regulator/fixed.c | 9 +
1 file changed, 5 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator
The core already handles the off_on_delay if we set it. So parse the
official device-tree property and set it if present.
Signed-off-by: Marco Felsch
---
drivers/regulator/fixed.c | 4
1 file changed, 4 insertions(+)
diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c
index
166-3-peng@oss.nxp.com
| Signed-off-by: Abel Vesa
Unlike the original patch this one also makes sure that the removed
gates are enabled as they are needed for their consumers.
Signed-off-by: Marco Felsch
---
drivers/clk/imx/clk-imx8mn.c | 65 ++--
1 file c
166-2-peng@oss.nxp.com
| Signed-off-by: Abel Vesa
Unlike the original patch this one also makes sure that the removed
gates are enabled as they are needed for their consumers.
Signed-off-by: Marco Felsch
---
drivers/clk/imx/clk-imx8mm.c | 65 ++--
1 file c
Drop the Barebox local property since the upstream/linux device-tree is
handling it now within the phy and we get an error:
ERROR: gpiolib: _gpio_request: gpio-118 (ethernet-phy@0) status -16
ERROR: miibus0: failed to request reset gpio for: ethernet-phy@0
Signed-off-by: Marco Felsch
Cc: Stefano Babic
8<-----
Signed-off-by: Marco Felsch
---
arch/arm/dts/imx8mm-evk.dts | 4
1 file changed, 4 insertions(+)
diff --git a/arch/arm/dts/imx8mm-evk.dts b/arch/arm/dts/imx8mm-evk.dts
index 854e9db869..6c06ad7e47 100644
--- a/arch/arm/dts/imx8mm-
On 22-05-24, Ahmad Fatoum wrote:
> Hi,
>
> On 24.05.22 09:30, Alexander Shiyan wrote:
> > For the correct work of GPIO driver, we need to define aliases.
> > Let's add them to the devicetree.
> >
> > Signed-off-by: Alexander Shiyan
> > ---
> > arch/arm/dts/am335x-myirtech-myd.dts | 7 +++
>
On 22-05-24, Alexander Shiyan wrote:
> вт, 24 мая 2022 г. в 12:03, Marco Felsch :
> > On 22-05-24, Alexander Shiyan wrote:
> > > For the correct work of GPIO driver, we need to define aliases.
> > > Let's add them to the devicetree.
> ...
> > > +
On 22-05-24, Alexander Shiyan wrote:
> For the correct work of GPIO driver, we need to define aliases.
> Let's add them to the devicetree.
>
> Signed-off-by: Alexander Shiyan
> ---
> arch/arm/dts/am335x-myirtech-myd.dts | 7 +++
> 1 file changed, 7 insertions(+)
>
> diff --git
On 22-05-23, Ahmad Fatoum wrote:
> On 23.05.22 10:22, Marco Felsch wrote:
> > On 22-05-23, Ahmad Fatoum wrote:
> >> Hello Marco,
> >>
> >> On 23.05.22 10:00, Marco Felsch wrote:
> >>> Hi Ahmad,
> >>>
> >>> On 22-05-23, A
On 22-05-23, Ahmad Fatoum wrote:
> Hello Marco,
>
> On 23.05.22 10:00, Marco Felsch wrote:
> > Hi Ahmad,
> >
> > On 22-05-23, Ahmad Fatoum wrote:
> >> A successful probe results in two lines logged:
> >>
> >> usb251xb usb2514b0: Hub co
501 - 600 of 783 matches
Mail list logo