Julius Werner has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/29948 )
Change subject: soc/qualcomm/qcs405: Support for new SoC
......................................................................
Patch Set 11:
(1 comment)
https://review.coreboot.org/#/c/29948/2/src/soc/qualcomm/qcs405/timer.c
File src/soc/qualcomm/qcs405/timer.c:
https://review.coreboot.org/#/c/29948/2/src/soc/qualcomm/qcs405/timer.c@18
PS2, Line 18:
Hi Julius, […]
Thanks, sounds good. I don't really mind what order you do things in as long as duplicate code does not get landed, but keep in mind that you may end up duplicating work if you try to polish WIP mistral patches too much before aligning them to what Cheza had (especially because a lot of additional work had already been done on those Cheza patches).
--
To view, visit
https://review.coreboot.org/c/coreboot/+/29948
To unsubscribe, or for help writing mail filters, visit
https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: Ia379cf375e4459ed55cc36cb8a0a92cab18b705e
Gerrit-Change-Number: 29948
Gerrit-PatchSet: 11
Gerrit-Owner: nsekar@codeaurora.org
Gerrit-Reviewer: Julius Werner
jwerner@chromium.org
Gerrit-Reviewer: Martin Roth
martinroth@google.com
Gerrit-Reviewer: Patrick Georgi
pgeorgi@google.com
Gerrit-Reviewer: build bot (Jenkins)
no-reply@coreboot.org
Gerrit-Reviewer: mturney mturney
mturney@codeaurora.org
Gerrit-Reviewer: nsekar@codeaurora.org
Gerrit-CC: Sricharan Ramabadhran
srichara@qualcomm.corp-partner.google.com
Gerrit-Comment-Date: Wed, 27 Feb 2019 21:42:14 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: Julius Werner
jwerner@chromium.org
Comment-In-Reply-To: Sricharan Ramabadhran
srichara@qualcomm.corp-partner.google.com
Gerrit-MessageType: comment