Re: [PATCH V7 7/8] ptp: Added a clock driver for the IXP46x.

From: Richard Cochran
Date: Sun Jan 02 2011 - 04:12:55 EST


On Sun, Jan 02, 2011 at 09:45:05AM +0100, Pavel Machek wrote:
> Hi!!
>
> > +struct ixp46x_channel_ctl {
> > + u32 Ch_Control; /* 0x40 Time Synchronization Channel Control */
> > + u32 Ch_Event; /* 0x44 Time Synchronization Channel Event */
> > + u32 TxSnapLo; /* 0x48 Transmit Snapshot Low Register */
> > + u32 TxSnapHi; /* 0x4C Transmit Snapshot High Register */
>
> CouldWeGetRidOfCamelCase?

I agree that CamelCase is ugly and in bad taste. However, I make an
exception when the register level programmer's manual uses this style.

IMHO, it is better to use the exact same mnemonics as in the
manual. That way, when the next person comes along to make a change to
the code, with manual in hand, he will immediately know what is what.

Thanks,
Richard
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/