nvidia-tegra-modules.git/drivers/media, branch gpu-paging

driver:imx185:Add info message for expected i2c error.

2022-01-18T08:24:34+00:00

Sensor not return ACK when SW_RESET is transferred.

bug 200741759

Change-Id: I77639f84bcda653ba2d3adfd68727725a0c00e1f
Signed-off-by: Praveen AC 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2653996
Tested-by: mobile promotions 
Reviewed-by: Anubhav Rai 
Reviewed-by: Shubham Chandra 
Reviewed-by: Ajith Kumar 
Reviewed-by: Sudhir Vyas 
Reviewed-by: Frank Chen 
Reviewed-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

driver: imx219: Fix for i2c error write mode

2022-01-10T21:10:00+00:00

Add enough time delay after sensor soft reset
to avoid i2c error write mode.

bug 200768012

Change-Id: Iacabee71957168f4eec15da45194d52bd1464fbf
Signed-off-by: Praveen AC 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2649706
Tested-by: mobile promotions 
Reviewed-by: svcacv 
Reviewed-by: Nithin Varghese 
Reviewed-by: Anubhav Rai 
Reviewed-by: Shubham Chandra 
Reviewed-by: Sudhir Vyas 
Reviewed-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

media: tegra: camera: vi2: refactor low-latency mode code

2022-01-07T14:10:42+00:00

A frame capture may fail somewhere between the frame start and the frame
end,e.g., the below error prints:

2021-11-04T10:11:31.561+00:00 kernel: [  794.777229] tc358840 1-001f: event: resolution change (EQ_BYPS=1): 3840x2160p29.94 (4400x2250)
2021-11-04T10:11:32.167+00:00 kernel: [  795.379917] video4linux video0: TEGRA_VI_CSI_ERROR_STATUS 0x00000000
2021-11-04T10:11:32.167+00:00 kernel: [  795.386292] video4linux video0: TEGRA_VI_CSI_ERROR_STATUS 0x00000000
2021-11-04T10:11:32.191+00:00 kernel: [  795.404274] vi 54080000.vi: tegra_channel_error_status:error 20022 frame 0

This means checking SOF in the capture thread doesn't help avoid a failure
in the release thread. Hence we can simplify the capture thread to program only
the capture released settings, skip the checking of SOF and leave the checking
of vi/csi status and the recovery process to be done in the release thread.

Also simplify vi2_channel_stop_streaming() to not invoke tegra_channel_capture_done()
as the buffers shall be well handled in tegra_channel_queued_buf_done() with
VB2_BUF_STATE_ERROR state.

In the original implementation, chan->syncpt[index][] will be re-allocated for
every single stop_streaming()/start_streaming() pair, so the syncpt IDs may
change. This is causing the problem: the syncpt IDs armed for the previous
failing capture are still queued in the FIFO and will be used for the next
successfull frame capture, however new syncpt IDs are allocated for the new
coming frames, e.g, the below error prints:

2021-11-23T08:12:59.038+00:00 kernel: [  590.029312] tegra_channel_capture_frame_multi_thread[570]: rv 4, cv 3, stm Y
2021-11-23T08:12:59.038+00:00 kernel: [  590.036865] tegra_channel_enable_stream[212] -- begin
2021-11-23T08:12:59.044+00:00 kernel: [  590.042437] tegra_channel_enable_stream[221] -- end
2021-11-23T08:12:59.263+00:00 kernel: [  590.251676] video4linux video0: frame start syncpt timeout!0
2021-11-23T08:12:59.263+00:00 kernel: [  590.257419] tegra_channel_ec_recover[337] -- begin

2021-12-03T11:14:36.763+00:00 kernel: [ 1049.109289] vi2_channel_start_streaming[912] -- begin
2021-12-03T11:14:36.763+00:00 kernel: [ 1049.115121] vi2_channel_start_streaming[949]: syncpt id 24, max 5064, min 5064
2021-12-03T11:14:36.771+00:00 kernel: [ 1049.123116] vi2_channel_start_streaming[949]: syncpt id 28, max 1614, min 1614
--> the syncpt id being used for this streaming are 24/28, these two syncpts are used since the first streaming
...
2021-12-03T11:14:39.244+00:00 kernel: [ 1051.597885] tegra_channel_capture_frame_multi_thread[624] -- end: release_thresh 5101
2021-12-03T11:14:39.303+00:00 kernel: [ 1051.663550] tegra_channel_release_frame[701] -- end: buf->thresh 5101
2021-12-03T11:14:39.350+00:00 kernel: [ 1051.703617] vi2_channel_stop_streaming[1065] -- end
--> no problem observed, and capture cancelled due to stop_streaming
...
2021-12-03T11:15:09.524+00:00 kernel: [ 1081.871259] vi2_channel_start_streaming[912] -- begin
2021-12-03T11:15:09.524+00:00 kernel: [ 1081.876860] vi2_channel_start_streaming[949]: syncpt id 36, max 0, min 0
2021-12-03T11:15:09.524+00:00 kernel: [ 1081.884187] vi2_channel_start_streaming[949]: syncpt id 38, max 0, min 0
--> the syncpt id being used now are changed to 36/38
...
2021-12-03T11:15:09.534+00:00 kernel: [ 1081.891728] tegra_channel_capture_frame_multi_thread[514] -- begin
2021-12-03T11:15:09.534+00:00 kernel: [ 1081.891771] vi2_channel_start_streaming[992] -- end
2021-12-03T11:15:09.552+00:00 kernel: [ 1081.904384] tegra_channel_capture_frame_multi_thread[569]: rv 119, cv 119, stm N
2021-12-03T11:15:09.606+00:00 kernel: [ 1081.958030] tegra_channel_capture_frame_multi_thread[624] -- end: release_thresh 1
2021-12-03T11:15:09.606+00:00 kernel: [ 1081.958086] tegra_channel_release_frame[669] -- begin
2021-12-03T11:15:09.817+00:00 kernel: [ 1082.162133] video4linux video0: tegra_channel_release_frame: MW_ACK_DONE syncpoint time out!0
2021-12-03T11:15:09.817+00:00 kernel: [ 1082.170683] tegra_channel_release_frame[680]: buf->thresh[0] 1
2021-12-03T11:15:09.817+00:00 kernel: [ 1082.176972] tegra_channel_release_frame[683]: syncpt 36, hw val 0
2021-12-03T11:15:09.829+00:00 kernel: [ 1082.183715] tegra_channel_ec_recover[337] -- begin

Given Hw doesn't support SYNCPT FIFO reset per stream, reuse the same syncpt IDs
until the channel is powered off.

Also skip invoking tegra_channel_capture_done() from vi2_channel_stop_streaming()
and tegra_channel_release_frame() from tegra_channel_stop_kthreads() as streaming
shall be stoped immediately and the queued buffers shall be well handled in
tegra_channel_queued_buf_done() with the VB2_BUF_STATE_ERROR state.

Bug 3423623

Change-Id: I53280bc9b1b3c33054d766aa920eb082b3311d92
Signed-off-by: Ken Chang 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2645502
Reviewed-by: Frank Chen 
Reviewed-by: Jerry Chang 
Reviewed-by: Anubhav Rai 
Reviewed-by: Bibek Basu 
Reviewed-by: mobile promotions 
Tested-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

media:i2c: fix imx477 unable streaming

2021-11-24T12:39:22+00:00

Fix timeout for VI mode capture on second camera.

bug 3429708

Change-Id: I18c2a3ec6585325124da1393275bd1406f74720a
Signed-off-by: snchen 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2622692
Tested-by: mobile promotions 
Reviewed-by: svcacv 
Reviewed-by: Anubhav Rai 
Reviewed-by: Frank Chen 
Reviewed-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

camera: modify debugfs_name size avoid boot failed

2021-10-29T12:40:00+00:00

fixed when the size of debugfs_name in sensor driver
more than 9 cause system can't boot.

bug 200759786

Change-Id: Ic34c7a2118095c82c58a1ef01561cedbacd27b2f
Signed-off-by: Shan Neng Chen 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2571796
Reviewed-by: svc-mobile-coverity 
Reviewed-by: Anubhav Rai 
Reviewed-by: Sudhir Vyas 
Reviewed-by: mobile promotions 
Tested-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

