In order to run binaries targeting the Cortex-M7, BOOTAUX is
required.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mp_beacon_defconfig b/configs/imx8mp_beacon_defconfig
index 288626c1511..9f6fabc894f 100644
--- a/configs/imx8mp_beacon_defconfig
+++ b/configs/imx8mp_beacon_defconfig
In order to run binaries targeting the Cortex-M7,
BOOTAUX is required.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mn_beacon_defconfig b/configs/imx8mn_beacon_defconfig
index c605567d9a6..115288aa27f 100644
--- a/configs/imx8mn_beacon_defconfig
+++ b/configs/imx8mn_beacon_defconfig
In order to run binaries targeting the Cortex-M4,
BOOTAUX is required.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mm_beacon_defconfig b/configs/imx8mm_beacon_defconfig
index e4eca19a10b..27007bd8687 100644
--- a/configs/imx8mm_beacon_defconfig
+++ b/configs/imx8mm_beacon_defconfig
ecause the osc_32k clock is not available
> in the SPL phase.
>
> Fix this problem by passing the 'bootph-all' and 'bootph-pre-ram'
> properties to make the osc_32k clock available in SPL.
>
> This also aligns with imx8mn and imx8mp-u-boot.dtsi files.
I
On Thu, Apr 24, 2025 at 9:37 PM Adam Ford wrote:
>
> On Thu, Apr 24, 2025 at 9:35 PM Fabio Estevam wrote:
> >
> > On Thu, Apr 24, 2025 at 11:20 PM Marek Vasut wrote:
> >
> > > Probably some timeout in the esdhc driver, maybe some clock are not
> >
On Thu, Apr 24, 2025 at 9:35 PM Fabio Estevam wrote:
>
> On Thu, Apr 24, 2025 at 11:20 PM Marek Vasut wrote:
>
> > Probably some timeout in the esdhc driver, maybe some clock are not
> > enabled ? I don't see it on the MX8MP.
>
> Same here. I didn't see this error on an imx8mp-evk.
If memory ser
doing this.
> >
> > It works for me:
> >
> > Tested-by: Fabio Estevam
>
> Can you send that as a fix ?
>
> And also, fix this patch up and drop the bogus second half of the commit
> message and do a V3.
Feel free to add:
Tested-by: Adam Ford # imx8mp-beacon
adam
>
> Thanks
On Sat, Apr 19, 2025 at 2:12 PM Fabio Estevam wrote:
>
> Hi Marek,
>
> On Fri, Apr 18, 2025 at 11:53 AM Marek Vasut wrote:
>
> > The "clock-output-names" is NEVER used for any clock look up.
>
> In Linux, "clock-output-names" is used to register the name of the
> fixed-rate clock.
>
> From Linux
On Fri, Apr 18, 2025 at 9:53 AM Marek Vasut wrote:
>
> On 4/17/25 7:11 PM, Fabio Estevam wrote:
> > From: Fabio Estevam
> >
> > Currently, fixed-rate clocks in U-Boot are named based on their devicetree
> > node names. For example, given the following node:
> >
> > osc_24m: clock-osc-24m {
> >
On Thu, Apr 17, 2025 at 6:31 AM Marek Vasut wrote:
>
> On 4/17/25 12:51 PM, Adam Ford wrote:
> > On Thu, Apr 17, 2025 at 2:24 AM Marek Vasut wrote:
> >>
> >> On 4/17/25 1:58 AM, Fabio Estevam wrote:
> >>> On Wed, Apr 16, 2025 at 8:47 PM Marek Vasut w
u-boot=> clk dump
> Rate Usecnt Name
> --
> 327680 |-- osc_32k
> 2400 5|-- osc_24m
>
> This change restores i.MX8MP USB functionality by ensuring the proper clock
>
On Thu, Apr 17, 2025 at 2:24 AM Marek Vasut wrote:
>
> On 4/17/25 1:58 AM, Fabio Estevam wrote:
> > On Wed, Apr 16, 2025 at 8:47 PM Marek Vasut wrote:
> >
> >> You are not supposed to use "clock-output-names" for clock look up.
> >> You are supposed to use "clocks"/"clock-names" DT properties and
On Wed, Apr 16, 2025 at 9:19 AM Christoph Niedermaier
wrote:
>
> From: Francesco Dolcini
> Sent: Wednesday, April 16, 2025 11:27 AM
> > On Tue, Apr 15, 2025 at 02:13:30PM -0300, Fabio Estevam wrote:
> >> On Tue, Apr 15, 2025 at 1:55 PM Marek Vasut wrote:
> >>> Do you have af9cdd1ccd2d ("Revert "
On Wed, Apr 16, 2025 at 5:34 PM Adam Ford wrote:
>
> On Wed, Apr 16, 2025 at 9:19 AM Christoph Niedermaier
> wrote:
> >
> > From: Francesco Dolcini
> > Sent: Wednesday, April 16, 2025 11:27 AM
> > > On Tue, Apr 15, 2025 at 02:13:30PM -0300, Fabio Estevam wro
The UART clocks were added around the same time some other clock
updates were happening, so converting clock-osc-24 back to osc_24
was missed on the UART clocks for imx8mm and imx8mn, so update
them here.
Fixes: b4734c9c333b ("clk: imx: Convert clock-osc-* back to osc_*")
Signed-off-by:
On Tue, Apr 15, 2025 at 12:13 PM Fabio Estevam wrote:
>
> On Tue, Apr 15, 2025 at 1:55 PM Marek Vasut wrote:
>
> > Do you have af9cdd1ccd2d ("Revert "arm64: dts: imx8mn: Include 32kHz
> > oscillator clock in SPL DTs"") in place ? If so, try and revert it, does
> > it help ?
>
> It seems I have no
On Tue, Apr 15, 2025 at 9:28 AM Francesco Dolcini wrote:
>
> Hello Marex, Fabio, all
>
> On Sun, Mar 23, 2025 at 04:58:45PM +0100, Marek Vasut wrote:
> > Convert clock-osc-24m back to osc_24m and clock-osc-32k back to osc_32k.
> > These are the clock which match clock tables in Linux. This is now
/clk/imx/clk-pllv3.c| 9 +-
> drivers/clk/imx/clk.h | 116 +-
> include/clk.h | 9 +
> include/linux/clk-provider.h | 10 +-
> 21 files changed, 783 insertions(+), 769 deletions(-)
For the series:
Tested-by: Adam Ford #imx8[
On Thu, Apr 3, 2025 at 2:39 AM Miquel Raynal wrote:
>
> Reparenting a clock C with a new parent P means that C will only
> continue clocking if P is already clocking when the mux is updated. In
> case the parent is currently disabled, failures (stalls) are likely to
> happen.
Marek made some chan
(-40C to 105C) at 28C
Model: Beacon EmbeddedWorks i.MX8MPlus Development kit
Signed-off-by: Adam Ford
---
drivers/cpu/imx8_cpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
V2: No change
diff --git a/drivers/cpu/imx8_cpu.c b/drivers/cpu/imx8_cpu.c
index 53d31b3c0bf..faa6a0704a6 1
On Thu, Apr 3, 2025 at 2:39 AM Miquel Raynal wrote:
>
> Add support for the LCD interfaces (LCDIF1/2). When probed, these
> interfaces request numerous clocks and power domains, attach the bridge
> and look for a panel in order to retrieve its capabilities and
> properties.
>
> There is a similar
On Thu, Apr 3, 2025 at 2:39 AM Miquel Raynal wrote:
>
> This block delivers power and clocks to the whole display and rendering
> pipeline.
>
> Signed-off-by: Miquel Raynal
> ---
> drivers/power/domain/Kconfig | 7 ++
> drivers/power/domain/Makefile | 1 +
> drivers/power/
On Fri, Mar 21, 2025 at 7:10 AM Fabio Estevam wrote:
>
> On Thu, Mar 20, 2025 at 11:27 PM Marek Vasut wrote:
>
> > Yes, I know. But then, I can safely say it is desired on hardware I
> > maintain, I think we could try and run a quick poll and see if others
> > see it the same way ?
>
> I think it
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
---
configs/imx8mn_beacon_defconfig | 2 ++
1 file changed, 2 insertions(+)
V2: No Change
diff --git a/configs/imx8mn_beacon_defconfig b/configs
On Tue, Mar 18, 2025 at 9:13 PM Marek Vasut wrote:
>
> On 3/19/25 2:42 AM, Adam Ford wrote:
> > Much of the data that is display by imx8_cpu.c is also
> > displayed from arch/arm/mach-imx/cpu.c, except the
> > speed grade and temperature unless the SoC is an i.MX9.
&g
On Wed, Mar 26, 2025 at 10:01 AM Miquel Raynal
wrote:
>
> These are all the clocks needed to get an LCD panel working, going
> through one of the LCDIF and the LDB. The media AXI and APB clocks are
> also described.
>
You might need to rebase off the 'next' branch. Marek V submitted a
bunch of c
On Tue, Mar 25, 2025 at 5:00 PM Fabio Estevam wrote:
>
> Hi Adam,
>
> On Tue, Mar 25, 2025 at 9:37 AM Adam Ford wrote:
>
> > Once this is complete, I'll have a smaller patch series which shrinks
> > my 8MP SPL code by around 4k by #ifdef'ing out dead code.
On Tue, Mar 25, 2025 at 9:14 AM Marek Vasut wrote:
>
> On 3/25/25 1:37 PM, Adam Ford wrote:
> > On Tue, Mar 25, 2025 at 6:36 AM Fabio Estevam wrote:
> >>
> >> On Tue, Mar 25, 2025 at 12:37 AM Marek Vasut
> >> wrote:
> >>
> >>>
On Tue, Mar 25, 2025 at 6:36 AM Fabio Estevam wrote:
>
> On Tue, Mar 25, 2025 at 12:37 AM Marek Vasut wrote:
>
> > Please drop the fullstop at the end of Subject and then add
> >
> > Reviewed-by: Marek Vasut
> >
> > To the whole series.
>
> Adam, no need to resend. I will take care of it, thanks
the previous implementation
for any CPU, make IMX8M imply IMX_TMU so it will be always displayed
unless a user decides to disable it.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx8m/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach
depends
on CPU, so both have to be selected.
V2: Changed the check in imx8_cpu to drop the IMX9 check and instead
imply IMX_TMU in the IMX9 family
Adam Ford (7):
imx: imx9: Imply IMX_TMU
cpu: imx8_cpu: Expand get_imx_type_str list of supported CPUs
imx: imx8m: Imply IMX_TMU
The imx8_cpu driver is a CPU Driver that supports the i.MX8M
family, and when it is enabled, it acts as an alternative to
arch/arm/mach-imx/cpu.c, but the imx8_cpu supports the driver
model where cpu.c does not. Imply this newer driver by default.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx
The imx8_cpu driver is a CPU Driver that supports the i.MX8Q
family. When it is enabled, it acts as an alternative to
arch/arm/mach-imx/cpu.c, but the imx8_cpu supports the driver
model where cpu.c does not. Imply this newer driver by default.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx
enabled in every IMX9, select this driver by default for the
platform.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx9/Kconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-imx/imx9/Kconfig b/arch/arm/mach-imx/imx9/Kconfig
index c3e758e103a..1ccdb1cf64f 100644
--- a/arch/arm
MPlus Development kit
After:
CPU: NXP i.MX8MP Rev1.1 A53 at 1200 MHz
CPU: Industrial temperature grade (-40C to 105C) at 28C
Model: Beacon EmbeddedWorks i.MX8MPlus Development kit
Signed-off-by: Adam Ford
---
drivers/cpu/imx8_cpu.c | 10 +-
1 file changed, 5 insertions(+), 5 dele
The imx8_cpu is capable of running on IMX8, IMX8M, and IMX9
families, but the CPU list is limited on the 8M variants. Expand
this list to show more variants and their respective names.
Signed-off-by: Adam Ford
---
drivers/cpu/imx8_cpu.c | 44 +++---
1 file
make imx8_cpu act like the previous
implementation for any CPU, make IMX9 imply IMX_TMU, so
it will be always displayed unless a user decides to
disable it.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx9/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-imx/imx9
On Mon, Mar 24, 2025 at 6:34 PM Fabio Estevam wrote:
>
> Hi Adam,
>
> On Mon, Mar 24, 2025 at 8:27 PM Adam Ford wrote:
> >
> > There is a config option to run the PMIC at nominal voltages
> > which is not enabled on the i.MX8MP Beacon kit, so it the PMIC
GIC clock to nominal
if the PMIC is in overdrive, and sets the GIC clock to overdrive
if the PMIC is in nominal. Fix this by inverting the logic on the
check.
Fixes: ab53bd43dbde ("arm64: imx: Add support for imx8mp-beacon-kit")
Signed-off-by: Adam Ford
diff --git a/board/beacon/imx8
On Sat, Mar 22, 2025 at 6:31 PM Marek Vasut wrote:
>
> On 3/22/25 7:38 PM, Adam Ford wrote:
> > Much of the data that is display by imx8_cpu.c is also displayed from
> > arch/arm/mach-imx/cpu.c, except the temperature grade and active
> > temperature are only displayed whe
On Sat, Mar 22, 2025 at 1:45 PM Fabio Estevam wrote:
>
> Hi Marek,
>
> On Wed, Mar 19, 2025 at 12:19 AM Marek Vasut wrote:
> >
> > Add clk_resolve_parent_clk() to resolve parent clock udevice name
> > based on clock-names DT property. This is used in SoC clock drivers
> > to look up the clock nam
On Sat, Mar 22, 2025 at 6:31 PM Marek Vasut wrote:
>
> On 3/22/25 7:38 PM, Adam Ford wrote:
> > The imx8_cpu driver is a CPU Driver that supports the i.MX8Q
> > family. When it is enabled, it acts as an alternative to
> > arch/arm/mach-imx/cpu.c, but the imx8_cpu suppo
the previous implementation
for any CPU, make IMX8M imply IMX_TMU so it will be always displayed
unless a user decides to disable it.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx8m/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-imx/imx8m/Kconfig b/arch/arm/mach
The imx8_cpu driver is a CPU Driver that supports the i.MX9
family. When it is enabled, it acts as an alternative to
arch/arm/mach-imx/cpu.c, but the imx8_cpu supports the driver
model where cpu.c does not. Imply this newer driver by default.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx9
U and CPU_IMX to IMX8, IMX8M and IMX9 families
V2: Changed the check in imx8_cpu to drop the IMX9 check and instead
imply IMX_TMU in the IMX9 family
Adam Ford (6):
imx: imx9: Imply IMX_TMU
imx: imx8m: Imply IMX_TMU
cpu: imx8_cpu: Print Speed grade if IMX_THERMAL or IMX_TMU
imx: imx8m:
The imx8_cpu driver is a CPU Driver that supports the i.MX8Q
family. When it is enabled, it acts as an alternative to
arch/arm/mach-imx/cpu.c, but the imx8_cpu supports the driver
model where cpu.c does not. Imply this newer driver by default.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx
The imx8_cpu driver is a CPU Driver that supports the i.MX8M
family, and when it is enabled, it acts as an alternative to
arch/arm/mach-imx/cpu.c, but the imx8_cpu supports the driver
model where cpu.c does not. Imply this newer driver by default.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx
(-40C to 105C) at 28C
Model: Beacon EmbeddedWorks i.MX8MPlus Development kit
Signed-off-by: Adam Ford
---
drivers/cpu/imx8_cpu.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/cpu/imx8_cpu.c b/drivers/cpu/imx8_cpu.c
index 53d31b3c0bf..faa6a0704a6 100644
--- a/driver
make imx8_cpu act like the previous
implementation for any CPU, make IMX9 imply IMX_TMU, so
it will be always displayed unless a user decides to
disable it.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx9/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/mach-imx/imx9
make imx8_cpu act like the previous implementation for any CPU, make IMX9
imply IMX_TMU so it will be always displayed unless a user decides to
disable it.
Signed-off-by: Adam Ford
---
arch/arm/mach-imx/imx9/Kconfig | 1 +
1 file changed, 1 insertion(+)
V2: New to series
diff --git a/arch/arm
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
---
configs/imx8mm_beacon_defconfig | 2 ++
1 file changed, 2 insertions(+)
V2: No Change
diff --git a/configs/imx8mm_beacon_defconfig b/configs
On Thu, Mar 20, 2025 at 8:55 PM Marek Vasut wrote:
>
> On 3/21/25 2:31 AM, Adam Ford wrote:
> > Enable CPU_IMX to register the ARM core and identify the
> > speed and temperature information using the driver model.
> >
> > Signed-off-by: Adam Ford
> I am tempted
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
---
configs/imx8mp_beacon_defconfig | 2 ++
1 file changed, 2 insertions(+)
V2: No Change
diff --git a/configs/imx8mp_beacon_defconfig b/configs
On Wed, Mar 19, 2025 at 12:27 PM Fabio Estevam wrote:
>
> On Wed, Mar 19, 2025 at 1:39 PM Adam Ford wrote:
>
> > Sorry about all the noise.
> >
> > I think I forgot to type 'git add' before the "git commit --amend" so
> > the fix didn'
On Wed, Mar 19, 2025 at 6:42 AM Fabio Estevam wrote:
>
> On Wednesday, March 19, 2025 00:23 -03, Marek Vasut wrote:
>
> > Sigh, get maintainers missed Fabio, sorry, +CCed now.
I noticed that Fabio was being left off too when I sent some stuff.
adam
> >
> > Fabio, do you want me to resend this w
On Wed, Mar 19, 2025 at 11:15 AM Fabio Estevam wrote:
>
> On Tue, Mar 18, 2025 at 8:42 PM Adam Ford wrote:
>
> > + if (of_machine_is_compatible("fsl,imx6qp")
>
> Missing closing parenthesis.
>
> > - clk_dm(IMX6QDL_CLK_ECSPI_ROOT,
> &
MX8MP Rev1.1 A53 at 1200 MHz
Model: Beacon EmbeddedWorks i.MX8MPlus Development kit
After:
CPU: NXP i.MX8MP Rev1.1 A53 at 1200 MHz
CPU: Industrial temperature grade (-40C to 105C) at 28C
Model: Beacon EmbeddedWorks i.MX8MPlus Development kit
Signed-off-by: Adam Ford
diff --git a/drivers/cpu/imx8_
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mn_beacon_defconfig b/configs/imx8mn_beacon_defconfig
index 69af7cee092..93b32155aa9 100644
--- a/configs/imx8mn_beacon_defconfig
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mp_beacon_defconfig b/configs/imx8mp_beacon_defconfig
index 9709f628f9c..2893270d0c1 100644
--- a/configs/imx8mp_beacon_defconfig
Enable CPU_IMX to register the ARM core and identify the
speed and temperature information using the driver model.
Signed-off-by: Adam Ford
diff --git a/configs/imx8mm_beacon_defconfig b/configs/imx8mm_beacon_defconfig
index 56e18893b77..bcb60a63795 100644
--- a/configs/imx8mm_beacon_defconfig
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
drivers/clk/imx/clk-imx8mm.c | 34 ++
1 file changed, 34 insertions(+)
V3: No change
V2: No Change
diff --git a
The ECSPI clock has the ability to select between pll3_60m and
osc on the imx6qp, where it's fixed on other variants. Fix this
by adding using a helper function to determine SoC variant and
register the clock accordingly.
Signed-off-by: Adam Ford
Reviewed-by: Michael Trimarchi
Review
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
board/beacon/imx8mn/spl.c | 2 --
1 file changed, 2 deletions(-)
V3: No Change
V2: No change
diff --git a/board/beacon/imx8mn/spl.c b/board/beacon
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
drivers/clk/imx/clk-imx8mn.c | 32
1 file changed, 32 insertions(+)
V3: No change
V2: No Change
diff --git a
,
imx8m[mnqp], several imxrt, imx9. If/when the UART clock
registration is done for older boards, this limitation
could be updated.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
drivers/serial/serial_mxc.c | 11 +++
include/dm/platform_data/serial_mxc.h | 3 +++
2 files
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
board/beacon/imx8mp/spl.c | 2 --
1 file changed, 2 deletions(-)
V3: No change
V2: No change
diff --git a/board/beacon/imx8mp/spl.c b/board/beacon
In preparation to remove manual references for enabling some clocks,
enable SPL_CLK_IMX8MP which automatically enables SPL_CCF and
SPL_CLK_COMPOSITE_CCF which permit various drivers to activate
their respective clocks automatically.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
configs
If SPL_CLK_IMX8MP is selected alone, it causes a build error.
The clock composite is required when using the clock framework, so
select it when SPL_CLK_IMX8MP is enabled. This is already being
done outside of SPL.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
drivers/clk/imx/Kconfig | 1
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable. This requires a small re-order
of a couple functions.
Signed-off-by: Adam Ford
Reviewed-by: Peng Fan
---
board/beacon/imx8mm/spl.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
V3: No
In order to use the driver model and clock system to enable UART
clocks from the serial driver, it's necessary to register the UART
clocks. With the helper function to check for imx6qp vs other
variants, the UART can register for both scenarios.
Signed-off-by: Adam Ford
Reviewed-by: Pen
ot;clk dump" and "dm tree" show the UART clocks are
registred and the one clocking the console UART is enabled and running.
Adam Ford (10):
clk: imx6q: Properly handle imx6qp ESPI clk_sels
clk: imx6q: Register UART clocks
clk: imx8mm: register UART clocks
clk: imx8mn: register
On Sat, Mar 15, 2025 at 11:17 AM Fabio Estevam wrote:
>
> Hi Adam,
>
> On Sat, Mar 15, 2025 at 12:37 PM Adam Ford wrote:
>
> > I ran the config that failed on my V3, and it passes that build test,
> > but I don't want to waste your time if it fails on others
On Tue, Mar 11, 2025 at 9:31 PM Fabio Estevam wrote:
>
> On Tue, Mar 11, 2025 at 9:31 PM Adam Ford wrote:
>
> > ---
> > V2: Fix name of "ECSPI" and elimiate helper function, and fix build error.
>
> Still get errors in CI:
>
> https://source.denx.de/
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
---
drivers/clk/imx/clk-imx8mm.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/drivers/clk/imx/clk-imx8mm.c b/drivers/clk/imx/clk
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
---
board/beacon/imx8mp/spl.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/board/beacon/imx8mp/spl.c b/board/beacon/imx8mp/spl.c
index 30d577f7e0e..6b357d90a3f 100644
In preparation to remove manual references for enabling some clocks,
enable SPL_CLK_IMX8MP which automatically enables SPL_CCF and
SPL_CLK_COMPOSITE_CCF which permit various drivers to activate
their respective clocks automatically.
Signed-off-by: Adam Ford
---
configs/imx8mp_beacon_defconfig
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
---
board/beacon/imx8mn/spl.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/board/beacon/imx8mn/spl.c b/board/beacon/imx8mn/spl.c
index f03841e5a01..e91d3fdcf5e 100644
If SPL_CLK_IMX8MP is selected alone, it causes a build error.
The clock composite is required when using the clock framework, so
select it when SPL_CLK_IMX8MP is enabled. This is already being
done outside of SPL.
Signed-off-by: Adam Ford
---
drivers/clk/imx/Kconfig | 1 +
1 file changed, 1
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable. This requires a small re-order
of a couple functions.
Signed-off-by: Adam Ford
---
board/beacon/imx8mm/spl.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/board/beacon/imx8mm
,
imx8m[mnqp], several imxrt, imx9. If/when the UART clock
registration is done for older boards, this limitation
could be updated.
Signed-off-by: Adam Ford
---
drivers/serial/serial_mxc.c | 11 +++
include/dm/platform_data/serial_mxc.h | 1 +
2 files changed, 12 insertions
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
---
drivers/clk/imx/clk-imx8mn.c | 32
1 file changed, 32 insertions(+)
diff --git a/drivers/clk/imx/clk-imx8mn.c b/drivers/clk/imx/clk
In order to use the driver model and clock system to enable UART
clocks from the serial driver, it's necessary to register the UART
clocks. With the helper function to check for imx6qp vs other
variants, the UART can register for both scenarios.
Signed-off-by: Adam Ford
---
V2: Directly
The ECSPI clock has the ability to select between pll3_60m and
osc on the imx6qp, where it's fixed on other variants. Fix this
by adding using a helper function to determine SoC variant and
register the clock accordingly.
Signed-off-by: Adam Ford
Reviewed-by: Michael Trimarchi
---
driver
ot;clk dump" and "dm tree" show the UART clocks are
registred and the one clocking the console UART is enabled and running.
Adam Ford (10):
clk: imx6q: Properly handle imx6qp ESPI clk_sels
clk: imx6q: Register UART clocks
clk: imx8mm: register UART clocks
clk: imx8mn: reg
On Thu, Mar 6, 2025 at 9:57 AM Fabio Estevam wrote:
>
> On Sun, Mar 2, 2025 at 1:53 PM Adam Ford wrote:
> >
> > The ECSPI clock has the ability to select between pll3_60m and
> > osc on the imx6qp, where it's fixed on other variants. Fix this
> > by adding u
On Thu, Mar 6, 2025 at 9:55 AM Fabio Estevam wrote:
>
> On Sun, Mar 2, 2025 at 1:53 PM Adam Ford wrote:
>
> > @@ -312,7 +313,17 @@ int mxc_serial_setbrg(struct udevice *dev, int
> > baudrate)
> > static int mxc_serial_probe(struct udevice *dev)
> > {
> &g
On Mon, Mar 3, 2025 at 6:34 AM Michael Nazzareno Trimarchi
wrote:
>
> Hi Adam
>
> On Sun, Mar 2, 2025 at 5:53 PM Adam Ford wrote:
>>
>> The ECSPI clock has the ability to select between pll3_60m and
>> osc on the imx6qp, where it's fixed on other variants. F
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
---
drivers/clk/imx/clk-imx8mm.c | 34 ++
1 file changed, 34 insertions(+)
diff --git a/drivers/clk/imx/clk-imx8mm.c b/drivers/clk/imx/clk
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
---
board/beacon/imx8mp/spl.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/board/beacon/imx8mp/spl.c b/board/beacon/imx8mp/spl.c
index 30d577f7e0e..6b357d90a3f 100644
In order to use the driver model and clock system to enable UART
clocks from the serial driver, it's necessary to register the UART
clocks. With the helper function to check for imx6qp vs other
variants, the UART can register for both scenarios.
Signed-off-by: Adam Ford
---
drivers/clk/im
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable.
Signed-off-by: Adam Ford
---
board/beacon/imx8mn/spl.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/board/beacon/imx8mn/spl.c b/board/beacon/imx8mn/spl.c
index f03841e5a01..e91d3fdcf5e 100644
The ECSPI clock has the ability to select between pll3_60m and
osc on the imx6qp, where it's fixed on other variants. Fix this
by adding using a helper function to determine SoC variant and
register the clock accordingly.
Signed-off-by: Adam Ford
---
drivers/clk/imx/clk-imx6q.c
ot;clk dump" and "dm tree" show the UART clocks are
registred and the one clocking the console UART is enabled and running.
Adam Ford (10):
clk: imx6q: Properly handle imx6qp ESPI clk_sels
clk: imx6q: Register UART clocks
clk: imx8mm: register UART clocks
clk: imx8mn: reg
,
imx8m[mnqp], several imxrt, imx9. If/when the UART clock
registration is done for older boards, this limitation
could be updated.
Signed-off-by: Adam Ford
---
drivers/serial/serial_mxc.c | 11 +++
include/dm/platform_data/serial_mxc.h | 1 +
2 files changed, 12 insertions
In preparation to remove manual references for enabling some clocks,
enable SPL_CLK_IMX8MP which automatically enables SPL_CCF and
SPL_CLK_COMPOSITE_CCF which permit various drivers to activate
their respective clocks automatically.
Signed-off-by: Adam Ford
---
configs/imx8mp_beacon_defconfig
Now that the UART driver can enable the required clocks, remove
the hard-coded clock enable. This requires a small re-order
of a couple functions.
Signed-off-by: Adam Ford
---
board/beacon/imx8mm/spl.c | 6 +-
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/board/beacon/imx8mm
If SPL_CLK_IMX8MP is selected alone, it causes a build error.
The clock composite is required when using the clock framework, so
select it when SPL_CLK_IMX8MP is enabled. This is already being
done outside of SPL.
Signed-off-by: Adam Ford
---
drivers/clk/imx/Kconfig | 1 +
1 file changed, 1
In order to let the serial driver enable the clocks, the UART clocks
must be registered first.
Signed-off-by: Adam Ford
---
drivers/clk/imx/clk-imx8mn.c | 32
1 file changed, 32 insertions(+)
diff --git a/drivers/clk/imx/clk-imx8mn.c b/drivers/clk/imx/clk
ak boards where this function is
> empty.
Isn't there a config option to enable/disable board_early_init_f? If
they are do-nothing functions, can the config option be disabled?
adam
>
> Signed-off-by: Marek Vasut
> ---
> Cc: Adam Ford
> Cc: Biju Das
> Cc: Lad Prabh
On Fri, Nov 22, 2024 at 8:33 AM Simon Glass wrote:
>
> Hi Marek,
>
> On Thu, 21 Nov 2024 at 18:59, Marek Vasut wrote:
> >
> > Since 50cdd3f74af3 ("clk: imx: clk-imx8mn Fix nand and spi clock parent"),
> > drivers/clk/imx/clk-imx8mn.c clk_get_by_name(dev, "osc_24m", &osc_24m_clk)
> > fails with er
1 - 100 of 1163 matches
Mail list logo
