[PATCH] dm: Use parenthesis for the device_get_ops macro argument

Use parenthesis for the device_get_ops macro argument. This prevents errors when using an expression for the parameter.
Signed-off-by: Sughosh Ganu sughosh.ganu@linaro.org --- include/dm/device.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/dm/device.h b/include/dm/device.h index f790041706..77c27ba92e 100644 --- a/include/dm/device.h +++ b/include/dm/device.h @@ -212,7 +212,7 @@ struct udevice_rt { #define DM_MAX_SEQ_STR 3
/* Returns the operations for a device */ -#define device_get_ops(dev) (dev->driver->ops) +#define device_get_ops(dev) ((dev)->driver->ops)
#if CONFIG_IS_ENABLED(OF_PLATDATA_RT) u32 dev_get_flags(const struct udevice *dev);

On Sat, 29 Jan 2022 at 12:21, Sughosh Ganu sughosh.ganu@linaro.org wrote:
Use parenthesis for the device_get_ops macro argument. This prevents errors when using an expression for the parameter.
Signed-off-by: Sughosh Ganu sughosh.ganu@linaro.org
include/dm/device.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org

On Sat, 29 Jan 2022 at 12:21, Sughosh Ganu sughosh.ganu@linaro.org wrote:
Use parenthesis for the device_get_ops macro argument. This prevents errors when using an expression for the parameter.
Signed-off-by: Sughosh Ganu sughosh.ganu@linaro.org
include/dm/device.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass sjg@chromium.org
Applied to u-boot-dm, thanks!
participants (2)
-
Simon Glass
-
Sughosh Ganu