Patch "media: ipu-bridge: increase sensor_name size" has been added to the 6.6-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    media: ipu-bridge: increase sensor_name size

to the 6.6-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     media-ipu-bridge-increase-sensor_name-size.patch
and it can be found in the queue-6.6 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 9a6cfbbbcfa170f36ac56319adf0deacd3412365
Author: Hans Verkuil <hverkuil-cisco@xxxxxxxxx>
Date:   Sat Sep 23 17:20:49 2023 +0200

    media: ipu-bridge: increase sensor_name size
    
    [ Upstream commit 83d0d4cc1423194b580356966107379490edd02e ]
    
    Fixes this compiler warning:
    
    In file included from include/linux/property.h:14,
                     from include/linux/acpi.h:16,
                     from drivers/media/pci/intel/ipu-bridge.c:4:
    In function 'ipu_bridge_init_swnode_names',
        inlined from 'ipu_bridge_create_connection_swnodes' at drivers/media/pci/intel/ipu-bridge.c:445:2,
        inlined from 'ipu_bridge_connect_sensor' at drivers/media/pci/intel/ipu-bridge.c:656:3:
    include/linux/fwnode.h:81:49: warning: '%u' directive output may be truncated writing between 1 and 3 bytes into a region of size 2 [-Wformat-truncation=]
       81 | #define SWNODE_GRAPH_PORT_NAME_FMT              "port@%u"
          |                                                 ^~~~~~~~~
    drivers/media/pci/intel/ipu-bridge.c:384:18: note: in expansion of macro 'SWNODE_GRAPH_PORT_NAME_FMT'
      384 |                  SWNODE_GRAPH_PORT_NAME_FMT, sensor->link);
          |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~
    include/linux/fwnode.h: In function 'ipu_bridge_connect_sensor':
    include/linux/fwnode.h:81:55: note: format string is defined here
       81 | #define SWNODE_GRAPH_PORT_NAME_FMT              "port@%u"
          |                                                       ^~
    In function 'ipu_bridge_init_swnode_names',
        inlined from 'ipu_bridge_create_connection_swnodes' at drivers/media/pci/intel/ipu-bridge.c:445:2,
        inlined from 'ipu_bridge_connect_sensor' at drivers/media/pci/intel/ipu-bridge.c:656:3:
    include/linux/fwnode.h:81:49: note: directive argument in the range [0, 255]
       81 | #define SWNODE_GRAPH_PORT_NAME_FMT              "port@%u"
          |                                                 ^~~~~~~~~
    drivers/media/pci/intel/ipu-bridge.c:384:18: note: in expansion of macro 'SWNODE_GRAPH_PORT_NAME_FMT'
      384 |                  SWNODE_GRAPH_PORT_NAME_FMT, sensor->link);
          |                  ^~~~~~~~~~~~~~~~~~~~~~~~~~
    drivers/media/pci/intel/ipu-bridge.c:382:9: note: 'snprintf' output between 7 and 9 bytes into a destination of size 7
      382 |         snprintf(sensor->node_names.remote_port,
          |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      383 |                  sizeof(sensor->node_names.remote_port),
          |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
      384 |                  SWNODE_GRAPH_PORT_NAME_FMT, sensor->link);
          |                  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    
    Signed-off-by: Hans Verkuil <hverkuil-cisco@xxxxxxxxx>
    Acked-by: Arnd Bergmann <arnd@xxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/include/media/ipu-bridge.h b/include/media/ipu-bridge.h
index bdc654a455216..783bda6d5cc3f 100644
--- a/include/media/ipu-bridge.h
+++ b/include/media/ipu-bridge.h
@@ -108,7 +108,7 @@ struct ipu_node_names {
 	char ivsc_sensor_port[7];
 	char ivsc_ipu_port[7];
 	char endpoint[11];
-	char remote_port[7];
+	char remote_port[9];
 	char vcm[16];
 };
 



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux