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

SDMODEL H7 V2 #26935

Merged
merged 3 commits into from
May 14, 2024
Merged

SDMODEL H7 V2 #26935

merged 3 commits into from
May 14, 2024

Conversation

andyp1per
Copy link
Collaborator

No description provided.

@okok125
Copy link

okok125 commented May 1, 2024

Thank You . Andy

@okok125
Copy link

okok125 commented May 5, 2024

README.md.txt

Copy link
Collaborator

@Hwurzburg Hwurzburg left a comment

Choose a reason for hiding this comment

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

rather than me correcting the readme, why not just make the README.md have the two board images that are different from the V1, mention the addition of the compass, and refer directly to the V1 readme file on github....will avoid ommisissions and mistakes

@okok125
Copy link

okok125 commented May 5, 2024

Change the default value for UART4 to GPS and for UART3 to Telem3. Added built-in IST8310 compass.
It is still possible to add a 2nd compass via SCL and SDA.
README.md.txt

@okok125
Copy link

okok125 commented May 5, 2024

rather than me correcting the readme, why not just make the README.md have the two board images that are different from the V1, mention the addition of the compass, and refer directly to the V1 readme file on github....will avoid ommisissions and mistakes

Everything follows your instructions

@Hwurzburg Hwurzburg added the WikiNeeded needs wiki update label May 8, 2024
@andyp1per andyp1per force-pushed the pr-sdmodel-v2 branch 2 times, most recently from 6cd7b9b to f9e243e Compare May 8, 2024 07:32
@andyp1per
Copy link
Collaborator Author

README updated

libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
libraries/AP_HAL_ChibiOS/hwdef/SDMODELH7V2/README.md Outdated Show resolved Hide resolved
@Hwurzburg
Copy link
Collaborator

Hwurzburg commented May 13, 2024

Andy, I can make the readme changes if you wish, and not all my review comments appear here....but if you look at the readme file itself, they are there?????don't know why

@Hwurzburg Hwurzburg force-pushed the pr-sdmodel-v2 branch 2 times, most recently from 2b57492 to 24bdbd9 Compare May 13, 2024 17:17
@tridge tridge merged commit 2891b37 into ArduPilot:master May 14, 2024
45 of 46 checks passed
@andyp1per andyp1per deleted the pr-sdmodel-v2 branch May 14, 2024 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 4.5.3-beta1
Development

Successfully merging this pull request may close these issues.

None yet

6 participants