On Thu, Nov 05, 2015 at 10:34:48AM +0300, Dan Carpenter wrote: > On Wed, Nov 04, 2015 at 09:06:08PM -0500, ira.weiny@xxxxxxxxx wrote: > > > +#define CCE_PCIE_CTRL_XMT_MARGIN_GEN1_GEN2_OVERWRITE_ENABLE_MASK 0x1ull > > There is absolutely no point to having a variable name which is nine > bajillion characters long since we are not able to use it and have to > instead use macro majic to hide the first twelve zillion characters. > The long names are directly from the hardware specification. > > If > we can't even see the name when it is used, then why does it exist? Also > macro magic breaks grep and cscope. Just think of a better name to > begin with and get rid of the PC macro. We did not want the PC macro but using the "real" names caused checkpatch failures. Indeed using the PC macro reduces the readability of the code in terms of following the hardware spec because we lose the "real" name. Is this an example where we should ignore the line lengths of checkpatch to preserve the readability of the code? Thanks, Ira _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
