Messages in this thread | | | Date | Wed, 18 Jan 2023 11:31:18 +0100 | From | Johan Hovold <> | Subject | Re: [PATCH v4 11/19] x86/ioapic: Use irq_domain_create_hierarchy() |
| |
On Tue, Jan 17, 2023 at 10:41:36PM +0100, Thomas Gleixner wrote: > On Mon, Jan 16 2023 at 14:50, Johan Hovold wrote: > > > > - ip->irqdomain = irq_domain_create_linear(fn, hwirqs, cfg->ops, > > - (void *)(long)ioapic); > > - > > + ip->irqdomain = irq_domain_create_hierarchy(parent, 0, hwirqs, fn, > > + cfg->ops, > > + (void *)(long)ioapic); > > The 80 character limit has been lifted quite some time ago. Please use > the 100 which are now the norm.
Not everyone agrees, including coding-style.rst:
The preferred limit on the length of a single line is 80 columns.
Statements longer than 80 columns should be broken into sensible chunks, unless exceeding 80 columns significantly increases readability and does not hide information.
I go above 80 chars when it improves readability, but it's still a soft limit for many of us.
I'll change the above, but not sure trying too hard to fit everything in one line really improves things in cases like:
- uv_domain = irq_domain_create_hierarchy(x86_vector_domain, 0, 0, fn, - &uv_domain_ops, NULL); + uv_domain = irq_domain_create_hierarchy(x86_vector_domain, 0, 0, fn, &uv_domain_ops, NULL);
Johan
| |