Enable Cadence QSPI Driver #46443
Merged
Enable Cadence QSPI Driver #46443
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ngboonkhai
requested review from
galak,
carlocaione,
nashif and
nvlsianpu
as code owners
zephyrbot
added
area: Devicetree
area: Flash
area: ARM64
galak
reviewed
Jun 10, 2022
carlocaione
requested changes
Jun 11, 2022
SgrrZhf
reviewed
Jun 13, 2022
nvlsianpu
reviewed
Jun 13, 2022
ngboonkhai
force-pushed
the
agilex_qspi_bringup
branch
6 times, most recently
from
db69362 to
0c9ced4
Compare
cfriedt
reviewed
Jun 22, 2022
ngboonkhai
requested review from
nashif,
carlocaione,
XenuIsWatching and
keith-packard
and removed request for
XenuIsWatching
|
Hi @nashif @carlocaione resolved all comments, ci all green, please approve. |
|
Still need to resolve the license question with TSC as this being BSD based. |
stephanosio
reviewed
Aug 31, 2022
This has been approved by the TSC already. |
ngboonkhai
force-pushed
the
agilex_qspi_bringup
branch
from
28f53cc to
d64821b
Compare
#46444 |
|
Hi @stephanosio @galak @nashif @carlocaione resolved all comments, ci all green. |
This patch is to increase kernel VM size to 1.5MB and memory size to 2MB to ready for future driver bring up. Signed-off-by: Boon Khai Ng <[email protected]>
…ment This patch is to add Cadence QSPI NOR flash device tree element for the Cadence QSPI NOR flash driver bringup Signed-off-by: Boon Khai Ng <[email protected]>
This patch is to enable new driver, Cadence QSPI NOR flash for Intel SoC FPGA Agilex Family Signed-off-by: Boon Khai Ng <[email protected]>
Add code owner for Cadence QSPI NOR Flash Driver. Signed-off-by: Boon Khai Ng <[email protected]>
This patch is to enable QSPI at Intel SoC FPGA Agilex Board Signed-off-by: Boon Khai Ng <[email protected]>
ngboonkhai
force-pushed
the
agilex_qspi_bringup
branch
from
d64821b to
bfd6545
Compare
|
Having conflict with Kconfig and CMakelist.txt, rebased to main branch. |
|
Hi @stephanosio @galak @nashif @carlocaione ci all green after rebase, please review. |
carlocaione
approved these changes
Sep 1, 2022
stephanosio
approved these changes
Sep 1, 2022
nashif
approved these changes
Sep 1, 2022
|
@nvlsianpu can you please take another look? |
nvlsianpu
approved these changes
Sep 1, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is to enable the Cadence QSPI Driver
The low level driver code is leverage from Trusted-Firmware-A (BSD-3 Clause)
https://github.com/ARM-software/arm-trusted-firmware
Submited an Issue for the Zephyr TSC to review on the Trusted-Firmware-A licensing.
#46444