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

Add/fix flash controller nodes for NXP kinetis SoCs #5778

Closed
galak opened this issue Jan 22, 2018 · 3 comments
Closed

Add/fix flash controller nodes for NXP kinetis SoCs #5778

galak opened this issue Jan 22, 2018 · 3 comments
Assignees
Labels
area: ARM ARM (32-bit) Architecture area: Devicetree area: Flash bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP

Comments

@galak
Copy link
Collaborator

galak commented Jan 22, 2018

Its looks like the only flash controller node that might be correct is dts/arm/nxp/nxp_k6x.dtsi.

Several dtsi don't have flash controller nodes at all:

  • dts/arm/nxp/nxp_kl25z.dtsi
  • dts/arm/nxp/nxp_kw40z.dtsi
  • dts/arm/nxp/nxp_kw41z.dtsi

And dts/arm/nxp/nxp_kw2xd.dtsi has copied the k64 node.

Need to update all these .dtsi to have proper flash controller nodes.

@galak
Copy link
Collaborator Author

galak commented Jan 25, 2018

It looks like kw2xd is probably reasonable to be copied from k64. Only difference is in # of cache tags & cache data tags.

@galak
Copy link
Collaborator Author

galak commented Jan 25, 2018

Added write-block-size to all dt's based on FSL_FEATURE_FLASH_PFLASH_BLOCK_WRITE_UNIT_SIZE. This is in PR #5830.

@nashif nashif added the bug The issue is a bug, or the PR is fixing a bug label Feb 6, 2018
@nashif
Copy link
Member

nashif commented Feb 6, 2018

fixed

@nashif nashif closed this as completed Feb 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: ARM ARM (32-bit) Architecture area: Devicetree area: Flash bug The issue is a bug, or the PR is fixing a bug platform: NXP NXP
Projects
None yet
Development

No branches or pull requests

3 participants