V4L/DVB: V4L2: avoid name conflicts in macros
authorGuennadi Liakhovetski <g.liakhovetski@gmx.de>
Thu, 12 Aug 2010 20:16:00 +0000 (17:16 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Thu, 21 Oct 2010 03:04:48 +0000 (01:04 -0200)
"sd" and "err" are too common names to be used in macros for local variables.
Prefix them with an underscore to avoid name clashing.

[mchehab@redhat.com: whitespace cleanups]
Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Reviewed-by: Andy Walls <awalls@md.metrocast.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx18/cx18-driver.h
drivers/media/video/ivtv/ivtv-driver.h
include/media/v4l2-device.h

index 9bc51a99376bc22231e7c97977d26261de4974b4..77be58c1096bd2c6dad5e854dbfc8f8b18b872e2 100644 (file)
@@ -674,18 +674,25 @@ static inline int cx18_raw_vbi(const struct cx18 *cx)
 
 /* Call the specified callback for all subdevs with a grp_id bit matching the
  * mask in hw (if 0, then match them all). Ignore any errors. */
-#define cx18_call_hw(cx, hw, o, f, args...) \
-       __v4l2_device_call_subdevs(&(cx)->v4l2_dev, \
-                                  !(hw) || (sd->grp_id & (hw)), o, f , ##args)
+#define cx18_call_hw(cx, hw, o, f, args...)                            \
+       do {                                                            \
+               struct v4l2_subdev *__sd;                               \
+               __v4l2_device_call_subdevs_p(&(cx)->v4l2_dev, __sd,     \
+                       !(hw) || (__sd->grp_id & (hw)), o, f , ##args); \
+       } while (0)
 
 #define cx18_call_all(cx, o, f, args...) cx18_call_hw(cx, 0, o, f , ##args)
 
 /* Call the specified callback for all subdevs with a grp_id bit matching the
  * mask in hw (if 0, then match them all). If the callback returns an error
  * other than 0 or -ENOIOCTLCMD, then return with that error code. */
-#define cx18_call_hw_err(cx, hw, o, f, args...) \
-       __v4l2_device_call_subdevs_until_err( \
-                  &(cx)->v4l2_dev, !(hw) || (sd->grp_id & (hw)), o, f , ##args)
+#define cx18_call_hw_err(cx, hw, o, f, args...)                                \
+({                                                                     \
+       struct v4l2_subdev *__sd;                                       \
+       __v4l2_device_call_subdevs_until_err_p(&(cx)->v4l2_dev,         \
+                       __sd, !(hw) || (__sd->grp_id & (hw)), o, f,     \
+                       ##args);                                        \
+})
 
 #define cx18_call_all_err(cx, o, f, args...) \
        cx18_call_hw_err(cx, 0, o, f , ##args)
index 75803141481e30aee323ca4ee14fa34ebea88590..04bacdbd10bbf6053fbb0e9ff162bfd12bf1d357 100644 (file)
@@ -811,15 +811,23 @@ static inline int ivtv_raw_vbi(const struct ivtv *itv)
 /* Call the specified callback for all subdevs matching hw (if 0, then
    match them all). Ignore any errors. */
 #define ivtv_call_hw(itv, hw, o, f, args...)                           \
-       __v4l2_device_call_subdevs(&(itv)->v4l2_dev, !(hw) || (sd->grp_id & (hw)), o, f , ##args)
+       do {                                                            \
+               struct v4l2_subdev *__sd;                               \
+               __v4l2_device_call_subdevs_p(&(itv)->v4l2_dev, __sd,    \
+                       !(hw) || (__sd->grp_id & (hw)), o, f , ##args); \
+       } while (0)
 
 #define ivtv_call_all(itv, o, f, args...) ivtv_call_hw(itv, 0, o, f , ##args)
 
 /* Call the specified callback for all subdevs matching hw (if 0, then
    match them all). If the callback returns an error other than 0 or
    -ENOIOCTLCMD, then return with that error code. */
-#define ivtv_call_hw_err(itv, hw, o, f, args...)               \
-       __v4l2_device_call_subdevs_until_err(&(itv)->v4l2_dev, !(hw) || (sd->grp_id & (hw)), o, f , ##args)
+#define ivtv_call_hw_err(itv, hw, o, f, args...)                       \
+({                                                                     \
+       struct v4l2_subdev *__sd;                                       \
+       __v4l2_device_call_subdevs_until_err_p(&(itv)->v4l2_dev, __sd,  \
+               !(hw) || (__sd->grp_id & (hw)), o, f , ##args);         \
+})
 
 #define ivtv_call_all_err(itv, o, f, args...) ivtv_call_hw_err(itv, 0, o, f , ##args)
 
index 8bcbd7a0271cbb6c5143c1f40de2a211d6845baf..6648036b728d7e32beb09fb91757931265780823 100644 (file)
@@ -101,46 +101,67 @@ void v4l2_device_unregister_subdev(struct v4l2_subdev *sd);
 /* Call the specified callback for all subdevs matching the condition.
    Ignore any errors. Note that you cannot add or delete a subdev
    while walking the subdevs list. */
-#define __v4l2_device_call_subdevs(v4l2_dev, cond, o, f, args...)      \
+#define __v4l2_device_call_subdevs_p(v4l2_dev, sd, cond, o, f, args...)        \
        do {                                                            \
-               struct v4l2_subdev *sd;                                 \
+               list_for_each_entry((sd), &(v4l2_dev)->subdevs, list)   \
+                       if ((cond) && (sd)->ops->o && (sd)->ops->o->f)  \
+                               (sd)->ops->o->f((sd) , ##args);         \
+       } while (0)
+
+#define __v4l2_device_call_subdevs(v4l2_dev, cond, o, f, args...)      \
+       do {                                                            \
+               struct v4l2_subdev *__sd;                               \
                                                                        \
-               list_for_each_entry(sd, &(v4l2_dev)->subdevs, list)     \
-                       if ((cond) && sd->ops->o && sd->ops->o->f)      \
-                               sd->ops->o->f(sd , ##args);             \
+               __v4l2_device_call_subdevs_p(v4l2_dev, __sd, cond, o,   \
+                                               f , ##args);            \
        } while (0)
 
 /* Call the specified callback for all subdevs matching the condition.
    If the callback returns an error other than 0 or -ENOIOCTLCMD, then
    return with that error code. Note that you cannot add or delete a
    subdev while walking the subdevs list. */
-#define __v4l2_device_call_subdevs_until_err(v4l2_dev, cond, o, f, args...) \
+#define __v4l2_device_call_subdevs_until_err_p(v4l2_dev, sd, cond, o, f, args...) \
 ({                                                                     \
-       struct v4l2_subdev *sd;                                         \
-       long err = 0;                                                   \
+       long __err = 0;                                                 \
                                                                        \
-       list_for_each_entry(sd, &(v4l2_dev)->subdevs, list) {           \
-               if ((cond) && sd->ops->o && sd->ops->o->f)              \
-                       err = sd->ops->o->f(sd , ##args);               \
-               if (err && err != -ENOIOCTLCMD)                         \
+       list_for_each_entry((sd), &(v4l2_dev)->subdevs, list) {         \
+               if ((cond) && (sd)->ops->o && (sd)->ops->o->f)          \
+                       __err = (sd)->ops->o->f((sd) , ##args);         \
+               if (__err && __err != -ENOIOCTLCMD)                     \
                        break;                                          \
        }                                                               \
-       (err == -ENOIOCTLCMD) ? 0 : err;                                \
+       (__err == -ENOIOCTLCMD) ? 0 : __err;                            \
+})
+
+#define __v4l2_device_call_subdevs_until_err(v4l2_dev, cond, o, f, args...) \
+({                                                                     \
+       struct v4l2_subdev *__sd;                                       \
+       __v4l2_device_call_subdevs_until_err_p(v4l2_dev, __sd, cond, o, \
+                                               f, args...);            \
 })
 
 /* Call the specified callback for all subdevs matching grp_id (if 0, then
    match them all). Ignore any errors. Note that you cannot add or delete
    a subdev while walking the subdevs list. */
-#define v4l2_device_call_all(v4l2_dev, grpid, o, f, args...)           \
-       __v4l2_device_call_subdevs(v4l2_dev,                            \
-                       !(grpid) || sd->grp_id == (grpid), o, f , ##args)
+#define v4l2_device_call_all(v4l2_dev, grpid, o, f, args...)           \
+       do {                                                            \
+               struct v4l2_subdev *__sd;                               \
+                                                                       \
+               __v4l2_device_call_subdevs_p(v4l2_dev, __sd,            \
+                       !(grpid) || __sd->grp_id == (grpid), o, f ,     \
+                       ##args);                                        \
+       } while (0)
 
 /* Call the specified callback for all subdevs matching grp_id (if 0, then
    match them all). If the callback returns an error other than 0 or
    -ENOIOCTLCMD, then return with that error code. Note that you cannot
    add or delete a subdev while walking the subdevs list. */
 #define v4l2_device_call_until_err(v4l2_dev, grpid, o, f, args...)     \
-       __v4l2_device_call_subdevs_until_err(v4l2_dev,                  \
-                      !(grpid) || sd->grp_id == (grpid), o, f , ##args)
+({                                                                     \
+       struct v4l2_subdev *__sd;                                       \
+       __v4l2_device_call_subdevs_until_err_p(v4l2_dev, __sd,          \
+                       !(grpid) || __sd->grp_id == (grpid), o, f ,     \
+                       ##args);                                        \
+})
 
 #endif