[LinuxBIOS] #31: Do proper checking for flash erase for SST FWH parts
LinuxBIOS
svn at openbios.org
Mon Nov 6 10:21:01 CET 2006
#31: Do proper checking for flash erase for SST FWH parts
-----------------------------------+----------------------------------------
Reporter: stepan | Owner: stepan
Type: defect | Status: assigned
Priority: minor | Milestone: Enhance the flashrom utility
Component: flashrom | Version: v2
Resolution: | Keywords:
Due_close: MM/DD/YYYY | Include_gantt: 0
Dependencies: | Due_assign: MM/DD/YYYY
Patchstatus: patch needs review |
-----------------------------------+----------------------------------------
Comment (by uwe):
Replying to [comment:3 segher]:
> Every patch needs review, unless it already got it, and then
> it's either back to the drawing table, or that patch get's
> checked in :-)
Sure, but we need a state which tells us that there is a patch at all (not
all tickets have a patch), and in which state it is (has it already been
reviewed? does it need more work? is it ready to be comitted?) etc.
This is mostly used for the shiny new "Patch Queue" I created yesterday:
http://tracker.linuxbios.org/trac/LinuxBIOS/report/9
(or click View Tickets -> Patch Queue)
That should make it easier to track which tickets have patches and in
which state. Reviewers or people who want to commit patches which have
been approved can simply work on the patch queue and make it smaller.
> If total_size is indeed the size of flash that got erased,
> it looks good to me. But Stefan himself knows best, he
> should just go ahead and apply the patch if no one complains
> soon (like, wait for tomorrow morning or something like that).
Add an Acked-by: if it looks good to you. One important part of the sign-
off procedure is that we have sort of a group-review, i.e. at least one
other developer has to review the patch and give it his/her ACK (the other
important part is copyright and authorship tracking, of course).
--
Ticket URL: <http://tracker.linuxbios.org/trac/LinuxBIOS/ticket/31#comment:4>
LinuxBIOS <http://www.linuxbios.org/>
More information about the coreboot
mailing list