Skip to content

stm32/flashpage: use void pointer for flash address#15410

Merged
aabadie merged 1 commit intoRIOT-OS:masterfrom
bergzand:pr/stm32/flashpage_voidptr
Nov 9, 2020
Merged

stm32/flashpage: use void pointer for flash address#15410
aabadie merged 1 commit intoRIOT-OS:masterfrom
bergzand:pr/stm32/flashpage_voidptr

Conversation

@bergzand
Copy link
Copy Markdown
Member

@bergzand bergzand commented Nov 9, 2020

Contribution description

The stm32 flashpage implementation has a number of ifdefs to assign different pointer types to store the flashpage address. This address is only used as void pointers in the called functions. This PR removes the ifdefs with the different pointer sizes, replacing it with a single line using a void pointer.

Testing procedure

Compilation should be enough. Testing can be done using tests/periph_flashpage on the various stm32 devices.

Issues/PRs references

None

@bergzand bergzand added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 9, 2020
@bergzand bergzand requested review from aabadie and fjmolinas November 9, 2020 13:35
@aabadie
Copy link
Copy Markdown
Contributor

aabadie commented Nov 9, 2020

Tested tests/periph_flashpage and tests/mtd_flashpage on iotlab-m3 (stm32f1), b-l072z-lrwan1 (stm32l0), nucleo-l152re, nucleo-l412kb (l4) and nucleo-l552ze-q): all are successful.

@aabadie aabadie merged commit 0031b14 into RIOT-OS:master Nov 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: drivers Area: Device drivers CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants