Hi,
the flashrom project gets donation offers roughly twice a month, and
quite a few times people wanted to donate more than 50 EUR / 60 USD. We
can't accept those donations right now because there is no legal entity
behind the flashrom project.
At the same time, we could use some money to improve flasher support by
buying some flash chips and programmers so all developers can easily
test changes. Besides that, posters/handouts for LinuxTag/FOSDEM/...
booths are currently paid for by those flashrom guys who go to those
exhibitions.
My proposal is to create a nonprofit club in Germany (eingetragener
Verein) in the same way as KDE and OpenEmbedded did it.
Given that the majority of the flashrom team lives somewhere in the EU,
starting or joining a nonprofit in the USA wouldn't make sense. An
informal request for comments among those who are involved in flashrom
did yield some noticeable interest, and I guess now is the time to make
this plan public.
coreboot is not an immediate target of this nonprofit club because I
don't want to interfere with any plans which may be ongoing on the
coreboot side. Nonwithstanding the above, if the coreboot project
leaders decide that participating in a nonprofit in Germany would make
sense, there's no reason not to do this together. Then again, the
majority of coreboot developers live in the USA and a nonprofit outside
the USA might not make sense for them.
Side note: We (coreboot/flashrom) won a substantial amount of money (to
be redeemed in hardware) in a "project beauty contest" held by
Thomas-Krenn.com, a nice server manufacturer. We need a legal entity to
accept that hardware.
Comments?
Regards,
Carl-Daniel
On Sat, 3 Mar 2012 21:11:11 +0100
Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> wrote:
> - chipsets
>
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
> ---
> print_wiki.c | 36 +++++++++++++++++++++++-------------
> 1 files changed, 23 insertions(+), 13 deletions(-)
refined, combined with the other two and committed in r1556.
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
On Sat, 3 Mar 2012 21:11:12 +0100
Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> wrote:
> - boards
>
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
> ---
> print_wiki.c | 26 ++++++++++++++++----------
> 1 files changed, 16 insertions(+), 10 deletions(-)
refined, combined with the other two and committed in r1556.
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
On Sat, 3 Mar 2012 21:11:13 +0100
Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at> wrote:
> - chips
>
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
> ---
> print_wiki.c | 30 ++++++++++++++++++------------
> 1 files changed, 18 insertions(+), 12 deletions(-)
refined, combined with the other two and committed in r1556.
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner
Am 03.03.2012 21:11 schrieb Stefan Tauner:
> Previously boards in the wiki were tagged either as working or as known
> bad. But we added support to various boards via board enables that were
> then never tested because the owners have not reported back. This can
> now be tagged with NT and is shown appropriately.
>
> Also, the underlying data structure indicating state was converted from
> macros to an enum while preserving original integer values.
>
> Because all lines specifying supported boards and laptops were touched
> anyway, this patch also re-indents them.
>
> ---
> TODO: change other occurrences to use it. wanted to get feedack first.
>
> Signed-off-by: Stefan Tauner <stefan.tauner(a)student.tuwien.ac.at>
>
> diff --git a/print_wiki.c b/print_wiki.c
> index 377154d..9a9cd83 100644
> --- a/print_wiki.c
> +++ b/print_wiki.c
> @@ -136,9 +136,9 @@ static void wiki_helper(const char *devicetype, int cols,
> const struct board_match *b = board_matches;
>
> for (i = 0; boards[i].vendor != NULL; i++) {
> - if (boards[i].working)
> + if (boards[i].working == OK)
> boardcount_good++;
> - else
> + if (boards[i].working == BAD)
> boardcount_bad++;
You could replace that construct with a switch(), and it might make
sense to count untested boards as well.
> }
>
> @@ -171,7 +171,8 @@ static void wiki_helper(const char *devicetype, int cols,
> b[k].lb_vendor ? b[k].lb_vendor : "",
> b[k].lb_vendor ? ":" : "",
> b[k].lb_vendor ? b[k].lb_part : "",
> - (boards[i].working) ? "OK" : "No");
> + (boards[i].working == OK) ? "OK" :
> + (boards[i].working == NT) ? "?3" : "No");
The ?3 looks odd, but I assume you tested it.
>
> if (boards[i].note) {
> printf("<sup>%d</sup>\n", num_notes + 1);
> diff --git a/print.c b/print.c
> index 1fdeac7..544a846 100644
> --- a/print.c
> +++ b/print.c
> @@ -545,428 +545,427 @@ void print_supported(void)
> [...]
> - B("ZOTAC", "Fusion-ITX WiFi (FUSION350-A-E)", 1, NULL, NULL),
> - B("ZOTAC", "GeForce 8200", 1, "http://pden.zotac.com/index.php?page=shop.product_details&product_id=129&ca…", NULL),
> - B("ZOTAC", "H67-ITX WiFi (H67ITX-C-E)", 0, NULL, "Probing works (Winbond W25Q32, 4096 kB, SPI), but parts of the flash are problematic: descriptor is r/o (conforming to ICH reqs), ME region is locked."),
> - B("ZOTAC", "ZBOX HD-ID11", 1, "http://pdde.zotac.com/index.php?page=shop.product_details&product_id=240&ca…", NULL),
> + B("A-Trend", "ATC-6220", OK, "http://www.motherboard.cz/mb/atrend/atc6220.htm", NULL),
> + B("abit", "A-S78H", OK, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?pMODEL_NA…", NULL),
> + B("abit", "AN-M2", OK, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=…", NULL),
> + B("abit", "AV8", OK, "http://www.abit.com.tw/page/en/motherboard/motherboard_detail.php?DEFTITLE=…", NULL),
Is it possible that this patch made the board table one or two tabs
wider? The patch looks like that, and while I agree that some
files/sections should not have line length limits, adding another 16
columns of whitespace is something I'd like to avoid.
print_wiki related code is something I rarely touch (except for
programmer additions), so please don't expect in-depth reviews from me.
A cursory review suggests that the patch at least doesn't make the code
worse and I don't have any strong feelings about this code. If you feel
this patch is beneficial, I can send a weak Acked-by, more like Meh-by.
Regards,
Carl-Daniel
--
http://www.hailfinger.org/
On Fri, 27 Jul 2012 21:43:07 +0000
"Corcoran, Mike" <mike.corcoran(a)wright.edu> wrote:
> Successful read, erase, and write. Thanks again!
Thanks! Unfortunately i missed that someone has tested that already
last month and i just did not commit it yet, so the additional testing i
requested was unnecessary, sorry!
> I also documented the use of your tool on my online docs web site so others can solve
> Similar problems and find your tool as well:
> http://www.wright.edu/~mike.corcoran/htmldocs/peripherals/sata/sil3124/sil3…
awesome :)
--
Kind regards/Mit freundlichen Grüßen, Stefan Tauner