Skip to content

UPSTREAM-PEND: backport flash driver support for MSPM0 - #60

Draft
Sunil-Hegde wants to merge 13 commits into
TexasInstruments:v4.4.0-ti-nextfrom
Sunil-Hegde:mspm0_flash_downstream
Draft

UPSTREAM-PEND: backport flash driver support for MSPM0#60
Sunil-Hegde wants to merge 13 commits into
TexasInstruments:v4.4.0-ti-nextfrom
Sunil-Hegde:mspm0_flash_downstream

Conversation

@Sunil-Hegde

@Sunil-Hegde Sunil-Hegde commented Oct 1, 2026 •
edited
Loading

Copy link
Copy Markdown

This is not merged upstream and is based on HAL. Added TI specific commits addressing the open reviews.

pr: 91500

lp_mspm0g3519, tests/drivers/flash/common

*** Booting Zephyr OS build v4.4.0-80-g0c2acf5ab176 ***
Running TESTSUITE flash_driver
===================================================================
START - test_flash_copy
Test will run on device flash-controller@400cd000
 PASS - test_flash_copy in 0.034 seconds
===================================================================
START - test_flash_erase
Test will run on device flash-controller@400cd000
 PASS - test_flash_erase in 0.014 seconds
===================================================================
START - test_flash_fill
Test will run on device flash-controller@400cd000
 PASS - test_flash_fill in 0.018 seconds
===================================================================
START - test_flash_flatten
Test will run on device flash-controller@400cd000
 PASS - test_flash_flatten in 0.021 seconds
===================================================================
START - test_flash_page_layout
Test will run on device flash-controller@400cd000
start_offset=0x70000	size=1024	index=448
page_count=512
 PASS - test_flash_page_layout in 0.013 seconds
===================================================================
START - test_flash_write_read_over_the_packet_limit
Test will run on device flash-controller@400cd000
Given bus controller does not have 'packet_data_limit' property
 SKIP - test_flash_write_read_over_the_packet_limit in 0.011 seconds
===================================================================
START - test_get_size
Test will run on device flash-controller@400cd000
 SKIP - test_get_size in 0.005 seconds
===================================================================
START - test_read_unaligned_address
Test will run on device flash-controller@400cd000
 PASS - test_read_unaligned_address in 0.013 seconds
===================================================================
START - test_supply_gpios_control
Test will run on device flash-controller@400cd000
 SKIP - test_supply_gpios_control in 0.008 seconds
===================================================================
TESTSUITE flash_driver succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [flash_driver]: pass = 6, fail = 0, skip = 3, total = 9 duration = 0.137 seconds
 - PASS - [flash_driver.test_flash_copy] duration = 0.034 seconds
 - PASS - [flash_driver.test_flash_erase] duration = 0.014 seconds
 - PASS - [flash_driver.test_flash_fill] duration = 0.018 seconds
 - PASS - [flash_driver.test_flash_flatten] duration = 0.021 seconds
 - PASS - [flash_driver.test_flash_page_layout] duration = 0.013 seconds
 - SKIP - [flash_driver.test_flash_write_read_over_the_packet_limit] duration = 0.011 seconds
 - SKIP - [flash_driver.test_get_size] duration = 0.005 seconds
 - PASS - [flash_driver.test_read_unaligned_address] duration = 0.013 seconds
 - SKIP - [flash_driver.test_supply_gpios_control] duration = 0.008 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

tests/drivers/flash_api:

*** Boot*** Booting Zephyr OS build v4.4.0-81-gfd7a35affa6b ***
Running TESTSUITE flash_api
===================================================================
START - test_get_size
 PASS - test_get_size in 0.001 seconds
===================================================================
TESTSUITE flash_api succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [flash_api]: pass = 1, fail = 0, skip = 0, total = 1 duration = 0.001 seconds
 - PASS - [flash_api.test_get_size] duration = 0.001 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

I have tested erase blocks and negative tests as well but logs are too big to paste here.

@Sunil-Hegde
Sunil-Hegde force-pushed the mspm0_flash_downstream branch 2 times, most recently from 427a605 to fd7a35a Compare October 5, 2026 08:24
@Aman-Lachhiramka-ti

Aman-Lachhiramka-ti commented Oct 5, 2026

Copy link
Copy Markdown

Better to add some tests and samples hardware runtime logs in this PR as this is from linumiz

@Sunil-Hegde
Sunil-Hegde force-pushed the mspm0_flash_downstream branch from fd7a35a to 64cc985 Compare October 6, 2026 08:45
karthi012 and others added 13 commits October 7, 2026 14:48
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: dts: bindings: flash_controller: add ti mspm0 flash co…
8f71c7f 
…ntr...

