Skip to content

frdm-kl43z: NXP Kinetis FRDM-KL43Z development board#9012

Merged
benpicco merged 6 commits intoRIOT-OS:masterfrom
jnohlgard:pr/frdm-kl43z
Oct 25, 2020
Merged

frdm-kl43z: NXP Kinetis FRDM-KL43Z development board#9012
benpicco merged 6 commits intoRIOT-OS:masterfrom
jnohlgard:pr/frdm-kl43z

Conversation

@jnohlgard
Copy link
Copy Markdown
Member

@jnohlgard jnohlgard commented Apr 24, 2018 •
edited
Loading

Contribution description

Add initial CPU support for NXP Kinetis KL43Z and an initial configuration for the FRDM-KL43Z development board.

This PR lays the foundation for the KL series support and will enable easily adding support for other Kinetis L based boards.

What works?

  • Clock configuration (MCG_Lite), 2 MHz, 8 MHz, 48 MHz, external clock
  • UART/LPUART
  • GPIO
  • I2C
  • Timers (RTC/RTT, PIT, LPTMR)
  • MMA8451 accelerometer
  • MAG3110 magnetometer

What is missing?

  • SPI (different hardware module than used in the other supported Kinetis CPUs)
  • PWM
  • Segment LCD driver

Issues/PRs references

#7154

@jnohlgard jnohlgard added Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: new feature The issue requests / The PR implemements a new feature for RIOT CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Area: boards Area: Board ports labels Apr 24, 2018
@jnohlgard jnohlgard added this to the Release 2018.07 milestone Apr 24, 2018
@jnohlgard
Copy link
Copy Markdown
Member Author

jnohlgard commented Apr 24, 2018 •
edited
Loading

To be able to flash this board, the on board debugger must be updated to a J-Link or CMSIS-DAP firmware. The factory programmed firmware is a P&E debugger which we don't have a configuration for.
Follow the instructions at https://www.nxp.com/support/developer-resources/run-time-software/kinetis-developer-resources/ides-for-kinetis-mcus/opensda-serial-and-debug-adapter:OPENSDA for updating the debugger firmware.
I recommend using CMSIS-DAP, it seems to work well and it is open source.

@jnohlgard jnohlgard force-pushed the pr/frdm-kl43z branch 3 times, most recently from 28089ce to 22227ce Compare April 25, 2018 15:47
@jnohlgard jnohlgard mentioned this pull request Apr 30, 2018
Closed
@jnohlgard jnohlgard mentioned this pull request Jun 26, 2018
Closed
@jnohlgard jnohlgard force-pushed the pr/frdm-kl43z branch 2 times, most recently from 6e279cf to 1dd41b7 Compare October 26, 2018 17:29
@jnohlgard
Copy link
Copy Markdown
Member Author

jnohlgard commented Oct 26, 2018

Rebased and updated configuration to match upstream changes. Added configuration for onboard inertial sensors (MAG3110 and MMA8451Q).

Tested:

  • UART
  • sensors
  • LEDs
  • button (SW3, via SAUL)
  • timers
  • xtimer

Not tested:

  • ADC

@jnohlgard jnohlgard mentioned this pull request Nov 9, 2018
Merged
@jnohlgard jnohlgard mentioned this pull request Nov 27, 2018
Merged
@jnohlgard jnohlgard force-pushed the pr/frdm-kl43z branch 2 times, most recently from bc75056 to f87b1ad Compare November 29, 2018 13:25
@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Jan 17, 2020
@benpicco
Copy link
Copy Markdown
Contributor

benpicco commented Jan 17, 2020

Welp, I think we should still be able to get this in - if you're still interested, let's start with a rebase

@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Jan 17, 2020
@stale
Copy link
Copy Markdown

stale bot commented Jul 20, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you want me to ignore this issue, please mark it with the "State: don't stale" label. Thank you for your contributions.

@stale stale bot added the State: stale State: The issue / PR has no activity for >185 days label Jul 20, 2020
@stale stale bot closed this Aug 21, 2020
@aabadie aabadie reopened this Aug 22, 2020
@stale stale bot removed the State: stale State: The issue / PR has no activity for >185 days label Aug 22, 2020
@benpicco
Copy link
Copy Markdown
Contributor

benpicco commented Oct 24, 2020

No regressions on the openlabs-kw41z-mini.

Joakim Nohlgård added 5 commits October 24, 2020 22:08
@benpicco
kinetis: Add KL RAM sizes to kinetis-info.mk
945bf11 
Gives correct information for all KL parts, according to the product
catalog on nxp.com
@benpicco
kinetis: Add support for combined GPIO PORTC,D IRQs
9ea0cec 
Used in KL43 and others
@benpicco
kinetis: Add KL43Z vendor file
1a724f4
@benpicco
kinetis: Add KL43Z support
238f91f
@benpicco
kinetis: Add support for MCG_Lite hardware
dc84ccd 
MCG_Lite is used in many KL parts and is a less advanced clock generator
than the full MCG used in the K series. This change lets the MCG_Lite
and MCG share the same user facing API, with some configuration
differences.
@benpicco benpicco merged commit d0521af into RIOT-OS:master Oct 25, 2020
@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Oct 25, 2020

This was merged before I could finish my comment below!

No regressions on the openlabs-kw41z-mini.

Ok, but do we know if this particular board and its provided features work ? Do you have the board to test @benpicco. It would be nice to have some test results before merging.

@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Oct 25, 2020 •
edited
Loading

I really wanted to block this PR until we get at least the test results from someone. Now there's no guarantee that things are working on that board. And sorry @benpicco, but I also don't like that you clearly took over this PR from someone else and kind of auto-ACKed yourself. It's good to have things in fast(er) but we at least need a minimum of testing before merging, especially for boards.

@jnohlgard jnohlgard deleted the pr/frdm-kl43z branch January 11, 2021 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@benpicco benpicco benpicco approved these changes

@fjmolinas fjmolinas Awaiting requested review from fjmolinas

@MrKevinWeiss MrKevinWeiss Awaiting requested review from MrKevinWeiss MrKevinWeiss is a code owner

@cgundogan cgundogan Awaiting requested review from cgundogan

@jia200x jia200x Awaiting requested review from jia200x jia200x is a code owner

@leandrolanzieri leandrolanzieri Awaiting requested review from leandrolanzieri leandrolanzieri is a code owner

@miri64 miri64 Awaiting requested review from miri64 miri64 is a code owner

Assignees

No one assigned

Labels

Area: boards Area: Board ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: new feature The issue requests / The PR implemements a new feature for RIOT

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@jnohlgard @benpicco @aabadie @cladmi @miri64 @jia200x @MrKevinWeiss