Skip to content

Pass boot payload to serial init#159

Merged
SamTebbs33 merged 1 commit intodevelopfrom
feature/serial-init-payload
Jun 13, 2020
Merged

Pass boot payload to serial init#159
SamTebbs33 merged 1 commit intodevelopfrom
feature/serial-init-payload

Conversation

@SamTebbs33
Copy link
Collaborator

Closes #158

@SamTebbs33 SamTebbs33 requested a review from DrDeano June 9, 2020 12:48
@SamTebbs33 SamTebbs33 force-pushed the feature/serial-init-payload branch 2 times, most recently from 989cdd3 to cc48ec1 Compare June 9, 2020 12:55
Copy link
Member

@DrDeano DrDeano left a comment

Choose a reason for hiding this comment

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

Some refactoring

/// The Serial instance constructed with the function used to write bytes
///
pub fn initSerial() Serial {
pub fn initSerial(payload: BootPayload) Serial {
Copy link
Member

Choose a reason for hiding this comment

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

Maybe could have the name the same in kamin: boot_payload

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure thing

Copy link
Member

Choose a reason for hiding this comment

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

You missed the other 2 files :P

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh I didn't know there were more lines to change

@SamTebbs33 SamTebbs33 force-pushed the feature/serial-init-payload branch from 9192169 to 5101990 Compare June 12, 2020 10:48
Copy link
Member

@DrDeano DrDeano left a comment

Choose a reason for hiding this comment

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

:)

@SamTebbs33 SamTebbs33 merged commit c4083b0 into develop Jun 13, 2020
@SamTebbs33 SamTebbs33 deleted the feature/serial-init-payload branch June 13, 2020 19:37
@SamTebbs33 SamTebbs33 mentioned this pull request Jul 20, 2020
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.

Pass the boot payload to serial.init

2 participants