Re: [PATCH v2 0/6] Stop NULLifying match pointer in of_match_device()
From: Rob Herring <robh@kernel.org>
Date: 2021-01-27 07:00:02
Also in:
linux-devicetree, linux-hwmon, linux-media, linux-omap, linux-renesas-soc, linux-serial, lkml
On Fri, Jan 22, 2021 at 9:44 PM Stephen Boyd [off-list ref] wrote:
(This is a continuation of this series[1] per Rob's request. I've picked up the acks, etc. with b4 and compile tested the patches along with an arm64 allmodconfig build. Presumably Rob will pick these up directly.) of_match_device() uses of_match_ptr() to make the match table argument NULL via the pre-processor when CONFIG_OF=n. This makes life harder for compilers who think that match tables are never used and warn about unused variables when CONFIG_OF=n. This series changes various callers to use of_device_get_match_data() instead, which doesn't have this problem, and removes the of_match_ptr() usage from of_match_device() so that the compiler can stop complaining about unused variables. It will do dead code elimination instead and remove the match table if it isn't actually used. [1] https://lore.kernel.org/r/20191004214334.149976-1-swboyd@chromium.org (local) Cc: Alexandre Torgue <redacted> Cc: Arnd Bergmann <arnd@arndb.de> Cc: "David S. Miller" <davem@davemloft.net> Cc: Frank Rowand <redacted> Cc: Geert Uytterhoeven <geert@linux-m68k.org> Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org> Cc: Grygorii Strashko <grygorii.strashko@ti.com> Cc: Guenter Roeck <linux@roeck-us.net> Cc: Jacopo Mondi <jacopo@jmondi.org> Cc: Jean Delvare <jdelvare@suse.com> Cc: Jiri Slaby <redacted> Cc: <redacted> Cc: <redacted>, Cc: <redacted> Cc: <redacted> Cc: <redacted> Cc: <redacted> Cc: <redacted> Cc: Mauro Carvalho Chehab <mchehab@kernel.org> Cc: Maxime Coquelin <mcoquelin.stm32@gmail.com> Cc: Richard Leitner <richard.leitner@skidata.com> Cc: Rob Herring <robh+dt@kernel.org> Stephen Boyd (6): media: renesas-ceu: Use of_device_get_match_data() drivers: net: davinci_mdio: Use of_device_get_match_data() serial: stm32: Use of_device_get_match_data() usb: usb251xb: Use of_device_get_match_data() hwmon: (lm70) Avoid undefined reference to match table of/device: Don't NULLify match table in of_match_device() with CONFIG_OF=n drivers/hwmon/lm70.c | 2 +- drivers/media/platform/renesas-ceu.c | 2 +- drivers/net/ethernet/ti/davinci_mdio.c | 12 ++--- drivers/tty/serial/stm32-usart.c | 71 ++++++++++++-------------- drivers/tty/serial/stm32-usart.h | 2 +- drivers/usb/misc/usb251xb.c | 12 ++--- include/linux/of_device.h | 4 +- 7 files changed, 47 insertions(+), 58 deletions(-)
Series applied. Rob