media: rkisp1: rksip1-capture.c: Improve comments and fix typos
Improve the wording of the function description to increase readability. Fix three typos: s/during processing a frame/while processing a frame/ s/it also update/it also updates/ s/there's not buf in shadow/there's no buffer in a shadow register/ Replace the abbreviation 'buf' with the full word buffer, the abbreviation 'config' with the verb configure, and 'regs' with registers. The goal of this change is to ease the reading flow of the comment. Signed-off-by: Sebastian Fricke <sebastian.fricke@posteo.net> Reviewed-by: Dafna Hirschfeld <dafna.hirschfeld@collabora.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
This commit is contained in:
Родитель
b07006ff93
Коммит
71c4151812
|
@ -830,8 +830,8 @@ static void rkisp1_return_all_buffers(struct rkisp1_capture *cap,
|
||||||
}
|
}
|
||||||
|
|
||||||
/*
|
/*
|
||||||
* Most of registers inside rockchip ISP1 have shadow register since
|
* Most registers inside the rockchip ISP1 have shadow register since
|
||||||
* they must be not be changed during processing a frame.
|
* they must not be changed while processing a frame.
|
||||||
* Usually, each sub-module updates its shadow register after
|
* Usually, each sub-module updates its shadow register after
|
||||||
* processing the last pixel of a frame.
|
* processing the last pixel of a frame.
|
||||||
*/
|
*/
|
||||||
|
@ -847,14 +847,14 @@ static void rkisp1_cap_stream_enable(struct rkisp1_capture *cap)
|
||||||
spin_lock_irq(&cap->buf.lock);
|
spin_lock_irq(&cap->buf.lock);
|
||||||
rkisp1_set_next_buf(cap);
|
rkisp1_set_next_buf(cap);
|
||||||
cap->ops->enable(cap);
|
cap->ops->enable(cap);
|
||||||
/* It's safe to config ACTIVE and SHADOW regs for the
|
/* It's safe to configure ACTIVE and SHADOW registers for the
|
||||||
* first stream. While when the second is starting, do NOT
|
* first stream. While when the second is starting, do NOT
|
||||||
* force update because it also update the first one.
|
* force update because it also updates the first one.
|
||||||
*
|
*
|
||||||
* The latter case would drop one more buf(that is 2) since
|
* The latter case would drop one more buffer(that is 2) since
|
||||||
* there's not buf in shadow when the second FE received. This's
|
* there's no buffer in a shadow register when the second FE received.
|
||||||
* also required because the second FE maybe corrupt especially
|
* This's also required because the second FE maybe corrupt
|
||||||
* when run at 120fps.
|
* especially when run at 120fps.
|
||||||
*/
|
*/
|
||||||
if (!other->is_streaming) {
|
if (!other->is_streaming) {
|
||||||
/* force cfg update */
|
/* force cfg update */
|
||||||
|
|
Загрузка…
Ссылка в новой задаче