Messages in this thread | | | Subject | Re: [PATCH] arm: msm: remove board file for Nexus One (ie. mahimahi) | From | Paul Bolle <> | Date | Mon, 19 May 2014 13:53:28 +0200 |
| |
Daniel,
On Thu, 2014-05-15 at 21:57 +0000, dwalker@fifo99.com wrote: > On Thu, May 15, 2014 at 08:10:13PM +0200, Paul Bolle wrote: > > This is not something I get to decide. Nevertheless, given that this > > file shouldn't have been merged to begin with, I'd appreciate it if some > > deadline could be agreed upon. > > I think I merged it actually, but there's no rules about what gets merged. How when what order, etc. > It's all free form.
There do not seem to be formal rules. But there surely are some requirements for code to be added. One of the requirements is, I think, that it should build. This file cannot be built: it is not wired into a Makefile and it also includes, what appears to be, its own header file, but that header is not part of the tree.
Even the most dubious of code in drivers/staging is expected to "compile properly"!
> > That being said, I'm not sure how having just this file in mainline > > helps your development efforts. It seems it did receive some updates > > for, well, treewide stuff. But it surely didn't get build coverage or > > runtime testing. So would you lose much if it only remains in your > > development tree? > > It's effort to remove it.. Your asking for it to get removed, then re-added.. That sounds > like a fairly large amount of effort vs just leaving it in place.
Yes, it takes some effort to remove code. And it will also take effort to re-add that code later. But I think that's a risk one runs with code that has clearly never been buildable.
Paul Bolle
| |