On Mon, 22 Aug 2011 15:01:51 -0700 Andrew Morton <[email protected]> wrote:
> On Sun, 21 Aug 2011 17:01:28 -0700 > Philip Prindeville <[email protected]> wrote: > > > Attached. > > oookay. I'll await a formal patch with all the cc's, etc. Also, if you email the patch inline (rather than as an attachment), then we can include comments inline as well. > > As for the "upstream" version: if/when this patch is merged, it *is* > the upstream version. The coreboot tree will need to drop the old > version and migrate to the new. IOW, make sure the coreboot folks are okay with what's being submitted as well! > > Please be sure to carefully describe the background to the code within > the final changelog. Lots of people won't even know what coreboot > *is*. Spell this out completely and be sure to tell the kernel > developers about the value of this patch to our users. > > I'd second this. I'm minimally familiar with coreboot, but I have no idea what the purpose behind coreboot BIOS tables are. -- coreboot mailing list: [email protected] http://www.coreboot.org/mailman/listinfo/coreboot

