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

Which sidepanel GPS ? #72

Open
kodsurf opened this issue Feb 8, 2023 · 2 comments
Open

Which sidepanel GPS ? #72

kodsurf opened this issue Feb 8, 2023 · 2 comments
Labels
question Further information is needed from PM/SE

Comments

@kodsurf
Copy link
Collaborator

kodsurf commented Feb 8, 2023

In a current develop branch gps module is initialized on LPC_UART0. However comment on the code says that it is sidepanel C (X-)

image
This looks incorrect.

My own docu "originaly" stated that LPC_UART0 is C/X-
image

Yesterday with flatbed setup it turned out that LPC_UART3 is C/X-


Please clarify to which sidepanel GPS should be connected by design, and check the code for gps init data.

@kodsurf
Copy link
Collaborator Author

kodsurf commented Feb 8, 2023

Ok, based on your adcs conversation, gps data to be supplied from ADCS. Meaning that climb_gps.c module will likely be deprecated.

If is ok - I will for now change gps UART to LPC_UART3 with direct commit to dev ?

@RobertK66
Copy link
Owner

My GPS code was based on (my!) assumption that we will need/have any GPS for time sync. The init structure allows for 2 additional IO pins which the GPS type I used provides. Code does not make use of it until now (see #15).

About 'the real thing' pls. refer to available CLIMB hardware design and Doku. The last known SatBusDefinition_v01_Table.pdf I have (4.8.2020!) does show UART usage: A-C: Stacie-A, Stacie-C B: S-Stacie D: thruster. What I remember is, that it was decided definitively to use a soft Uart (2 GPIOs) for Debug interface in future HW revisions - #12 (but I don't know where this is documented).

@WolfgangTreb , @AndyFHWN : Pls. clarify status of current design and hardware documentation!

@RobertK66 RobertK66 added the question Further information is needed from PM/SE label Feb 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is needed from PM/SE
Projects
None yet
Development

No branches or pull requests

2 participants