Skip to content

Conversation

multiplemonomials
Copy link
Collaborator

@multiplemonomials multiplemonomials commented Sep 8, 2025

Summary of changes

I noticed that I made a mistake in #433 : I got confused about #ifdefs and thought that a given block of code was for KL46Z, when in fact it has been for KL43Z, which is a target that no longer exists in Mbed. This was then causing KL46Z to fail to compile, because it was compiling code for the wrong CPU!

As it turns out, the code in clk_freqs.h and clk_freqs.c is for both KL25Z and KL46Z!

Impact of changes

  • Build fixed for KL46Z

Migration actions required

Documentation

None


Pull request type

[X] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[X] Tests / results supplied as part of this PR

Able to compile all of Mbed for KL46Z now. Cannot test on hardware as I do not have one of these boards.


@multiplemonomials multiplemonomials merged commit bc8fc2f into master Sep 9, 2025
53 checks passed
@multiplemonomials multiplemonomials deleted the bugfix/kl46z-build branch September 9, 2025 02:54
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.

2 participants