PR: 91500

Add Flash Controller binding for Texas Instruments MSPM0 Family Series

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
Signed-off-by: Jackson Farley <j-farley@ti.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: drivers: flash: add ti mspm0 flash driver support
66e0a39 
PR: 91500

Add Flash driver for Texas Instruments MSPM0 Family Series
Currently driver supports only read,write,erase operations on
non-volatile flash MAIN memory region.

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
Signed-off-by: Jackson Farley <j-farley@ti.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: dts: arm: ti: mspm0: add support for flash controller
695d7d8 
PR: 91500

Add Flash controller support for MSPM0 and move non-volatile
flash memory under flash controller, bcoz it accessed via
flash controller driver

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: dts: arm: ti: mspm0: g: fix flash0 node
413e525 
PR: 91500

flash0 is moved to flash controller node bcoz non-volatile flash
memory is accessed through flash controller driver, so fixing it
for MSPM0 G series

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: dts: arm: ti: mspm0: l: fix flash0 node
4899141 
PR: 91500

flash0 is moved to flash controller node bcoz non-volatile flash
memory is accessed through flash controller driver, so fixing for
MSPM0 L series.

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: boards: ti: lp_mspm0g3507: add flash controller chosen…
3777583 
… pr...

PR: 91500

add flash controller chosen property to access non-volatile
flash memory for zephyr,flash chosen property node and adapt
partition layout to accomadate storage partition

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: boards: ti: lp_mspm0l2228: add flash controller chosen…
8e3ce49 
… pr...

PR: 91500

add flash controller chosen property to access non-volatile
flash memory for zephyr,flash chosen property node and adapt
partition layout to accomadate storage partition

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@karthi012 @Sunil-Hegde
UPSTREAM-PEND: dts: arm: ti: mspm0: fix License Copyright CI warning
11ad3d3 
PR: 91500

Copyright header is missing in these dtsi files, which
makes CI Compilance warning, so fixing it

Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@linumiz.com>
@Sunil-Hegde
TI: drivers: flash: flash_soc_ti_mspm0: Status check post any operation
8d09c4f 
Add status check post erase/write operation in-order to insure
interrupt was fired for SUCCESS. DONE interrupt is fired
for both SUCCESS and FAILURE.

Signed-off-by: Sunil Hegde <s-hegde3@ti.com>

---
Reason: Addressing review comments on pr 91500
@Sunil-Hegde
TI: drivers: flash: mspm0: Add executeClearStatus before unprotectSector
c55bfa3 
Driverlib suggests using DL_FlashCTL_executeClearStatus before
DL_FlashCTL_unprotectSector. Add the same in erase and write
APIs.

Signed-off-by: Sunil Hegde <s-hegde3@ti.com>

---
Reason: Review suggestions on pr 91500
@Sunil-Hegde
TI: drivers: flash: ti_mspm0: Account for non-zero base address
9ac3240 
Erase and Write use the offset assuming base address is 0
which is not the case on MSPM33. Use base_address_offset
in write and erase APIs same as read.

Signed-off-by: Sunil Hegde <s-hegde3@ti.com>

---
Reason: Review comments on pr 91500
@Sunil-Hegde
TI: drivers: flash: ti_mspm0: change valid_range chack for readability
010610e 
Change the valid_range check in if condition from comparing with 0
to not operation for readability.

Signed-off-by: Sunil Hegde <s-hegde3@ti.com>

---
Reason: Review comments on pr 91500
@Sunil-Hegde
TI: boards: ti: lp_mspm0g3519: Add flash node for g3519
0f6dc0a 
Add flash node and mark partitions for boot, image and storage
for mspm0g3519.

Signed-off-by: Sunil Hegde <s-hegde3@ti.com>

---
Reason: Add additional board support apart from the
original PR adds.
@Sunil-Hegde
Sunil-Hegde force-pushed the mspm0_flash_downstream branch from 64cc985 to 0f6dc0a Compare October 7, 2026 09:19
@Aman-Lachhiramka-ti

Aman-Lachhiramka-ti commented Oct 7, 2026 •
edited
Loading

Copy link
Copy Markdown

zephyrproject-rtos#115203

Check the upstream merged PR of updating zephyr partitioning happening again in this PR

@Sunil-Hegde
Sunil-Hegde marked this pull request as draft October 8, 2026 07:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Aman-Lachhiramka-ti Aman-Lachhiramka-ti Awaiting requested review from Aman-Lachhiramka-ti
@soumya-TI soumya-TI Awaiting requested review from soumya-TI
@natto1784 natto1784 Awaiting requested review from natto1784

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Sunil-Hegde @Aman-Lachhiramka-ti @karthi012