On Mon, Sep 27, 2021 at 04:45:01PM +0200, Wolfgang Denk wrote: > Dear Adam, > > commit 25e4ff45b17 "ARM: omap3_logic: Enable OMAP EHCI support for > SOM-LV Boards" added (among other things) these lines: > > ... > 243 MUX_VAL(CP(MCSPI2_SOMI), (IEN | PTD | DIS | M0)); > /*HSUSB2_DATA5*/ > 244 MUX_VAL(CP(MCSPI2_CS0), (IEN | PTD | EN | M0)); > /*HSUSB2_DATA6*/ > 245 MUX_VAL(CP(MCSPI2_CLK), (IEN | PTD | DIS | M0)); > /*HSUSB2_DATA7*/ > 246 MUX_VAL(CP(SYS_BOOT2), (IEN | PTD | DIS | M4)) /* GPIO_4 > */ > 247 MUX_VAL(CP(ETK_D10_ES2), (IDIS | PTU | DIS | M3)); > /*HSUSB2_CLK*/ > 248 MUX_VAL(CP(ETK_D11_ES2), (IDIS | PTU | DIS | M3)); > /*HSUSB2_STP*/ > 249 MUX_VAL(CP(ETK_D12_ES2), (IEN | PTU | DIS | M3)); > /*HSUSB2_DIR*/ > ... > > All the entries in set_muxconf_regs() have a terminating semicolon - > all except the entry in line # 246. > > I reckon this is a mistake and should be fixed?
Interesting. MUX_VAL is from arch/arm/include/asm/arch-omap3/mux.h and includes a semicolon, so that's how this didn't trip anything up. Most of the time it's not used with one either, from a quick grep. -- Tom
signature.asc
Description: PGP signature