<feed xmlns='http://www.w3.org/2005/Atom'>
<title>linux-toradex.git/drivers/media/spi, branch v4.20-rc6</title>
<subtitle>Linux kernel for Apalis and Colibri modules</subtitle>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/'/>
<entry>
<title>media: cxd2880-spi: Modified how to declare structure</title>
<updated>2018-05-05T14:37:12+00:00</updated>
<author>
<name>Yasunari Takiguchi</name>
<email>Yasunari.Takiguchi@sony.com</email>
</author>
<published>2018-04-26T06:39:17+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=447f0081fc33b7f794505cf561a03e6e0f4814c0'/>
<id>447f0081fc33b7f794505cf561a03e6e0f4814c0</id>
<content type='text'>
This is the modification of structure declaration for spi_transfer.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+samsung@kernel.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is the modification of structure declaration for spi_transfer.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+samsung@kernel.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>media: cxd2880-spi: avoid out-of-bounds access warning</title>
<updated>2018-05-04T15:27:00+00:00</updated>
<author>
<name>Arnd Bergmann</name>
<email>arnd@arndb.de</email>
</author>
<published>2018-03-13T12:09:11+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=53dcd70eb710607b2d4085ca91a433cd9feb7b41'/>
<id>53dcd70eb710607b2d4085ca91a433cd9feb7b41</id>
<content type='text'>
The -Warray-bounds warning in gcc-8 triggers for a newly added file:

drivers/media/spi/cxd2880-spi.c: In function 'cxd2880_write_reg':
drivers/media/spi/cxd2880-spi.c:111:3: error: 'memcpy' forming offset [133, 258] is out of the bounds [0, 132] of object 'send_data' with type 'u8[132]' {aka 'unsigned char[132]'} [-Werror=array-bounds]

The problem appears to be that we have two range checks in this function,
first comparing against BURST_WRITE_MAX (128) and then comparing against
a literal '255'. The logic checking the buffer size looks at the second
one and decides that this might be the actual maximum data length.

This is understandable behavior from the compiler, but the code is actually
safe. Since the first check is already shorter, we can remove the loop
and only leave that. To be on the safe side in case BURST_WRITE_MAX might
be increased, I'm leaving the check against U8_MAX.

Fixes: bd24fcddf6b8 ("media: cxd2880-spi: Add support for CXD2880 SPI interface")

Cc: Martin Sebor &lt;msebor@gmail.com&gt;
Signed-off-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
Reviewed-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+samsung@kernel.org&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The -Warray-bounds warning in gcc-8 triggers for a newly added file:

drivers/media/spi/cxd2880-spi.c: In function 'cxd2880_write_reg':
drivers/media/spi/cxd2880-spi.c:111:3: error: 'memcpy' forming offset [133, 258] is out of the bounds [0, 132] of object 'send_data' with type 'u8[132]' {aka 'unsigned char[132]'} [-Werror=array-bounds]

The problem appears to be that we have two range checks in this function,
first comparing against BURST_WRITE_MAX (128) and then comparing against
a literal '255'. The logic checking the buffer size looks at the second
one and decides that this might be the actual maximum data length.

This is understandable behavior from the compiler, but the code is actually
safe. Since the first check is already shorter, we can remove the loop
and only leave that. To be on the safe side in case BURST_WRITE_MAX might
be increased, I'm leaving the check against U8_MAX.

Fixes: bd24fcddf6b8 ("media: cxd2880-spi: Add support for CXD2880 SPI interface")

Cc: Martin Sebor &lt;msebor@gmail.com&gt;
Signed-off-by: Arnd Bergmann &lt;arnd@arndb.de&gt;
Reviewed-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab+samsung@kernel.org&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>media: cxd2880: Add all Makefile, Kconfig files and Update MAINTAINERS file for the driver</title>
<updated>2018-03-07T10:02:57+00:00</updated>
<author>
<name>Yasunari Takiguchi</name>
<email>Yasunari.Takiguchi@sony.com</email>
</author>
<published>2018-01-18T08:59:42+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=9dbaad428d692815f01cda36dc6ae8ae0a4e8bf4'/>
<id>9dbaad428d692815f01cda36dc6ae8ae0a4e8bf4</id>
<content type='text'>
This is the Makefile, Kconfig files of driver
and MAINTAINERS file update about the driver
for the Sony CXD2880 DVB-T2/T tuner + demodulator.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is the Makefile, Kconfig files of driver
and MAINTAINERS file update about the driver
for the Sony CXD2880 DVB-T2/T tuner + demodulator.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>media: cxd2880: Fix location of DVB headers</title>
<updated>2018-03-07T10:02:37+00:00</updated>
<author>
<name>Mauro Carvalho Chehab</name>
<email>mchehab@s-opensource.com</email>
</author>
<published>2018-03-07T10:02:37+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=7cbc3013f60dbc22955645443855c8f092d3c534'/>
<id>7cbc3013f60dbc22955645443855c8f092d3c534</id>
<content type='text'>
Fix a trivial conflict, where the location of DVB headers
got moved.

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Fix a trivial conflict, where the location of DVB headers
got moved.

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>media: cxd2880-spi: Add support for CXD2880 SPI interface</title>
<updated>2018-03-07T09:39:19+00:00</updated>
<author>
<name>Yasunari Takiguchi</name>
<email>Yasunari.Takiguchi@sony.com</email>
</author>
<published>2018-01-18T08:46:10+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=bd24fcddf6b822a6646b00cfc76a21f3f7bbc7e7'/>
<id>bd24fcddf6b822a6646b00cfc76a21f3f7bbc7e7</id>
<content type='text'>
This is the SPI adapter part of the driver for the
Sony CXD2880 DVB-T2/T tuner + demodulator.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
This is the SPI adapter part of the driver for the
Sony CXD2880 DVB-T2/T tuner + demodulator.

Signed-off-by: Yasunari Takiguchi &lt;Yasunari.Takiguchi@sony.com&gt;
Signed-off-by: Masayuki Yamamoto &lt;Masayuki.Yamamoto@sony.com&gt;
Signed-off-by: Hideki Nozawa &lt;Hideki.Nozawa@sony.com&gt;
Signed-off-by: Kota Yonezawa &lt;Kota.Yonezawa@sony.com&gt;
Signed-off-by: Toshihiko Matsumoto &lt;Toshihiko.Matsumoto@sony.com&gt;
Signed-off-by: Satoshi Watanabe &lt;Satoshi.C.Watanabe@sony.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>[media] gs1662: drop kfree for memory allocated with devm_kzalloc</title>
<updated>2016-10-24T19:59:08+00:00</updated>
<author>
<name>Wei Yongjun</name>
<email>weiyongjun1@huawei.com</email>
</author>
<published>2016-09-21T13:09:39+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=df94121f02ecce435d6b5277071eb94b764caa89'/>
<id>df94121f02ecce435d6b5277071eb94b764caa89</id>
<content type='text'>
It's not necessary to free memory allocated with devm_kzalloc
and using kfree leads to a double free.

Fixes: 7aae6e2df127 ("[media] Add GS1662 driver, a video serializer")

Signed-off-by: Wei Yongjun &lt;weiyongjun1@huawei.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
It's not necessary to free memory allocated with devm_kzalloc
and using kfree leads to a double free.

Fixes: 7aae6e2df127 ("[media] Add GS1662 driver, a video serializer")

Signed-off-by: Wei Yongjun &lt;weiyongjun1@huawei.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>[media] gs1662: remove .owner field for driver</title>
<updated>2016-10-24T19:58:17+00:00</updated>
<author>
<name>Wei Yongjun</name>
<email>weiyongjun1@huawei.com</email>
</author>
<published>2016-09-21T13:09:23+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=b7491e7e902a488503982c9be098cac578a94bdc'/>
<id>b7491e7e902a488503982c9be098cac578a94bdc</id>
<content type='text'>
Remove .owner field if calls are used which set it automatically.

Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

Signed-off-by: Wei Yongjun &lt;weiyongjun1@huawei.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
Remove .owner field if calls are used which set it automatically.

Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci

Signed-off-by: Wei Yongjun &lt;weiyongjun1@huawei.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>[media] gs1662: make checkpatch happy</title>
<updated>2016-09-19T17:39:49+00:00</updated>
<author>
<name>Mauro Carvalho Chehab</name>
<email>mchehab@s-opensource.com</email>
</author>
<published>2016-09-19T17:39:49+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=3c3ba5452e8a0e2dd9fb89222feb88f11ec6ff7a'/>
<id>3c3ba5452e8a0e2dd9fb89222feb88f11ec6ff7a</id>
<content type='text'>
WARNING: line over 80 characters
+			     GS_HEIGHT_MAX, GS_PIXELCLOCK_MIN, GS_PIXELCLOCK_MAX,

WARNING: line over 80 characters
+		if (v4l2_match_dv_timings(timings, &amp;reg_fmt[i].format, 0, false))

WARNING: Block comments use a trailing */ on a separate line
+	 * which looks like a video signal activity.*/

WARNING: else is not generally useful after a break or return
+		return gs_write_register(gs-&gt;pdev, REG_FORCE_FMT, reg_value);
+	} else {

WARNING: Block comments use a trailing */ on a separate line
+	 * which looks like a video signal activity.*/

ERROR: spaces required around that '=' (ctx:VxW)
+	.enum_dv_timings= gs_enum_dv_timings,
 	                ^

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
WARNING: line over 80 characters
+			     GS_HEIGHT_MAX, GS_PIXELCLOCK_MIN, GS_PIXELCLOCK_MAX,

WARNING: line over 80 characters
+		if (v4l2_match_dv_timings(timings, &amp;reg_fmt[i].format, 0, false))

WARNING: Block comments use a trailing */ on a separate line
+	 * which looks like a video signal activity.*/

WARNING: else is not generally useful after a break or return
+		return gs_write_register(gs-&gt;pdev, REG_FORCE_FMT, reg_value);
+	} else {

WARNING: Block comments use a trailing */ on a separate line
+	 * which looks like a video signal activity.*/

ERROR: spaces required around that '=' (ctx:VxW)
+	.enum_dv_timings= gs_enum_dv_timings,
 	                ^

Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>[media] media Kconfig: improve the spi integration</title>
<updated>2016-09-19T17:37:05+00:00</updated>
<author>
<name>Hans Verkuil</name>
<email>hverkuil@xs4all.nl</email>
</author>
<published>2016-09-15T15:18:14+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=8283a0929922b331dbd73db26520443136941c48'/>
<id>8283a0929922b331dbd73db26520443136941c48</id>
<content type='text'>
The SPI driver looked a bit lonely in the config menu, and it didn't
support the autoselect. Shift things around a bit so it looks more logical.

Signed-off-by: Hans Verkuil &lt;hans.verkuil@cisco.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
The SPI driver looked a bit lonely in the config menu, and it didn't
support the autoselect. Shift things around a bit so it looks more logical.

Signed-off-by: Hans Verkuil &lt;hans.verkuil@cisco.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
<entry>
<title>[media] Add GS1662 driver, a video serializer</title>
<updated>2016-09-19T17:36:24+00:00</updated>
<author>
<name>Charles-Antoine Couret</name>
<email>charles-antoine.couret@nexvision.fr</email>
</author>
<published>2016-09-15T15:29:51+00:00</published>
<link rel='alternate' type='text/html' href='https://git.toradex.cn/cgit/linux-toradex.git/commit/?id=7aae6e2df127f9f7a96c21e21a277dd7ff063e6a'/>
<id>7aae6e2df127f9f7a96c21e21a277dd7ff063e6a</id>
<content type='text'>
You can read datasheet here:
http://www.c-dis.net/media/871/GS1662_Datasheet.pdf

It's a component which supports HD and SD CEA or SDI formats
to SDI output. It's configured through SPI bus.

GS1662 driver is implemented as v4l2 subdev.

Signed-off-by: Charles-Antoine Couret &lt;charles-antoine.couret@nexvision.fr&gt;
Signed-off-by: Hans Verkuil &lt;hans.verkuil@cisco.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</content>
<content type='xhtml'>
<div xmlns='http://www.w3.org/1999/xhtml'>
<pre>
You can read datasheet here:
http://www.c-dis.net/media/871/GS1662_Datasheet.pdf

It's a component which supports HD and SD CEA or SDI formats
to SDI output. It's configured through SPI bus.

GS1662 driver is implemented as v4l2 subdev.

Signed-off-by: Charles-Antoine Couret &lt;charles-antoine.couret@nexvision.fr&gt;
Signed-off-by: Hans Verkuil &lt;hans.verkuil@cisco.com&gt;
Signed-off-by: Mauro Carvalho Chehab &lt;mchehab@s-opensource.com&gt;
</pre>
</div>
</content>
</entry>
</feed>
