[Software version control] How to version control nvidia and custom code?

[phdm]

@DaneLLL

If your patch is in the driver code, you may share to us and we will review to include it into future release.

I do not speak for the OP, but that’s what I do, and I am still waiting for feedback. See

Bandwidth problem when converting mipi RAW12 to T_R16_I or T_R16 on TX2

Here’s my patch proposal From 7bf0d69146a07dc313c79bb892e8d7f8a7306eef Mon Sep 17 00:00:00 2001 From: Philippe De Muyter <philippe.demuyter@macq.eu> Date: Thu, 25 Jan 2024 16:13:00 +0100 Subject: [PATCH 1/2] Revert "camera: correct the BW calculations" MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The comment justifying "camera: correct the BW calculations" is wrong. fmtinfo->bpp is documented in include/media/tegra_camera_core.h as * @bpp: bytes …

BUG: BW requirement is not updated when VI output format is changed (with patch proposal)

Hello, when trying to fix https://forums.developer.nvidia.com/t/bandwidth-problem-when-converting-mipi-raw12-to-t-r16-i-or-t-r16-on-tx2/279468/8 , I noticed that the bw field of cdev does not follow the output format of VI, and actually never changes, regardless of the number of bytes per pixel. Here is a fix proposal, in a dirty patch to keep it as small as possible. Please comment. From ec0c13d25308701ce01f0e6321ce8ade26d2deaa Mon Sep 17 00:00:00 2001 From: Philippe De Muyter <philippe.dem…

and

Pixels at right end of lines are garbled when capturing with v4l2, not when using argus

Hello Shane, Applying your suggestion, the following patch solves the problem diff --git a/include/media/tegra_camera_core.h b/include/media/tegra_camera_core.h index 788cf77dc..0a534d47b 100644 --- a/include/media/tegra_camera_core.h +++ b/include/media/tegra_camera_core.h @@ -23,7 +23,7 @@ /* Width alignment */ #define TEGRA_WIDTH_ALIGNMENT 1 /* Stride alignment */ -#define TEGRA_STRIDE_ALIGNMENT 1 +#define TEGRA_STRIDE_ALIGNMENT 64 /* Height alignment */ #define TEGRA_HEIGHT_ALIGNMENT…