Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[5.10-2.1.x-imx] Two fixes for minor issues observed on i.MX7 #510

Merged

Conversation

tq-schifferm
Copy link

These seem to be only cosmetic issues, leaving warnings and error messages in the kernel log:

  • fsl_sai printing errors for DTS without mclk0 (which are all SoC older than i.MX8). Fixed by replacing a linux-imx change with a mainline backport.
  • Duplicate ARM timer node on i.MX7D caused by unnecessary change in i.MX7S DTSI, fixed by partial revert.

tq-schifferm and others added 3 commits December 7, 2021 15:08
The timer node was moved from / to /soc by a previous commit, resulting
in the imx7d DTSI duplicating rather than amending the imx7s definition
of the timer. This causes the following warning in the kernel log:

    arch_timer: multiple nodes in dt, skipping

We also revert the CPU mask from 4 CPUs to 1 (imx7d will modify it to
2 CPUs), as having a value of 4 on a single-core SoC seems questionable;
I haven't found any documentation of the purpose of this mask though.

Fixes: a12e072 ("ARM: dts: imx7d: add low power support")
Signed-off-by: Matthias Schiffer <[email protected]>
On VF610, mclk0 = bus_clk;
On i.MX6SX/6UL/6ULL/7D, mclk0 = mclk1;
On i.MX7ULP, mclk0 = bus_clk;
On i.MX8QM/8QXP, mclk0 = bus_clk;
On i.MX8MQ/8MN/8MM/8MP, mclk0 = bus_clk;

So add variable mclk0_is_mclk1 in fsl_sai_soc_data to
distinguish these platforms.

Signed-off-by: Shengjiu Wang <[email protected]>
Acked-by: Nicolin Chen <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Mark Brown <[email protected]>
(cherry picked from commit 53233e4)
[Matthias Schiffer: adjusted for linux-imx 5.10]
Signed-off-by: Matthias Schiffer <[email protected]>
@otavio otavio merged commit 97b03e7 into Freescale:5.10-2.1.x-imx Dec 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants