Skip to content

Conversation

@npal-cy
Copy link
Contributor

@npal-cy npal-cy commented Nov 14, 2024

Problem#1: not defined zephyr_console for [email protected]/cy8c6347/m4
https://github.com/zephyrproject-rtos/zephyr/actions/runs/11524969451/job/32695831764?pr=80025

- west twister -p [email protected]/cy8c6347/m4 -s tests/drivers/build_all/w1/drivers.w1.build
INFO    - or with west:
INFO    - west build -p -b [email protected]/cy8c6347/m4 tests/drivers/build_all/w1 -T drivers.w1.build

Problem#2: arm_irq_zero_latency_levels:
https://github.com/zephyrproject-rtos/zephyr/actions/runs/11524969451/job/32695832470?pr=80025

INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) tests/arch/arm/arm_irq_advanced_features/arch.arm.irq_advanced_features on cy8cproto_062_4343w/cy8c624abzi_s2d44 error (Build failure)
INFO    - 2) tests/arch/arm/arm_irq_zero_latency_levels/arch.arm.irq_zero_latency_levels on cy8cproto_062_4343w/cy8c624abzi_s2d44 error (Build failure)

Copy link
Contributor

@nordicjm nordicjm left a comment

Choose a reason for hiding this comment

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

You have a revision 1.0.0 board so the board targets must exist

@npal-cy
Copy link
Contributor Author

npal-cy commented Nov 14, 2024

For https://github.com/zephyrproject-rtos/zephyr/actions/runs/11524969451/job/32695832470?pr=80025

"Invalid interrupt priority. Values must not exceed IRQ_PRIO_LOWEST".

Default GPIO interrupt priority is 6, in arch.arm.irq_zero_latency_levels test with CONFIG_ZERO_LATENCY_LEVELS=2, IRQ_PRIO_LOWEST will be 4.

Options we have here:

  1. add board overlay files in tests/arch/arm/arm_irq_zero_latency_levels, where change DPIO priority for used GPIO port.

  2. rework default interrutp priority from 6 to defined in IFX_DEFAULT_IRQ_PRIORITY/override.dtsi similar to how Nordic uses.
    -- add override.dtsi in tests/arch/arm/arm_irq_zero_latency_levels (dts\arm\infineon)

@nordicjm any suggestion how to handle this in proper way?

@nordicjm
Copy link
Contributor

nordicjm commented Nov 14, 2024

What you have linked isn't even related to this because what you have linked is:

-- Found west (found suitable version "1.2.0", minimum required is "0.14.0")
-- Board: cy8cproto_062_4343w, qualifiers: cy8c624abzi_s2d44
-- Found host-tools: zephyr 0.17.0 (/opt/toolchains/zephyr-sdk-0.17.0)

INFO    - The following issues were found (showing the top 10 items):
INFO    - 1) tests/arch/arm/arm_irq_advanced_features/arch.arm.irq_advanced_features on cy8cproto_062_4343w/cy8c624abzi_s2d44 error (Build failure)
INFO    - 2) tests/arch/arm/arm_irq_zero_latency_levels/arch.arm.irq_zero_latency_levels on cy8cproto_062_4343w/cy8c624abzi_s2d44 error (Build failure)

Notice how there is no version in the board or output? No version specified means it will use the default, which in

is 0.0.0, not 1.0.0

In fact what you've linked to isn't even this board so it has no resemblance at all

@npal-cy npal-cy changed the title cy8ckit_062_ble: board: remove ver 1.0.0 for m4 Fix build error for cy8ckit_062_ble/cy8cproto_062_4343w boards Nov 14, 2024
@npal-cy
Copy link
Contributor Author

npal-cy commented Nov 14, 2024

Hi @nordicjm, i updated title/description and provided fix for problem#1/problem#2.

Those changes should unblock #80025 (comment)

nordicjm
nordicjm previously approved these changes Nov 14, 2024
@npal-cy npal-cy marked this pull request as ready for review November 14, 2024 13:33
jhedberg
jhedberg previously approved these changes Nov 14, 2024
Moved uatr6 to common dts for m4

- cm0 uses uart-5 in ver 0.0.0 and uart-5 in ver 1.0.0 (for
using of Arduino headers).

- cm4 by default uses uart-2 (for ver 0.0.0 and ver 1.0.0)

Signed-off-by: Nazar Palamar <[email protected]>
Remove CONFIG_GPIO from defconfigs for Infineon boards.
Applications, drivers will enable GPIO if need.

Added 'select GPIO' from spi/Kconfig.ifx_cat1
Added 'select GPIO' from wifi/infineon/Kconfig.airoc

Signed-off-by: Nazar Palamar <[email protected]>
@mmahadevan108 mmahadevan108 merged commit 7e1b00d into zephyrproject-rtos:main Nov 14, 2024
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: SPI SPI bus area: Wi-Fi Wi-Fi platform: Infineon Infineon Technologies AG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants