media: drop use of MEDIA_API_VERSION
authorHans Verkuil <hans.verkuil@cisco.com>
Fri, 28 Jul 2017 07:25:06 +0000 (03:25 -0400)
committerMauro Carvalho Chehab <mchehab@s-opensource.com>
Tue, 8 Aug 2017 10:03:15 +0000 (06:03 -0400)
Set media_version to LINUX_VERSION_CODE, just as we did for
driver_version.

Nobody ever rememebers to update the version number, but
LINUX_VERSION_CODE will always be updated.

Move the MEDIA_API_VERSION define to the ifndef __KERNEL__ section of the
media.h header. That way kernelspace can't accidentally start to use
it again.

Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
drivers/media/media-device.c
include/uapi/linux/media.h

index 979e4307d248c3e4b6cffc79964d3f1fd3863441..e79f72b8b858077f7b646caf7606afb42a830411 100644 (file)
@@ -69,9 +69,9 @@ static int media_device_get_info(struct media_device *dev,
        strlcpy(info->serial, dev->serial, sizeof(info->serial));
        strlcpy(info->bus_info, dev->bus_info, sizeof(info->bus_info));
 
-       info->media_version = MEDIA_API_VERSION;
+       info->media_version = LINUX_VERSION_CODE;
+       info->driver_version = info->media_version;
        info->hw_revision = dev->hw_revision;
-       info->driver_version = LINUX_VERSION_CODE;
 
        return 0;
 }
index fac96c64fe513b3e17dff640cfa516683e91e553..4865f1e713398b63f41fca7a84fae1637fc44385 100644 (file)
@@ -30,8 +30,6 @@
 #include <linux/types.h>
 #include <linux/version.h>
 
-#define MEDIA_API_VERSION      KERNEL_VERSION(0, 1, 0)
-
 struct media_device_info {
        char driver[16];
        char model[32];
@@ -187,6 +185,9 @@ struct media_device_info {
 #define MEDIA_ENT_T_V4L2_SUBDEV_LENS   MEDIA_ENT_F_LENS
 #define MEDIA_ENT_T_V4L2_SUBDEV_DECODER        MEDIA_ENT_F_ATV_DECODER
 #define MEDIA_ENT_T_V4L2_SUBDEV_TUNER  MEDIA_ENT_F_TUNER
+
+/* Obsolete symbol for media_version, no longer used in the kernel */
+#define MEDIA_API_VERSION              KERNEL_VERSION(0, 1, 0)
 #endif
 
 /* Entity flags */