kernel: nvidia-oot: Fix Static issues S11/27

Fix TAINTED_SCALAR and Directive 4.14

Jira CAMERASW-29899

Change-Id: I1712d392b8271be0db912d536027d960f343e3fe
Signed-off-by: Yuyuan Chen <yuyuanc@nvidia.com>
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3273387
Reviewed-by: Sumeet Gupta <sumeetg@nvidia.com>
GVS: buildbot_gerritrpt <buildbot_gerritrpt@nvidia.com>
Reviewed-by: Frank Chen <frankc@nvidia.com>
Reviewed-by: Ankur Pawar <ankurp@nvidia.com>
Tested-by: mobile promotions <svcmobile_promotions@nvidia.com>
Reviewed-by: mobile promotions <svcmobile_promotions@nvidia.com>
Reviewed-by: Ryan Li <ryanli@nvidia.com>
This commit is contained in:
Yuyuan Chen
2025-01-06 02:18:45 +00:00
committed by Jon Hunter
parent 8e33949582
commit 188b622e75
4 changed files with 32 additions and 10 deletions

View File

@@ -1,5 +1,5 @@
// SPDX-License-Identifier: GPL-2.0-only // SPDX-License-Identifier: GPL-2.0-only
// SPDX-FileCopyrightText: Copyright (c) 2017-2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. // SPDX-FileCopyrightText: Copyright (c) 2017-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
/** /**
* @file drivers/media/platform/tegra/camera/fusa-capture/capture-isp-channel.c * @file drivers/media/platform/tegra/camera/fusa-capture/capture-isp-channel.c
@@ -335,7 +335,7 @@ static long isp_channel_ioctl(
switch (_IOC_NR(cmd)) { switch (_IOC_NR(cmd)) {
case _IOC_NR(ISP_CAPTURE_SETUP): { case _IOC_NR(ISP_CAPTURE_SETUP): {
struct isp_capture_setup setup; struct isp_capture_setup setup = {};
if (copy_from_user(&setup, ptr, sizeof(setup))) if (copy_from_user(&setup, ptr, sizeof(setup)))
break; break;
@@ -389,7 +389,7 @@ static long isp_channel_ioctl(
} }
case _IOC_NR(ISP_CAPTURE_REQUEST): { case _IOC_NR(ISP_CAPTURE_REQUEST): {
struct isp_capture_req req; struct isp_capture_req req = {};
if (copy_from_user(&req, ptr, sizeof(req))) if (copy_from_user(&req, ptr, sizeof(req)))
break; break;
@@ -413,7 +413,7 @@ static long isp_channel_ioctl(
} }
case _IOC_NR(ISP_CAPTURE_PROGRAM_REQUEST): { case _IOC_NR(ISP_CAPTURE_PROGRAM_REQUEST): {
struct isp_program_req program_req; struct isp_program_req program_req = {};
if (copy_from_user(&program_req, ptr, sizeof(program_req))) if (copy_from_user(&program_req, ptr, sizeof(program_req)))
break; break;
@@ -434,7 +434,7 @@ static long isp_channel_ioctl(
} }
case _IOC_NR(ISP_CAPTURE_REQUEST_EX): { case _IOC_NR(ISP_CAPTURE_REQUEST_EX): {
struct isp_capture_req_ex req; struct isp_capture_req_ex req = {};
if (copy_from_user(&req, ptr, sizeof(req))) if (copy_from_user(&req, ptr, sizeof(req)))
break; break;

View File

@@ -1,5 +1,5 @@
// SPDX-License-Identifier: GPL-2.0-only // SPDX-License-Identifier: GPL-2.0-only
// SPDX-FileCopyrightText: Copyright (c) 2017-2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. // SPDX-FileCopyrightText: Copyright (c) 2017-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
/** /**
* @file drivers/media/platform/tegra/camera/fusa-capture/capture-isp.c * @file drivers/media/platform/tegra/camera/fusa-capture/capture-isp.c
@@ -578,6 +578,11 @@ static void isp_capture_request_unpin(
} }
mutex_lock(&capture->capture_desc_ctx.unpins_list_lock); mutex_lock(&capture->capture_desc_ctx.unpins_list_lock);
if (buffer_index >= capture->program_desc_ctx.queue_depth) {
dev_err(chan->isp_dev,
"%s: buffer index is out of bound\n", __func__);
return;
}
unpins = &capture->capture_desc_ctx.unpins_list[buffer_index]; unpins = &capture->capture_desc_ctx.unpins_list[buffer_index];
if (unpins->num_unpins != 0U) { if (unpins->num_unpins != 0U) {
for (i = 0; i < unpins->num_unpins; i++) for (i = 0; i < unpins->num_unpins; i++)
@@ -609,6 +614,11 @@ static void isp_capture_program_request_unpin(
} }
mutex_lock(&capture->program_desc_ctx.unpins_list_lock); mutex_lock(&capture->program_desc_ctx.unpins_list_lock);
if (buffer_index >= capture->program_desc_ctx.queue_depth) {
dev_err(chan->isp_dev,
"%s: buffer index is out of bound\n", __func__);
return;
}
unpins = &capture->program_desc_ctx.unpins_list[buffer_index]; unpins = &capture->program_desc_ctx.unpins_list[buffer_index];
if (unpins->num_unpins != 0U) { if (unpins->num_unpins != 0U) {
for (i = 0; i < unpins->num_unpins; i++) for (i = 0; i < unpins->num_unpins; i++)
@@ -1115,6 +1125,12 @@ void isp_get_nvhost_device(
struct tegra_capture_isp_data *info = struct tegra_capture_isp_data *info =
platform_get_drvdata(chan->isp_capture_pdev); platform_get_drvdata(chan->isp_capture_pdev);
if (isp_inst >= MAX_ISP_UNITS) {
dev_err(chan->isp_dev,
"%s: ISP unit index is out of bound\n", __func__);
return;
}
chan->isp_dev = &info->isp_pdevices[isp_inst]->dev; chan->isp_dev = &info->isp_pdevices[isp_inst]->dev;
chan->ndev = info->isp_pdevices[isp_inst]; chan->ndev = info->isp_pdevices[isp_inst];
} }

View File

@@ -1,5 +1,5 @@
// SPDX-License-Identifier: GPL-2.0-only // SPDX-License-Identifier: GPL-2.0-only
// SPDX-FileCopyrightText: Copyright (c) 2017-2024 NVIDIA CORPORATION & AFFILIATES. All rights reserved. // SPDX-FileCopyrightText: Copyright (c) 2017-2025 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
/** /**
* @file drivers/media/platform/tegra/camera/fusa-capture/capture-vi-channel.c * @file drivers/media/platform/tegra/camera/fusa-capture/capture-vi-channel.c
@@ -403,7 +403,7 @@ static long vi_channel_ioctl(
switch (_IOC_NR(cmd)) { switch (_IOC_NR(cmd)) {
case _IOC_NR(VI_CAPTURE_SETUP): { case _IOC_NR(VI_CAPTURE_SETUP): {
struct vi_capture_setup setup; struct vi_capture_setup setup = {};
if (copy_from_user(&setup, ptr, sizeof(setup))) if (copy_from_user(&setup, ptr, sizeof(setup)))
break; break;
@@ -524,7 +524,7 @@ static long vi_channel_ioctl(
} }
case _IOC_NR(VI_CAPTURE_SET_CONFIG): { case _IOC_NR(VI_CAPTURE_SET_CONFIG): {
struct vi_capture_control_msg msg; struct vi_capture_control_msg msg = {};
if (copy_from_user(&msg, ptr, sizeof(msg))) if (copy_from_user(&msg, ptr, sizeof(msg)))
break; break;

View File

@@ -1,5 +1,5 @@
// SPDX-License-Identifier: GPL-2.0-only // SPDX-License-Identifier: GPL-2.0-only
/* SPDX-FileCopyrightText: Copyright (c) 2017-2024 NVIDIA CORPORATION & AFFILIATES. /* SPDX-FileCopyrightText: Copyright (c) 2017-2025 NVIDIA CORPORATION & AFFILIATES.
* All rights reserved. * All rights reserved.
* *
* *
@@ -580,6 +580,12 @@ void vi_get_nvhost_device(
struct tegra_capture_vi_data *info = struct tegra_capture_vi_data *info =
platform_get_drvdata(chan->vi_capture_pdev); platform_get_drvdata(chan->vi_capture_pdev);
if (setup->csi_stream_id >= MAX_NVCSI_STREAM_IDS) {
dev_err(chan->dev,
"%s: csi stream id is out of bound\n", __func__);
return;
}
vi_inst = info->vi_instance_table[setup->csi_stream_id]; vi_inst = info->vi_instance_table[setup->csi_stream_id];
if (vi_inst >= MAX_VI_UNITS) { if (vi_inst >= MAX_VI_UNITS) {