[PATCH] powerpc: Add FSL SOC library and setup code
Eugene Surovegin
ebs at ebshome.net
Sun Jan 15 10:47:54 EST 2006
On Sat, Jan 14, 2006 at 01:21:58PM -0600, Olof Johansson wrote:
> > +
> > +static phys_addr_t immrbase = -1;
>
> What does immr mean? Maybe a short comment would be good.
IMHO, this is not needed because _everybody_ who is working
with these chips know what IMMR means. And there cannot be _any_
confusion about it. Let's not add useless comments.
We don't add comments describing what MMU, PTE, PCI and IOMMU means,
do we? Any chip specific code has tons of strange abbreviations
which might be puzzling for anybody who isn't familiar with this chip
but are quite clear for anybody who are.
--
Eugene
More information about the Linuxppc-embedded
mailing list