-
-
Notifications
You must be signed in to change notification settings - Fork 20
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
Single generic testing stand setup guide #753
Conversation
bcd80aa
to
7b5810a
Compare
f529389
to
730ca8e
Compare
dead links fixed aside from link to deleted coreboot branch in old DTS version changelog, not sure how to resolve |
Let's replace with the link to the repository, with no specific branch. |
63278d2
to
3b1e9b8
Compare
CI fully satisfied |
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.
LGTM :)
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Filip Lewiński <[email protected]>
Signed-off-by: Michał Kopeć <[email protected]>
connection via serial and external flashing. | ||
* Sonoff - if we need line voltage control. | ||
* PiKVM - if it is not possible to read the device via serial or it is limited, | ||
it is possible to simulate the keyboard and read the image from HDMI. |
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.
Should the required wires and components be listed here in a comprehensive manner? I guess nobody should even start work without having all the elements.
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.
Fixed here
|:---------:|:-----------------------------:| | ||
| J11 pin 5 | PWR_ON# | | ||
| J11 pin 6 | RST# | | ||
| J15 pin 1 | GND | |
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.
@filipleple I have concerns regarding this table; in the case of PC Engines, pin5 is RST, and pin6 is PWR; otherwise, it doesn't work. Could that be some mistake? Can we check that for Protectli, as it seems to be a reference in this guide?
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.
@filipleple
In a few spots, we mention the RTE version.
I think we should add a note saying this documentation is based on RTE v1.1.
RTE v1.1
- J11 Pin 6: PWR
- J11 Pin 5: RST
- Schematic: RTE v1.1.0 PDF
RTE v1.0
- J11 Pin 8: RST
- J11 Pin 9: PWR
- Schematic: RTE v1.0.0 PDF
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.
Thank you! I've added a note recommending using RTE v1.1, and following notes accounting for the differences here
Addressing the new review in #971 |
No description provided.