<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-dev/drivers/media/platform/nxp, branch linus/master</title>
<subtitle>Linux kernel development work - see feature branches</subtitle>
<id>https://git.zx2c4.com/linux-dev/atom/drivers/media/platform/nxp?h=linus%2Fmaster</id>
<link rel='self' href='https://git.zx2c4.com/linux-dev/atom/drivers/media/platform/nxp?h=linus%2Fmaster'/>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/'/>
<updated>2022-05-13T09:02:20Z</updated>
<entry>
<title>media: platform: video-viu: Do not select it by default</title>
<updated>2022-05-13T09:02:20Z</updated>
<author>
<name>Fabio Estevam</name>
<email>festevam@denx.de</email>
</author>
<published>2022-04-22T14:31:30Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=0daab1944318351a8b276e70a8da92936642cfb4'/>
<id>urn:sha1:0daab1944318351a8b276e70a8da92936642cfb4</id>
<content type='text'>
The video viu driver is not a vital one for booting purposes.

Remove the unneeded 'default y' option.

Signed-off-by: Fabio Estevam &lt;festevam@denx.de&gt;
Reviewed-by: Laurent Pinchart &lt;laurent.pinchart@ideasonboard.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Support dynamic resolution change</title>
<updated>2022-05-13T09:02:19Z</updated>
<author>
<name>Ming Qian</name>
<email>ming.qian@nxp.com</email>
</author>
<published>2022-03-23T09:05:58Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=b4e1fb8643daabba850e97df532191acffc23e6a'/>
<id>urn:sha1:b4e1fb8643daabba850e97df532191acffc23e6a</id>
<content type='text'>
To support dynamic resolution change,
driver should meet the following conditions:
1. the previous pictures are all decoded before source change event.
2. prevent decoding new resolution pictures with incorrect capture
   buffer, until user handle source change event and setup capture.
3. report correct fmt and resolution during source change.

Signed-off-by: Ming Qian &lt;ming.qian@nxp.com&gt;
Reviewed-by: Mirela Rabulea &lt;mirela.rabulea@nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Handle source change in a function</title>
<updated>2022-05-13T09:02:19Z</updated>
<author>
<name>Ming Qian</name>
<email>ming.qian@nxp.com</email>
</author>
<published>2022-03-23T09:05:57Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=831f87424dd3973612782983ef7352789795b4df'/>
<id>urn:sha1:831f87424dd3973612782983ef7352789795b4df</id>
<content type='text'>
Refine code to support dynamic resolution change

Signed-off-by: Ming Qian &lt;ming.qian@nxp.com&gt;
Reviewed-by: Mirela Rabulea &lt;mirela.rabulea@nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Propagate the output frame size to the capture side</title>
<updated>2022-05-13T09:02:19Z</updated>
<author>
<name>Ming Qian</name>
<email>ming.qian@nxp.com</email>
</author>
<published>2022-03-23T09:05:56Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=ef2feed1ece2366c817090bd9b48ea887921aabf'/>
<id>urn:sha1:ef2feed1ece2366c817090bd9b48ea887921aabf</id>
<content type='text'>
The GStreamer v4l2videodec only ever calls S_FMT on the output side
and then expects G_FMT on the capture side to return a valid format.

Signed-off-by: Ming Qian &lt;ming.qian@nxp.com&gt;
Reviewed-by: Mirela Rabulea &lt;mirela.rabulea@nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Identify and handle precision correctly</title>
<updated>2022-05-13T09:02:18Z</updated>
<author>
<name>Ming Qian</name>
<email>ming.qian@nxp.com</email>
</author>
<published>2022-03-23T09:05:55Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=bec0a3a67389ede106d0661a007edf832878d8b2'/>
<id>urn:sha1:bec0a3a67389ede106d0661a007edf832878d8b2</id>
<content type='text'>
The decoder will save the precision that was detected from jpeg header
and use it later, when choosing the pixel format and also calculate
bytesperline according to precision.

The 12bit jpeg is not supported yet,
but driver shouldn't led to serious problem if user enqueue a 12 bit jpeg.
And the 12bit jpeg is supported by hardware, driver may support it later.

[hverkuil: document the new precision field]

Signed-off-by: Ming Qian &lt;ming.qian@nxp.com&gt;
Reviewed-by: Mirela Rabulea &lt;mirela.rabulea@nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Refactor function mxc_jpeg_parse</title>
<updated>2022-05-13T09:02:18Z</updated>
<author>
<name>Ming Qian</name>
<email>ming.qian@nxp.com</email>
</author>
<published>2022-03-23T09:05:54Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=8dd504a3a0a5f73b4c137ce3afc35936a4ecd871'/>
<id>urn:sha1:8dd504a3a0a5f73b4c137ce3afc35936a4ecd871</id>
<content type='text'>
Refine code to support dynamic resolution change

