[BUGFIX] OBC SPI Driver - bus owner check

[ThenujaL]

Purpose

• Bug fix for checking if the current task is the mutex holder
• EDIT: Link to notion ticket

New Changes

• Updated mutex check to work without the task having to acquire the mutex
• Get the mutex holder task handle, get the current task handle and compare the two

Testing

• Tested build
• Wrote test tasks and verified behaviours. Test task pushed to new test branch here
• Test serial output:

Acquired mutex
PASS: isSpiBusOwner returned TRUE after acquiring mutex
Releasing mutex...
Released mutex
PASS: isSpiBusOwner returned FALSE after releasing mutex
PASSED isSpiBusOwner Test!

Outstanding Changes

• NA

EDIT 2025-03-01: Added more change details, on-board testing

[github-actions]

Pull reviewers stats

Stats of the last 120 days for UWOrbital:

	User	Total reviews	Time to review	Total comments
	Navtajh04	33	5d 16h 5m	276
	dgobalak	29	4d 10h 5m	213
	Gjjjiang	5	12d 17h 14m	16
	Yarik-Popov	5	1d 8h 47m	63
	manumanuk	2	1d 7h 24m	7
	JoelManYunLee	1	17h 57m	4
	TheSpaceDragon	1	14d 22h 25m	0

[Navtajh04]

LGTM

[dgobalak]

lgtm too, but let's test this before merging

[ThenujaL]

lgtm too, but let's test this before merging

@dgobalak Any suggestions on how to test this?

[Yarik-Popov]

Errors when running code on launchpad

[ThenujaL]

@Yarik-Popov Do we have any logs or any more details on what the errors were?

[Yarik-Popov]

@Yarik-Popov Do we have any logs or any more details on what the errors were?

Couldnt get logs as I'm having some difficulty with serial ports on my laptop, they stopped working for some reason.

I ran your code directly and added port enter/exit critical functions as before, both around xSemaphoreGetHolder and around both helper function calls. In all cases without additional modifications, the launchpad goes into an error state where the ERR LED keeps flashing. If I call this function before a while(1) inside the state_mgr init when the ERR LED doesn't turn on, but the normal led doesn't blink as state_mgr doesn't get to the end of its init. Hope this helps

[ThenujaL]

Added testing

[kepler452b123]

lgtm

[Yarik-Popov]

Lgtm

Already handled