2020 Hyundai Palisade - Harness Malfunction Error

[scottadair]

Describe the bug

About a week or so ago, something was committed to master which broke support for my car. I'm not getting a "Harness Malfunction" error. When I revert to release3, the error goes away.

Trying to figure out which commit broke. I am not using a Comma branded harness, mine is a custom harness that was made before the official ones were released. Has been working fine for many years now. Not sure what's changed in the code in the past couple weeks.

Which car does this affect?

2020 Hyundai Palisade

Provide a route where the issue occurs

4b8ae689eaa74602|2024-02-11--16-36-27--0

openpilot version

master

Additional info

No response

[scottadair]

I reverted this change, and the harness error went away.

bcd29a2

Any thoughts @sshane ?

[sshane]

What is your Discord? We can quickly determine the issue if you're up for trying a branch

In the mean time, please try master without alpha longitudinal to make sure that there's no errors from the extra queries (the queries are somehow causing the radar to not disable, checking your route)

[scottadair]

Yeah can certainly try master and will disable alpha long support.

My discord account is scottieadair

[scottadair]

On master with alpha long disable, everything seems to work. No dash errors

[sshane]

Caused by a refactor 2 months ago not checking if it's a CAN FD car before adding CANFD_HDA2 flag + the PR above to get the ADAS ECU: #29733

Then it tried to disable 0x730 (adas) instead of the radar

[Gr3gK1]

I installed the latest release-c3 2 days ago and received the "Harness Malfunction" error yesterday morning. Thought maybe it was a connection issue. Checked everything. Ask connections are perfect. Then the comma worked for a 3-hour drive, but today the same error occured. The conditions were the same - stopping behind a car with comma saying that control is temporarily unavailable. And then all hell breaks loose. The car starts flashing that all assist systems are malfunctioning and comma complains about the harness error. I have a 2021 Hyundai Palisade. Turning the car off and back on doesn't reset the system, but turning off, opening and closing the driver's door, then turning on again brings comma back to the functional state.

Alpha Long disabled has no errors. I think I have narrowed down the specific moment of the error to the engine idle during traffic light stops. When I press the A button to disable auto shut-off on idle, comma works fine without harness errors.

Does this sound like the same or different issue @sshane ? Thank you!