Skip to content

Comments

numbers.h: simplify #ifndef ZEPHYR for MIN and MAX#4130

Merged
lgirdwood merged 1 commit intothesofproject:mainfrom
marc-hb:ifndef-zep-min
May 4, 2021
Merged

numbers.h: simplify #ifndef ZEPHYR for MIN and MAX#4130
lgirdwood merged 1 commit intothesofproject:mainfrom
marc-hb:ifndef-zep-min

Conversation

@marc-hb
Copy link
Collaborator

@marc-hb marc-hb commented May 3, 2021

Undefining and re-defining MIN/MAX when already defined is
convoluted. Simply define them if not __ZEPHYR__ instead.

Signed-off-by: Marc Herbert marc.herbert@intel.com

Undefining and re-defining MIN/MAX when already defined is
convoluted. Simply define them if not __ZEPHYR__ instead.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Copy link
Member

@lgirdwood lgirdwood left a comment

Choose a reason for hiding this comment

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

Good stuff @marc-hb - need more like this for any and all duplicated macros. @kv2019i @lyakh fyi

@marc-hb marc-hb marked this pull request as ready for review May 4, 2021 05:58
@marc-hb
Copy link
Collaborator Author

marc-hb commented May 4, 2021

WHL_HDA failed everything in https://sof-ci.01.org/sof-pr-viewer/#/build/PR4130/build6388021 but the exact same happened in this entirely different and unrelated PR so it's hopefully unrelated: https://sof-ci.01.org/sof-pr-viewer/#/build/PR4131/build6388373

@zrombel, @kkarask or @wszypelt could you check what's going on with WHL_HDA?

https://sof-ci.01.org/sofpr/PR4130/build8910/devicetest was all green

@marc-hb marc-hb requested review from andyross, cujomalainey and lyakh May 4, 2021 06:07
@lgirdwood lgirdwood merged commit ce52555 into thesofproject:main May 4, 2021
@marc-hb marc-hb deleted the ifndef-zep-min branch September 2, 2021 23:49
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.

3 participants