Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chrysler: forward bus 0 to bus 2 #177

Merged
merged 2 commits into from
Apr 4, 2019
Merged

Conversation

adhintz
Copy link
Contributor

@adhintz adhintz commented Apr 3, 2019

This keeps the stock LKAS camera active so that we can copy data from its messages.

I'll open a PR in openpilot with code that uses this to support arbitrary Chrysler models, including 2019.

This keeps the stock LKAS camera active so that we can copy data from its messages.
adhintz added a commit to adhintz/openpilot that referenced this pull request Apr 3, 2019
…h likely support for arbitrary models.

This is done by copying most values from the stock LKAS camera which is active with commaai/panda#177
Forward LKAS_HEARTBIT messages from stock camera to bus 0.
@@ -125,12 +125,29 @@ static int chrysler_tx_hook(CAN_FIFOMailBox_TypeDef *to_send) {
return true;
}

static void chrysler_init(int16_t param) {
chrysler_camera_detected = 0;
Copy link
Contributor

@rbiasini rbiasini Apr 4, 2019

Choose a reason for hiding this comment

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

this needs to be set to 1 if camera msgs are on bus 0, right?

Copy link
Contributor Author

@adhintz adhintz Apr 4, 2019

Choose a reason for hiding this comment

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

Yeah, it already is up on line 50:

  // check if stock camera ECU is still online
  if (bus == 0 && addr == 0x292) {
    chrysler_camera_detected = 1;

Copy link
Contributor

Choose a reason for hiding this comment

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

ah, right!

@rbiasini rbiasini merged commit 1049502 into commaai:master Apr 4, 2019
rbiasini pushed a commit to commaai/openpilot that referenced this pull request Apr 5, 2019
* 2019 Chrysler Pacfica and 2019 Jeep Grand Cherokee support, along with likely support for arbitrary models.
This is done by copying most values from the stock LKAS camera which is active with commaai/panda#177

* No longer send LKAS_HEARTBIT because Panda now forwards it for us.

* Pacifica Hybrid 2018 combine fingerprints, add 808: 8

* panda chrysler: forward bus 0 to bus 2
copy of Panda commit: commaai/panda@1049502
legonigel pushed a commit that referenced this pull request Apr 13, 2019
* chrysler: forward bus 0 to bus 2
This keeps the stock LKAS camera active so that we can copy data from its messages.

* Only forward if giraffe switches are 1 0.
Forward LKAS_HEARTBIT messages from stock camera to bus 0.
dragonpilot pushed a commit to dragonpilot-community/dragonpilot that referenced this pull request Apr 7, 2020
…ai#590)

* 2019 Chrysler Pacfica and 2019 Jeep Grand Cherokee support, along with likely support for arbitrary models.
This is done by copying most values from the stock LKAS camera which is active with commaai/panda#177

* No longer send LKAS_HEARTBIT because Panda now forwards it for us.

* Pacifica Hybrid 2018 combine fingerprints, add 808: 8

* panda chrysler: forward bus 0 to bus 2
copy of Panda commit: commaai/panda@1049502
@adhintz adhintz deleted the auto-upstream branch January 20, 2022 00:05
leomonde pushed a commit to leomonde/volvopilotc2 that referenced this pull request Apr 5, 2024
* 2019 Chrysler Pacfica and 2019 Jeep Grand Cherokee support, along with likely support for arbitrary models.
This is done by copying most values from the stock LKAS camera which is active with commaai/panda#177

* No longer send LKAS_HEARTBIT because Panda now forwards it for us.

* Pacifica Hybrid 2018 combine fingerprints, add 808: 8

* panda chrysler: forward bus 0 to bus 2
copy of Panda commit: commaai/panda@1049502
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.

2 participants