> > We need to pick a method for accepting new chips and go with it. If you want whole files that are finished, that's fine with me. If you want to force the development to be done elsewhere, I'm fine with that too.
>
> I'm not sure if I understand this correctly. You seem to
> say that pushing whole files implies that you have to first
> finish the whole development and then break it apart again?
> Is that it?
>
> I guess that might be true for a small, core part of the
> code, but most likely not for the whole 11k LOC. Maybe
> I'm wrong here but the code looks too well organized to
> be hard to break apart, or rather to pick one topic after
> another from it.

We've already been working on the project for a bit while we've been waiting for permission to start upstreaming it. Because of that we've got a backlog of changes to push. It's not a large backlog at this point, but it's not insignificant. That work's just been done on individual machines so far - I've got to go through that code and break it into small cohesive bits.

So I could just push the changed files, which include changes from various different topics, but that would make it hard to review.

While I do work at Google, I'm a part of the coreboot community. I'm working at Google BECAUSE i'm a part of the community. I understand the issues. I don't like the blobs and lack of documentation either. Although it may still be going in the wrong direction I'm actively working to try to improve things.

Um, I'm confused that you bring your employment and blobs
into this. Did sb. (I?) bring that up somewhere and forgot?
Or is there an argument? If you wouldn't be a part of the
community (which is great that you are btw.) and working
for Google, this discussion would be less unexpected? I'm
really puzzled here.

No, you didn't say anything, but you can't deny that there's a lot of resentment about large companies (such as Google) in some sectors of the community. I'm trying to say (Not specifically to you) that I'm working as a part of the community, not just as a representative for Google. I feel the same frustration with the blob situation, and I'm sorry. My opinion is that it's best to work with the companies to show why it's better to publish. Other people feel like it's better to boycott and badmouth the companies. (Nico, I'm not pointing at you in saying this. We've all seen the discussions on IRC.) I get it - people feel strongly. We've both put countless hours into the project, far more than most people who want to sit on the sidelines and criticize.

If I were entering the community for the first time, trying to commit a new SOC, it would look insurmountable. I'm saying that we should have a defined method of how to push and maintain these things that we can agree upon as a community.

View Change

To view, visit change 32407. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I6809bd1eea304f76dd9000c079b3ed09f94dbd3b
Gerrit-Change-Number: 32407
Gerrit-PatchSet: 1
Gerrit-Owner: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Aaron Durbin <adurbin@chromium.org>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Nico Huber <nico.h@gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-CC: HAOUAS Elyes <ehaouas@noos.fr>
Gerrit-CC: Kyösti Mälkki <kyosti.malkki@gmail.com>
Gerrit-Comment-Date: Fri, 26 Apr 2019 19:00:40 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment