[coreboot] [commit] r6373 - in trunk/src/superio: fintek/f71805f fintek/f71859 fintek/f71863fg fintek/f71872 fintek/f71889 intel/i3100 ite/it8712f ite/it8716f smsc/lpc47b272 smsc/lpc47b397 smsc/lpc47m10x smsc...
Peter Stuge
peter at stuge.se
Mon Feb 21 04:27:01 CET 2011
Stefan Reinauer wrote:
>> +++ trunk/src/superio/smsc/lpc47n227/superio.c Sat Feb 19 15:51:31 2011 (r6373)
..
>> - lpc47n227_pnp_set_enable(dev, (dev->enabled) ? 1 : 0);
>> + lpc47n227_pnp_set_enable(dev, !!dev->enabled);
>
> How's this different?
This particular file did not have a functional change, but others
did.
> I like the ?: notion more than the :: notion.
I also like !! and I guess this file was changed to make this code
look the same everywhere.
Maybe it could even be moved to a common function.
//Peter
More information about the coreboot
mailing list