[coreboot] [RFC] mainboard.c: Unify name of device/board enable function
Peter Stuge
peter at stuge.se
Fri Feb 22 02:17:12 CET 2013
Paul Menzel wrote:
> struct chip_operations mainboard_ops = {
> .enable_dev = inagua_enable,
> };
>
> Naming this function is not uniform between the boards in the tree.
Depending on how mainboard_ops is used in all the boards maybe we
want to simply remove it completely.
> If there are no objections to unification, what should the name for the
> function be?
Function names should try to be descriptive. "enable_dev" is not very
descriptive. I like "mainboard_enable" because it makes output such
as
printk("%s: foo", __func__);
useful.
> Please comment on this, so a patch can be submitted.
I'd +2 a commit which renames all to mainboard_enable. It would be
very nice to also add a test for this to the Jenkins build.
//Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: not available
URL: <http://www.coreboot.org/pipermail/coreboot/attachments/20130222/7acfb63b/attachment.sig>
More information about the coreboot
mailing list