capture: Added NULL check for buf_ctx

2021-10-18T12:54:55+00:00

Added NULL check for capture->buf_ctx before using it in
vi_capture_shutdown.

Bug 200769105

Change-Id: Icf38a287054f0d264ed493d919dc06e1bfee8424
Signed-off-by: Devang Kubavat 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2589937
Reviewed-by: svcacv 
Reviewed-by: Jerry Chang 
Reviewed-by: Aditya Tomar 
Reviewed-by: Anubhav Rai 
Reviewed-by: Frank Chen 
Reviewed-by: mobile promotions 
Tested-by: Jerry Chang 
Tested-by: mobile promotions 
GVS: Gerrit_Virtual_Submit

tegra-vi: Fix return value check.

2021-08-13T20:10:46+00:00

This fixes coverity issue: 7112979, 7112981

Bug 200741055

Change-Id: Ie9eb8daa31eaed930889008a5fb20b5d87b86463
Signed-off-by: Achal Verma 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2575311
Reviewed-by: svc-mobile-coverity 
Reviewed-by: Phoenix Jung 
Reviewed-by: mobile promotions 
GVS: Gerrit_Virtual_Submit
Tested-by: mobile promotions

tegra-camera: Fix potential NULL ptr dereference

2021-08-12T19:24:50+00:00

This fix coverity issue : 9741833, 9741834

Bug 200741055

Change-Id: I0d6ab11a32fb7f29430adfe28c54a7b2b2a22698
Signed-off-by: Achal Verma 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2571929
Reviewed-by: svcacv 
Reviewed-by: Dmitry Pervushin 
Reviewed-by: Amrita Deshmukh 
GVS: Gerrit_Virtual_Submit
(cherry picked from commit 954e8d421ec24359a64542dd2777267b3d9bdb81)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2574546
Reviewed-by: svc-mobile-coverity 
Reviewed-by: Phoenix Jung 
Reviewed-by: mobile promotions 
Tested-by: mobile promotions

media:i2c:imx274 power on delay

2021-06-21T08:55:18+00:00

Add somelayency in imx274 power on. The AVDD on McCoy
is turned off by default and needs some time to be setup
before the reset GPIOs can be released.

Bug 200741282

Change-Id: I0696e0aab4239929552e355325ffbe6070a94484
Signed-off-by: arai 
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545868
Reviewed-by: svcacv 
Reviewed-by: Sudhir Vyas 
Reviewed-by: mobile promotions 
GVS: Gerrit_Virtual_Submit
Tested-by: mobile promotions

capture-isp: use separate ringbuffer to pass iovas to rtcpu

2021-06-21T08:55:13+00:00

Passing iova pointers using capture descriptor is not safe since
usermode application can access them.

Use additional ringbuffers shared between kernel and rtcpu to
pass iova pointers securely.

Related changes and bugfixes:

Use permanent vmap mapping for ISP capture and program descriptors
This should also improve performance and reduce fragmentation
of kernel address space.

Don't use relocs. Access descriptor like normal C structure.
Remove unused reloc API.

Fix map error reporting capture_common_pin_and_get_iova().

Allocate fixed size unpin buffer at setup time istead of
relying on kzalloc/kfree during each request. This is to simplify
critical runtime codepath and remove flexible array struct
member which is prohibited by MISRA C conventions.

Add unpin check to prevent submission of requests which are
still in progress

bug 2768992
bug 3231670
bug 200688288
bug 200690078

CAMASIL-7472
CAMASIL-7474

Signed-off-by: Kirill Artamonov 
Change-Id: Ie2c556cc61334ae478c7edc7e062e16e8b142031
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2487425
(cherry picked from commit 23186dfdd7b090e6b93c3a6daed60860225b9663)
Reviewed-on: https://git-master.nvidia.com/r/c/linux-nvidia/+/2545380
Reviewed-by: Semi Malinen 
Reviewed-by: Pekka Pessi 
Reviewed-by: Mika Liljeberg 
Reviewed-by: mobile promotions 
Tested-by: mobile promotions 
GVS: Gerrit_Virtual_Submit