Skip to content

drivers: ospi: hal: add HyperBus Initialization - #122

Open
GaneshRamani-AlifSemi wants to merge 1 commit into
alifsemi:mainfrom
GaneshRamani-AlifSemi:hyperbus_init
Open

drivers: ospi: hal: add HyperBus Initialization#122
GaneshRamani-AlifSemi wants to merge 1 commit into
alifsemi:mainfrom
GaneshRamani-AlifSemi:hyperbus_init

Conversation

@GaneshRamani-AlifSemi

Copy link
Copy Markdown
Contributor

update api to enable hyperbus settings in OSPI controllor.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the OSPI HyperBus XiP initialization API and expands the initialization sequence to program controller/XiP registers using a shared ospi_xip_config structure (including wait cycles and, on some SoCs, XiP CS selection).

Changes:

  • Changes ospi_hyperbus_xip_init() to take struct ospi_xip_config *xip_cfg instead of discrete parameters.
  • Adds explicit OSPI_CTRLR0 programming for HyperBus XiP init and switches XiP wait-cycles sourcing to xip_cfg->xip_wait_cycles.
  • Enables XiP slave-select via ospi_control_xip_ss() for specific SoC series using xip_cfg->xip_cs_pin.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
drivers/ospi/src/ospi.c Updates HyperBus XiP init implementation (controller register programming, wait cycles from config, conditional XiP SS control).
drivers/ospi/include/ospi.h Updates the HyperBus XiP init API declaration and its parameter documentation to use ospi_xip_config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread drivers/ospi/src/ospi.c Outdated
Comment thread drivers/ospi/src/ospi.c Outdated
@GaneshRamani-AlifSemi

Copy link
Copy Markdown
Contributor Author

copilots comments are reviewed and updated !

update api to enable hyperbus settings in OSPI controllor.

Signed-off-by: Ganesh Kumar R <ganesh.ramani@alifsemi.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants