media: imx-media-csc-scaler: Use a shorter name for driver
authorFabio Estevam <festevam@gmail.com>
Wed, 8 Apr 2020 13:10:05 +0000 (15:10 +0200)
committerMauro Carvalho Chehab <mchehab+huawei@kernel.org>
Tue, 21 Apr 2020 11:26:06 +0000 (13:26 +0200)
Currently v4l2-compliance tool returns the following output:

Compliance test for imx-media-csc-s device /dev/video8:

Driver Info:
        Driver name      : imx-media-csc-s
        Card type        : imx-media-csc-scaler
        Bus info         : platform:imx-media-csc-scaler

The driver name string is limited to 16 characters, so provide
a shorter name in order to get a better output.

While at it, use the same shorter name for driver, card and platform.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
drivers/staging/media/imx/imx-media-csc-scaler.c

index 489b6c1529f31f045f74af70e864cfac0936d4e6..fab1155a5958c1fcc90fbb8cd4465910a4ed8020 100644 (file)
@@ -25,6 +25,8 @@
 
 #define fh_to_ctx(__fh)        container_of(__fh, struct ipu_csc_scaler_ctx, fh)
 
+#define IMX_CSC_SCALER_NAME "imx-csc-scaler"
+
 enum {
        V4L2_M2M_SRC = 0,
        V4L2_M2M_DST = 1,
@@ -150,10 +152,10 @@ err:
 static int ipu_csc_scaler_querycap(struct file *file, void *priv,
                                   struct v4l2_capability *cap)
 {
-       strscpy(cap->driver, "imx-media-csc-scaler", sizeof(cap->driver));
-       strscpy(cap->card, "imx-media-csc-scaler", sizeof(cap->card));
-       strscpy(cap->bus_info, "platform:imx-media-csc-scaler",
-               sizeof(cap->bus_info));
+       strscpy(cap->driver, IMX_CSC_SCALER_NAME, sizeof(cap->driver));
+       strscpy(cap->card, IMX_CSC_SCALER_NAME, sizeof(cap->card));
+       snprintf(cap->bus_info, sizeof(cap->bus_info),
+                "platform:%s", IMX_CSC_SCALER_NAME);
 
        return 0;
 }