driver: loapic: add DTS binding for local APIC #60114
Conversation
As per zephyrproject-rtos#26393, Local APIC is using Kconfig based option for the base address. This patch adds DTS binding support in the driver, just like its conunter part I/O APIC. Signed-off-by: Umar Nisar <umarnisar@outlook.com>
Adjust the formatting of the patch submitted (spac -> tabs) Signed-off-by: Umar Nisar <umarnisar@outlook.com>
fix formatting of the patch (space -> tab) for modified sections Signed-off-by: Umar Nisar <umarnisar@outlook.com>
fix formatting for the modified section of the file (spaces -> tab) Signed-off-by: Umar Nisar <umarnisar@outlook.com>
nisarumar
requested review from
dcpleung,
nashif,
galak,
jhedberg and
aasthagr
as code owners
zephyrbot
added
area: Interrupt Controller
platform: X86
zephyrbot
requested review from
andyross,
ceolin,
laurenmurphyx64,
MaureenHelm and
tbursztyka
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Hello @nisarumar, and thank you very much for your first pull request to the Zephyr project!
A project maintainer just triggered our CI pipeline to run it against your PR and ensure it's compliant and doesn't cause any issues. You might want to take this opportunity to review the project's Contributor Expectations and make any updates to your pull request if necessary. 😊
|
Hi @nisarumar, and thanks for the PR! Just checking that you didn't close this by accident? |
|
Hi @kartben, |
|
Gotcha! Feel free to leave the PR in Draft state to iterate on it and get CI to run and provide feedback, and only put it out of draft when it's ready for review. Cheers! |
As highlighted in #26393, local APIC still used Kconfig based CONFIG_LOAPIC_BASE_ADDRESS. This patch set adds DTS support to local APIC. I/O APIC already uses DTS. Similar to I/O APIC:
.dtsifiles.Several Intel x86 architecture define a maximum set of LVT entries for local APIC, hence the reason for adding
num-irqs.