Conversation
paresh-bhagat12
requested review from
StaticRocket,
VeeruPrudhvi,
aniket-l,
cshilwant,
praneethbajjuri and
uditkumarti
as code owners
Update Snagfactory doc for 11.0 release. Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
|
@v-singh1 to review |
StaticRocket
left a comment
StaticRocket
left a comment
There was a problem hiding this comment.
I approve of the general changes here. Please address Praneeth's comments.
paresh-bhagat12
dismissed stale reviews from StaticRocket, cshilwant, and praneethbajjuri
via
ce5d0db
- Add the section to profile boot time inside the kernel using GPIO - Fix random whitespaces - Update boot time numbers and other sections for AM62P 11.0 Signed-off-by: Aashvij Shenai <a-shenai@ti.com> Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
Remove the section temporarily from AM62x as fast boot has some issues. Signed-off-by: Paresh Bhagat <p-bhagat@ti.com>
|
@StaticRocket to re-review |
| # 38 - CRTC ID | ||
| # 1920x1200 - resolution of panel | ||
| modetest -M tidss -s 40@38:1920x1200 & | ||
| (modetest -M tidss -s 40@38:1920x1200 0<&- 2>/tmp/output.log) & |
There was a problem hiding this comment.
| (modetest -M tidss -s 40@38:1920x1200 0<&- 2>/tmp/output.log) & | |
| modetest -M tidss -s 40@38:1920x1200 0<&- 1>/tmp/output.log 2>/tmp/output.log & |
That subshell execution shouldn't be necessary, unless there's something else you're trying to accomplish by running it under it's own shell.
I'll ack as is though, as I've stalled this out enough already. Other useful changes in this PR.
There was a problem hiding this comment.
Hi @StaticRocket not a problem, but we will have to merge this PR for now as we have a refresh release coming up for am62p and currently don't have the bandwidth for testing new changes. I will create a new PR next week with your suggestions and after testing. Thanks
There was a problem hiding this comment.
merged this fix based on the comment above
10 participants
No description provided.