On Mon, Jun 8, 2015 at 11:33 PM, hch@lst.de [off-list ref] wrote:

On Wed, Jun 03, 2015 at 07:24:34PM +0000, Williams, Dan J wrote:

quoted

quoted

quoted

+static inline struct acpi_nfit_memory_map *__to_nfit_memdev(struct nfit_mem *nfit_mem)

This line is over 80 characters.

I generally don't see the point of fixing up occasional small incursions
over 80 characters if it makes the code more amenable to grep.

Just use shorter type names or add a line break before the symbol
names.  Too long lines are highly annoying.

Note that "acpi_nfit_memory_map" is the identifier chosen by ACPICA,
I'm just a consumer.  That said I've gone and done a 80 column cleanup
across all the patches.