On Saturday 20 January 2018 10:43 PM, David Lechner wrote: > + LPSC(26, 0, gpio, pll1_sysclk2, gpio_clkdev, 0), > + LPSC(27, 0, timer0, pll1_auxclk, timer0_clkdev, 0), > + LPSC(28, 0, timer1, pll1_auxclk, NULL, 0), > + /* REVISIT: why can't this be disabled? */ > + LPSC(29, 0, timer2, pll1_auxclk, timer2_clkdev, > + LPSC_ALWAYS_ENABLED), > + LPSC(31, 0, arm, pll1_sysclk1, NULL, > + LPSC_ALWAYS_ENABLED), IMO, in this case the line break makes it more difficult to read so that checkpatch warning should be ignored. Thanks Sekhar -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html