-
Notifications
You must be signed in to change notification settings - Fork 55
Gen6 additions #360
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
Gen6 additions #360
Conversation
lsgunth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple interface issues to consider.
Lots of minor white-space, tabbing and code formatting issues.
The majority of the PR looks sound.
Add PSX/PFX device ids for gen6, add gen6 device decode functions and version decode functions. Gen6 lane info and link info added.
39b9a69 to
c10905d
Compare
Add RTC command to switchtec user for gen6 switchtec. Can set, get, reset the RTC. Add RTC MRPC command ID. and MRPC sub commands for RTC
84ce40d to
61fbf0c
Compare
| return 0; | ||
| } | ||
|
|
||
| int security_settings_get_gen6(struct switchtec_dev *dev, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If it's BL1 only, maybe can use some comments to explain.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added comments
lsgunth
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still seeing a hand full of whitespace issues. This is minor.
Besides that, I approve.
cbbc22a to
855d192
Compare
Add gen6 metadata and firmware partition types and flash info structures. Update the gen6 get firmware partition functions with Gen6 flash info structures. Add MRPC fw part command ids for gen6.
Gen6 does not have RIOT core partition
Add the updated command ids for mfg MRPC command calls for gen6. Update the gen5 mfg calls to 56 which will account for both cases, gen5/gen6. Bootresume is not supported in gen6 add check in boot_resume command.
855d192 to
44c4592
Compare
Add pipe datapath type for gen6. New loopback type and update function names which previosly called gen5 to gen56 since they support both now
Add MRPC PATGEN / MON subcommands ids and checks to functions
Add Gen6 and Gen6 Speed to LTSSM command
Add case for switchtec gen6 which uses same gen4 hdr write function
Added check for Gen6 and change to gen6 command id OSA
Compilation issue for declaration after label in some situations, moved array declaration to top of function block instead.
Add Gen6 function fw img get command read command to the existing fw-read command
Add new gen6 function to handle gen6 log dumping. Add gen6 log output structure and handling string function to decode the values sent from the log dum p MRPC.
Add check for running new security settings for the OTPs in BL1 stage of the boot. Add check in the info command Add new OTP MRPC get command as part of secure mfg settings.
44c4592 to
cc26005
Compare
Commands updated / supported with the Gen6 spec
• info
• log-dump
• log-parse
• test
• temp
• hard-reset
• fw-update
• fw-info
• fw-redundant
• fw-read
• fw-img-info
• rtc
• mfg ping
• mfg info
• mfg mailbox
• mfg fw-transfer
• mfg fw-execute