[coreboot] RFC: coding style: "standard" defines

ron minnich rminnich at gmail.com
Mon Feb 8 19:10:19 CET 2016


On Mon, Feb 8, 2016 at 9:46 AM Nico Huber <nico.h at gmx.de> wrote:

> On 08.02.2016 12:10, Patrick Georgi via coreboot wrote:
> > 2016-02-04 10:35 GMT+01:00 Patrick Georgi <pgeorgi at google.com>:
> >> during the review of some commits that are in the process of being
> >> upstreamed from Chrome OS, people noticed that chipset drivers like to
> >> define their own TRUE/FALSE defines (sometimes prefixed to), and I
> >> have seen a bunch of #define BIT{0-31} ..., too, because that seems to
> >> be the house rules in some firmware communities.
> > Seems like for the BIT defines, all variants are popular. Any
> > objection with moving them to src/include instead of having various
> > copies across the tree?
> How about something like src/include/please_dont_use.h ?
>
>

I don't know about that but I'd have no objection to a comment in the file
itself that these are vile constructs.

ron
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20160208/6f961b23/attachment.html>


More information about the coreboot mailing list