Signed-off-by: Ming Qian &lt;ming.qian@nxp.com&gt;
Reviewed-by: Mirela Rabulea &lt;mirela.rabulea@nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx-jpeg: Fix potential array out of bounds in queue_setup</title>
<updated>2022-05-13T09:02:17Z</updated>
<author>
<name>Mirela Rabulea</name>
<email>mirela.rabulea@oss.nxp.com</email>
</author>
<published>2022-03-08T13:16:57Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=e490fa1be5fe820029aa2ff2f70bcbbcebc11d28'/>
<id>urn:sha1:e490fa1be5fe820029aa2ff2f70bcbbcebc11d28</id>
<content type='text'>
Fix smatch warning in mxc_jpeg_queue_setup, check *nplanes against
current format:

drivers/media/platform/imx-jpeg/mxc-jpeg.c:1070
mxc_jpeg_queue_setup()
         warn: potential user controlled iterator 'i' (array size 2 vs 7)

Signed-off-by: Mirela Rabulea &lt;mirela.rabulea@oss.nxp.com&gt;
Signed-off-by: Hans Verkuil &lt;hverkuil-cisco@xs4all.nl&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: platform: imx-mipi-csis: Remove unneeded 'default n'</title>
<updated>2022-04-24T08:07:31Z</updated>
<author>
<name>Fabio Estevam</name>
<email>festevam@denx.de</email>
</author>
<published>2022-04-22T14:31:29Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=d0c19bed8cd3d005739c0a6374118c553564ef10'/>
<id>urn:sha1:d0c19bed8cd3d005739c0a6374118c553564ef10</id>
<content type='text'>
There is no need to pass 'default n' in Kconfig, as by default the
CONFIG_VIDEO_IMX_MIPI_CSIS option is not selected.

Signed-off-by: Fabio Estevam &lt;festevam@denx.de&gt;
Signed-off-by: Laurent Pinchart &lt;laurent.pinchart@ideasonboard.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx: imx-mipi-csis: Fix active format initialization on source pad</title>
<updated>2022-04-24T08:07:08Z</updated>
<author>
<name>Laurent Pinchart</name>
<email>laurent.pinchart@ideasonboard.com</email>
</author>
<published>2022-03-21T15:11:35Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=fe14b546d6e57542dbd4f5ccdb5a382904d26c5a'/>
<id>urn:sha1:fe14b546d6e57542dbd4f5ccdb5a382904d26c5a</id>
<content type='text'>
Commit 5c0701a0e791 ("media: imx: csis: Store pads format separately")
broke initialization of the active format on the source pad, as it
forgot to update the .init_cfg() handler. Fix it.

Fixes: 5c0701a0e791 ("media: imx: csis: Store pads format separately")
Signed-off-by: Laurent Pinchart &lt;laurent.pinchart@ideasonboard.com&gt;
Acked-by: Rui Miguel Silva &lt;rmfrfs@gmail.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
<entry>
<title>media: imx: imx-mipi-csis: Protect mipi_csis_dump_regs()</title>
<updated>2022-04-24T08:06:41Z</updated>
<author>
<name>Jacopo Mondi</name>
<email>jacopo@jmondi.org</email>
</author>
<published>2022-03-14T10:39:41Z</published>
<link rel='alternate' type='text/html' href='https://git.zx2c4.com/linux-dev/commit/?id=851b270bc41d006ac8dddf15265f8a0fa0df9877'/>
<id>urn:sha1:851b270bc41d006ac8dddf15265f8a0fa0df9877</id>
<content type='text'>
The mipi_csis_dump_regs() function accesses the interface registers
in order to printout their values for debug purposes.

As the function access the registers, it requires the interface to be
powered up. Currently this is only enforced in one of the function's
callers (mipi_csis_log_status)() but not when the function is called by
the debugfs attribute handler.

Make sure to access registers only if the interface is powered up and
remove the same check from the caller.

Signed-off-by: Jacopo Mondi &lt;jacopo@jmondi.org&gt;
Acked-by: Rui Miguel Silva &lt;rmfrfs@gmail.com&gt;
Signed-off-by: Laurent Pinchart &lt;laurent.pinchart@ideasonboard.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@kernel.org&gt;
</content>
</entry>
</feed>
