mbox series

[GIT,PULL,for,5.11] Smiapp driver fixes, begin conversion to CCS

Message ID 20201123231950.GD4351@valkosipuli.retiisi.org.uk (mailing list archive)
State New, archived
Headers show
Series [GIT,PULL,for,5.11] Smiapp driver fixes, begin conversion to CCS | expand

Pull-request

git://linuxtv.org/sailus/media_tree.git tags/for-5.11-3-signed

Message

Sakari Ailus Nov. 23, 2020, 11:19 p.m. UTC 
Hi Mauro,

Here's a set that prepares making the smiapp driver a modern MIPI CCS
driver. It includes bugfixes, switch to CCS limit definitions as well as
includes the DT bindings for the CCS device.

Please pull.


The following changes since commit 9463e07df8e0f93931e32c6f415d3f82bda63f35:

  media: v4l2-compat-ioctl32.c: add missing #ifdef CONFIG_COMPAT_32BIT_TIMEs (2020-11-17 07:14:46 +0100)

are available in the Git repository at:

  git://linuxtv.org/sailus/media_tree.git tags/for-5.11-3-signed

for you to fetch changes up to cd4f707bd17c2569c799a219d417c58261ea75d7:

  ccs: Request for "reset" GPIO (2020-11-18 13:13:24 +0200)

----------------------------------------------------------------
V4L2 patches for 5.11

----------------------------------------------------------------
Sakari Ailus (29):
      ccs: Add the generator for CCS register definitions and limits
      Documentation: ccs: Add CCS driver documentation
      smiapp: Import CCS definitions
      smiapp: Use CCS register flags
      smiapp: Calculate CCS limit offsets and limit buffer size
      smiapp: Remove macros for defining registers, merge definitions
      smiapp: Add macros for accessing CCS registers
      smiapp: Use MIPI CCS version and manufacturer ID information
      smiapp: Read CCS limit values
      smiapp: Switch to CCS limits
      smiapp: Obtain frame descriptor from CCS limits
      smiapp: Use CCS limits in reading data format descriptors
      smiapp: Use CCS limits in reading binning capabilities
      smiapp: Use CCS registers
      smiapp: Remove quirk function for writing a single 8-bit register
      smiapp: Rename register access functions
      smiapp: Internal rename to CCS
      smiapp: Differentiate CCS sensors from SMIA in subdev naming
      smiapp: Rename as "ccs"
      ccs: Remove profile concept
      ccs: Give all subdevs a function
      dt-bindings: nokia,smia: Fix link-frequencies documentation
      dt-bindings: nokia,smia: Make vana-supply optional
      dt-bindings: nokia,smia: Remove nokia,nvm-size property
      dt-bindings: nokia,smia: Convert to YAML
      dt-bindings: nokia,smia: Use better active polarity for reset
      dt-bindings: nokia,smia: Amend SMIA bindings with MIPI CCS support
      dt-bindings: mipi-ccs: Add bus-type for C-PHY support
      ccs: Request for "reset" GPIO

 .../devicetree/bindings/media/i2c/mipi-ccs.yaml    |  128 ++
 .../devicetree/bindings/media/i2c/nokia,smia.txt   |   66 -
 .../driver-api/media/drivers/ccs/ccs-regs.txt      | 1041 +++++++++++
 Documentation/driver-api/media/drivers/ccs/ccs.rst |   82 +
 .../driver-api/media/drivers/ccs/mk-ccs-regs       |  410 +++++
 Documentation/driver-api/media/drivers/index.rst   |    1 +
 MAINTAINERS                                        |   20 +-
 drivers/media/i2c/Kconfig                          |    2 +-
 drivers/media/i2c/Makefile                         |    2 +-
 drivers/media/i2c/{smiapp => ccs}/Kconfig          |    7 +-
 drivers/media/i2c/ccs/Makefile                     |    6 +
 .../i2c/{smiapp/smiapp-core.c => ccs/ccs-core.c}   | 1828 +++++++++++---------
 drivers/media/i2c/ccs/ccs-limits.c                 |  239 +++
 drivers/media/i2c/ccs/ccs-limits.h                 |  258 +++
 .../i2c/{smiapp/smiapp-quirk.c => ccs/ccs-quirk.c} |   92 +-
 .../i2c/{smiapp/smiapp-quirk.h => ccs/ccs-quirk.h} |   52 +-
 .../{smiapp/smiapp-regs.c => ccs/ccs-reg-access.c} |   85 +-
 drivers/media/i2c/ccs/ccs-reg-access.h             |   38 +
 drivers/media/i2c/ccs/ccs-regs.h                   |  953 ++++++++++
 drivers/media/i2c/{smiapp/smiapp.h => ccs/ccs.h}   |  164 +-
 drivers/media/i2c/ccs/smiapp-reg-defs.h            |  580 +++++++
 drivers/media/i2c/smiapp/Makefile                  |    6 -
 drivers/media/i2c/smiapp/smiapp-limits.c           |  118 --
 drivers/media/i2c/smiapp/smiapp-limits.h           |  114 --
 drivers/media/i2c/smiapp/smiapp-reg-defs.h         |  489 ------
 drivers/media/i2c/smiapp/smiapp-reg.h              |  116 --
 drivers/media/i2c/smiapp/smiapp-regs.h             |   36 -
 27 files changed, 4921 insertions(+), 2012 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/media/i2c/mipi-ccs.yaml
 delete mode 100644 Documentation/devicetree/bindings/media/i2c/nokia,smia.txt
 create mode 100644 Documentation/driver-api/media/drivers/ccs/ccs-regs.txt
 create mode 100644 Documentation/driver-api/media/drivers/ccs/ccs.rst
 create mode 100755 Documentation/driver-api/media/drivers/ccs/mk-ccs-regs
 rename drivers/media/i2c/{smiapp => ccs}/Kconfig (55%)
 create mode 100644 drivers/media/i2c/ccs/Makefile
 rename drivers/media/i2c/{smiapp/smiapp-core.c => ccs/ccs-core.c} (53%)
 create mode 100644 drivers/media/i2c/ccs/ccs-limits.c
 create mode 100644 drivers/media/i2c/ccs/ccs-limits.h
 rename drivers/media/i2c/{smiapp/smiapp-quirk.c => ccs/ccs-quirk.c} (66%)
 rename drivers/media/i2c/{smiapp/smiapp-quirk.h => ccs/ccs-quirk.h} (55%)
 rename drivers/media/i2c/{smiapp/smiapp-regs.c => ccs/ccs-reg-access.c} (66%)
 create mode 100644 drivers/media/i2c/ccs/ccs-reg-access.h
 create mode 100644 drivers/media/i2c/ccs/ccs-regs.h
 rename drivers/media/i2c/{smiapp/smiapp.h => ccs/ccs.h} (57%)
 create mode 100644 drivers/media/i2c/ccs/smiapp-reg-defs.h
 delete mode 100644 drivers/media/i2c/smiapp/Makefile
 delete mode 100644 drivers/media/i2c/smiapp/smiapp-limits.c
 delete mode 100644 drivers/media/i2c/smiapp/smiapp-limits.h
 delete mode 100644 drivers/media/i2c/smiapp/smiapp-reg-defs.h
 delete mode 100644 drivers/media/i2c/smiapp/smiapp-reg.h
 delete mode 100644 drivers/media/i2c/smiapp/smiapp-regs.h

Comments

Jenkins Nov. 23, 2020, 11:43 p.m. UTC | #1 
From: builder@linuxtv.org

Pull request: https://patchwork.linuxtv.org/project/linux-media/patch/20201123231950.GD4351@valkosipuli.retiisi.org.uk/
Build log: https://builder.linuxtv.org/job/patchwork/78341/
Build time: 00:11:57
Link: https://lore.kernel.org/linux-media/20201123231950.GD4351@valkosipuli.retiisi.org.uk

gpg: Signature made Mon 23 Nov 2020 11:09:35 PM UTC
gpg:                using DSA key 53AC58A5F5948636C04A1BF8141DFA54A1EC8DEA
gpg:                issuer "sakari.ailus@linux.intel.com"
gpg: Can't check signature: No public key

Summary: got 11/29 patches with issues, being 1 at build time, plus one error when buinding PDF document

Error/warnings:

patches/0001-ccs-Add-the-generator-for-CCS-register-definitions-a.patch:

   checkpatch.pl:
	$ cat patches/0001-ccs-Add-the-generator-for-CCS-register-definitions-a.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:26: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:1073: ERROR: do not set execute permissions for source files
	-:1079: WARNING: Missing or malformed SPDX-License-Identifier tag in line 2

patches/0002-Documentation-ccs-Add-CCS-driver-documentation.patch:

   checkpatch.pl:
	$ cat patches/0002-Documentation-ccs-Add-CCS-driver-documentation.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:16: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0003-smiapp-Import-CCS-definitions.patch:

   checkpatch.pl:
	$ cat patches/0003-smiapp-Import-CCS-definitions.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:21: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:197: WARNING: line length of 109 exceeds 100 columns
	-:198: WARNING: line length of 125 exceeds 100 columns
	-:200: WARNING: line length of 109 exceeds 100 columns
	-:201: WARNING: line length of 125 exceeds 100 columns
	-:235: WARNING: line length of 101 exceeds 100 columns
	-:246: WARNING: line length of 103 exceeds 100 columns
	-:280: CHECK: Prefer kernel type 'u32' over 'uint32_t'
	-:281: CHECK: Prefer kernel type 'u16' over 'uint16_t'
	-:282: CHECK: Prefer kernel type 'u16' over 'uint16_t'
	-:286: CHECK: Prefer using the BIT macro
	-:542: CHECK: Prefer using the BIT macro
	-:665: CHECK: Prefer using the BIT macro
	-:666: CHECK: Prefer using the BIT macro
	-:679: CHECK: Prefer using the BIT macro
	-:680: CHECK: Prefer using the BIT macro
	-:696: CHECK: Prefer using the BIT macro
	-:697: CHECK: Prefer using the BIT macro
	-:698: CHECK: Prefer using the BIT macro
	-:700: WARNING: line length of 104 exceeds 100 columns
	-:702: CHECK: Prefer using the BIT macro
	-:714: CHECK: Prefer using the BIT macro
	-:715: CHECK: Prefer using the BIT macro
	-:716: CHECK: Prefer using the BIT macro
	-:717: CHECK: Prefer using the BIT macro
	-:718: CHECK: Prefer using the BIT macro
	-:719: CHECK: Prefer using the BIT macro
	-:720: CHECK: Prefer using the BIT macro
	-:755: CHECK: Prefer using the BIT macro
	-:757: CHECK: Prefer using the BIT macro
	-:758: CHECK: Prefer using the BIT macro
	-:760: CHECK: Prefer using the BIT macro
	-:815: CHECK: Prefer using the BIT macro
	-:817: CHECK: Prefer using the BIT macro
	-:819: CHECK: Prefer using the BIT macro
	-:822: CHECK: Prefer using the BIT macro
	-:824: CHECK: Prefer using the BIT macro
	-:830: CHECK: Prefer using the BIT macro
	-:833: CHECK: Prefer using the BIT macro
	-:835: CHECK: Prefer using the BIT macro
	-:848: CHECK: Prefer using the BIT macro
	-:864: CHECK: Prefer using the BIT macro
	-:865: CHECK: Prefer using the BIT macro
	-:866: CHECK: Prefer using the BIT macro
	-:874: CHECK: Prefer using the BIT macro
	-:879: WARNING: line length of 116 exceeds 100 columns
	-:892: CHECK: Prefer using the BIT macro
	-:894: CHECK: Prefer using the BIT macro
	-:898: CHECK: Prefer using the BIT macro
	-:899: CHECK: Prefer using the BIT macro
	-:900: CHECK: Prefer using the BIT macro
	-:905: CHECK: Prefer using the BIT macro
	-:906: CHECK: Prefer using the BIT macro
	-:907: CHECK: Prefer using the BIT macro
	-:909: CHECK: Prefer using the BIT macro
	-:910: CHECK: Prefer using the BIT macro
	-:911: CHECK: Prefer using the BIT macro
	-:912: CHECK: Prefer using the BIT macro
	-:918: CHECK: Prefer using the BIT macro
	-:921: CHECK: Prefer using the BIT macro
	-:923: CHECK: Prefer using the BIT macro
	-:925: CHECK: Prefer using the BIT macro
	-:927: CHECK: Prefer using the BIT macro
	-:929: CHECK: Prefer using the BIT macro
	-:931: CHECK: Prefer using the BIT macro
	-:933: CHECK: Prefer using the BIT macro
	-:935: CHECK: Prefer using the BIT macro
	-:936: CHECK: Prefer using the BIT macro
	-:937: CHECK: Prefer using the BIT macro
	-:939: CHECK: Prefer using the BIT macro
	-:940: CHECK: Prefer using the BIT macro
	-:950: CHECK: Prefer using the BIT macro
	-:956: CHECK: Prefer using the BIT macro
	-:957: CHECK: Prefer using the BIT macro
	-:958: CHECK: Prefer using the BIT macro
	-:961: CHECK: Prefer using the BIT macro
	-:963: CHECK: Prefer using the BIT macro
	-:964: CHECK: Prefer using the BIT macro
	-:965: CHECK: Prefer using the BIT macro
	-:966: CHECK: Prefer using the BIT macro
	-:972: CHECK: Prefer using the BIT macro
	-:976: CHECK: Prefer using the BIT macro
	-:977: CHECK: Prefer using the BIT macro
	-:978: CHECK: Prefer using the BIT macro
	-:979: CHECK: Prefer using the BIT macro
	-:988: CHECK: Prefer using the BIT macro
	-:989: CHECK: Prefer using the BIT macro
	-:995: CHECK: Prefer using the BIT macro
	-:1008: CHECK: Prefer using the BIT macro
	-:1010: WARNING: line length of 110 exceeds 100 columns
	-:1011: WARNING: line length of 110 exceeds 100 columns
	-:1014: WARNING: line length of 110 exceeds 100 columns
	-:1015: WARNING: line length of 110 exceeds 100 columns
	-:1018: WARNING: line length of 110 exceeds 100 columns
	-:1019: WARNING: line length of 110 exceeds 100 columns
	-:1022: WARNING: line length of 110 exceeds 100 columns
	-:1023: WARNING: line length of 110 exceeds 100 columns
	-:1024: WARNING: line length of 110 exceeds 100 columns
	-:1025: WARNING: line length of 110 exceeds 100 columns
	-:1029: CHECK: Prefer using the BIT macro
	-:1030: CHECK: Prefer using the BIT macro
	-:1031: CHECK: Prefer using the BIT macro
	-:1032: CHECK: Prefer using the BIT macro
	-:1044: CHECK: Prefer using the BIT macro
	-:1045: CHECK: Prefer using the BIT macro
	-:1046: CHECK: Prefer using the BIT macro
	-:1047: CHECK: Prefer using the BIT macro
	-:1055: WARNING: line length of 110 exceeds 100 columns
	-:1056: WARNING: line length of 110 exceeds 100 columns
	-:1059: WARNING: line length of 110 exceeds 100 columns
	-:1060: WARNING: line length of 110 exceeds 100 columns
	-:1078: CHECK: Prefer using the BIT macro
	-:1079: CHECK: Prefer using the BIT macro
	-:1080: CHECK: Prefer using the BIT macro
	-:1081: CHECK: Prefer using the BIT macro
	-:1084: WARNING: line length of 110 exceeds 100 columns
	-:1085: WARNING: line length of 110 exceeds 100 columns
	-:1088: WARNING: line length of 110 exceeds 100 columns
	-:1089: WARNING: line length of 110 exceeds 100 columns
	-:1091: CHECK: Prefer using the BIT macro
	-:1092: CHECK: Prefer using the BIT macro
	-:1093: CHECK: Prefer using the BIT macro
	-:1094: CHECK: Prefer using the BIT macro
	-:1096: CHECK: Prefer using the BIT macro
	-:1102: CHECK: Prefer using the BIT macro
	-:1104: CHECK: Prefer using the BIT macro
	-:1136: CHECK: Prefer using the BIT macro
	-:1137: CHECK: Prefer using the BIT macro
	-:1138: CHECK: Prefer using the BIT macro
	-:1139: CHECK: Prefer using the BIT macro
	-:1140: CHECK: Prefer using the BIT macro
	-:1141: CHECK: Prefer using the BIT macro
	-:1142: CHECK: Prefer using the BIT macro
	-:1153: CHECK: Prefer using the BIT macro
	-:1154: CHECK: Prefer using the BIT macro
	-:1155: CHECK: Prefer using the BIT macro
	-:1156: CHECK: Prefer using the BIT macro
	-:1159: CHECK: Prefer using the BIT macro
	-:1160: CHECK: Prefer using the BIT macro
	-:1161: CHECK: Prefer using the BIT macro
	-:1163: CHECK: Prefer using the BIT macro
	-:1164: CHECK: Prefer using the BIT macro
	-:1165: CHECK: Prefer using the BIT macro
	-:1166: CHECK: Prefer using the BIT macro
	-:1167: CHECK: Prefer using the BIT macro
	-:1168: CHECK: Prefer using the BIT macro
	-:1173: WARNING: line length of 110 exceeds 100 columns
	-:1174: WARNING: line length of 110 exceeds 100 columns
	-:1175: WARNING: line length of 110 exceeds 100 columns
	-:1176: WARNING: line length of 110 exceeds 100 columns
	-:1177: WARNING: line length of 104 exceeds 100 columns
	-:1178: WARNING: line length of 104 exceeds 100 columns
	-:1180: CHECK: Prefer using the BIT macro
	-:1182: CHECK: Prefer using the BIT macro
	-:1183: CHECK: Prefer using the BIT macro
	-:1184: CHECK: Prefer using the BIT macro
	-:1185: CHECK: Prefer using the BIT macro
	-:1186: CHECK: Prefer using the BIT macro
	-:1194: CHECK: Prefer using the BIT macro
	-:1196: CHECK: Prefer using the BIT macro
	-:1203: CHECK: Prefer using the BIT macro
	-:1204: CHECK: Prefer using the BIT macro
	-:1205: CHECK: Prefer using the BIT macro
	-:1206: CHECK: Prefer using the BIT macro
	-:1207: CHECK: Prefer using the BIT macro
	-:1208: CHECK: Prefer using the BIT macro
	-:1209: CHECK: Prefer using the BIT macro
	-:1210: CHECK: Prefer using the BIT macro
	-:1212: CHECK: Prefer using the BIT macro
	-:1213: CHECK: Prefer using the BIT macro
	-:1214: CHECK: Prefer using the BIT macro
	-:1215: CHECK: Prefer using the BIT macro
	-:1216: CHECK: Prefer using the BIT macro
	-:1217: CHECK: Prefer using the BIT macro
	-:1218: CHECK: Prefer using the BIT macro
	-:1219: CHECK: Prefer using the BIT macro
	-:1221: CHECK: Prefer using the BIT macro
	-:1222: CHECK: Prefer using the BIT macro
	-:1227: CHECK: Prefer using the BIT macro
	-:1228: CHECK: Prefer using the BIT macro
	-:1229: CHECK: Prefer using the BIT macro
	-:1231: CHECK: Prefer using the BIT macro
	-:1232: CHECK: Prefer using the BIT macro
	-:1233: CHECK: Prefer using the BIT macro
	-:1234: CHECK: Prefer using the BIT macro
	-:1236: CHECK: Prefer using the BIT macro
	-:1237: CHECK: Prefer using the BIT macro
	-:1238: CHECK: Prefer using the BIT macro
	-:1239: CHECK: Prefer using the BIT macro
	-:1240: CHECK: Prefer using the BIT macro
	-:1241: CHECK: Prefer using the BIT macro
	-:1242: CHECK: Prefer using the BIT macro
	-:1243: CHECK: Prefer using the BIT macro
	-:1245: CHECK: Prefer using the BIT macro
	-:1246: CHECK: Prefer using the BIT macro
	-:1247: CHECK: Prefer using the BIT macro
	-:1248: CHECK: Prefer using the BIT macro
	-:1249: CHECK: Prefer using the BIT macro
	-:1250: CHECK: Prefer using the BIT macro
	-:1251: WARNING: line length of 151 exceeds 100 columns
	-:1251: CHECK: Macro argument reuse 'n' - possible side-effects?
	-:1255: CHECK: Prefer using the BIT macro
	-:1256: CHECK: Prefer using the BIT macro
	-:1257: CHECK: Prefer using the BIT macro
	-:1258: WARNING: line length of 151 exceeds 100 columns
	-:1258: CHECK: Macro argument reuse 'n' - possible side-effects?
	-:1262: CHECK: Prefer using the BIT macro
	-:1263: CHECK: Prefer using the BIT macro
	-:1264: CHECK: Prefer using the BIT macro
	-:1266: CHECK: Prefer using the BIT macro
	-:1268: CHECK: Prefer using the BIT macro
	-:1270: CHECK: Prefer using the BIT macro
	-:1272: CHECK: Prefer using the BIT macro
	-:1273: CHECK: Prefer using the BIT macro
	-:1274: CHECK: Prefer using the BIT macro
	-:1275: CHECK: Prefer using the BIT macro
	-:1276: CHECK: Prefer using the BIT macro
	-:1278: CHECK: Prefer using the BIT macro
	-:1279: CHECK: Prefer using the BIT macro
	-:1280: CHECK: Prefer using the BIT macro
	-:1282: CHECK: Prefer using the BIT macro
	-:1283: CHECK: Prefer using the BIT macro
	-:1284: CHECK: Prefer using the BIT macro
	-:1285: CHECK: Prefer using the BIT macro
	-:1286: CHECK: Prefer using the BIT macro
	-:1287: CHECK: Prefer using the BIT macro
	-:1288: CHECK: Prefer using the BIT macro
	-:1289: CHECK: Prefer using the BIT macro
	-:1291: CHECK: Prefer using the BIT macro
	-:1292: CHECK: Prefer using the BIT macro
	-:1293: CHECK: Prefer using the BIT macro
	-:1294: CHECK: Prefer using the BIT macro
	-:1295: CHECK: Prefer using the BIT macro
	-:1297: CHECK: Prefer using the BIT macro
	-:1298: CHECK: Prefer using the BIT macro
	-:1299: CHECK: Prefer using the BIT macro
	-:1300: CHECK: Prefer using the BIT macro
	-:1301: CHECK: Prefer using the BIT macro
	-:1302: CHECK: Prefer using the BIT macro
	-:1303: CHECK: Prefer using the BIT macro
	-:1304: CHECK: Prefer using the BIT macro
	-:1317: CHECK: Prefer using the BIT macro
	-:1327: CHECK: Prefer using the BIT macro
	-:1331: CHECK: Prefer using the BIT macro
	-:1333: CHECK: Prefer using the BIT macro
	-:1334: CHECK: Prefer using the BIT macro
	-:1335: CHECK: Prefer using the BIT macro
	-:1336: CHECK: Prefer using the BIT macro
	-:1337: CHECK: Prefer using the BIT macro
	-:1338: CHECK: Prefer using the BIT macro
	-:1339: CHECK: Prefer using the BIT macro
	-:1399: CHECK: Prefer using the BIT macro
	-:1400: CHECK: Prefer using the BIT macro
	-:1401: CHECK: Prefer using the BIT macro
	-:1402: CHECK: Prefer using the BIT macro
	-:1412: CHECK: Prefer using the BIT macro
	-:1413: CHECK: Prefer using the BIT macro
	-:1414: CHECK: Prefer using the BIT macro
	-:1415: CHECK: Prefer using the BIT macro
	-:1421: CHECK: Prefer using the BIT macro
	-:1422: CHECK: Prefer using the BIT macro
	-:1424: CHECK: Prefer using the BIT macro
	-:1425: CHECK: Prefer using the BIT macro
	-:1427: CHECK: Prefer using the BIT macro
	-:1430: CHECK: Prefer using the BIT macro
	-:1431: CHECK: Prefer using the BIT macro
	-:1432: CHECK: Prefer using the BIT macro
	-:1433: CHECK: Prefer using the BIT macro
	-:1435: CHECK: Prefer using the BIT macro
	-:1437: CHECK: Prefer using the BIT macro
	-:1438: CHECK: Prefer using the BIT macro
	-:1439: CHECK: Prefer using the BIT macro
	-:1441: CHECK: Prefer using the BIT macro
	-:1442: CHECK: Prefer using the BIT macro
	-:1443: CHECK: Prefer using the BIT macro
	-:1444: CHECK: Prefer using the BIT macro
	-:1445: CHECK: Prefer using the BIT macro
	-:1447: CHECK: Prefer using the BIT macro
	-:1448: CHECK: Prefer using the BIT macro
	-:1455: CHECK: Prefer using the BIT macro
	-:1457: CHECK: Prefer using the BIT macro
	-:1459: CHECK: Prefer using the BIT macro
	-:1460: CHECK: Prefer using the BIT macro
	-:1464: CHECK: Prefer using the BIT macro
	-:1465: CHECK: Prefer using the BIT macro
	-:1466: CHECK: Prefer using the BIT macro
	-:1467: CHECK: Prefer using the BIT macro
	-:1468: CHECK: Prefer using the BIT macro
	-:1469: CHECK: Prefer using the BIT macro
	-:1470: CHECK: Prefer using the BIT macro
	-:1471: CHECK: Prefer using the BIT macro
	-:1473: CHECK: Prefer using the BIT macro
	-:1474: CHECK: Prefer using the BIT macro
	-:1475: CHECK: Prefer using the BIT macro
	-:1477: CHECK: Prefer using the BIT macro
	-:1478: CHECK: Prefer using the BIT macro
	-:1479: CHECK: Prefer using the BIT macro
	-:1480: CHECK: Prefer using the BIT macro
	-:1481: CHECK: Prefer using the BIT macro
	-:1483: CHECK: Prefer using the BIT macro
	-:1484: CHECK: Prefer using the BIT macro

patches/0006-smiapp-Remove-macros-for-defining-registers-merge-de.patch:

   checkpatch.pl:
	$ cat patches/0006-smiapp-Remove-macros-for-defining-registers-merge-de.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:546: WARNING: line length of 121 exceeds 100 columns
	-:547: WARNING: line length of 120 exceeds 100 columns
	-:559: WARNING: line length of 102 exceeds 100 columns
	-:884: WARNING: line length of 108 exceeds 100 columns
	-:885: WARNING: line length of 108 exceeds 100 columns
	-:888: WARNING: line length of 108 exceeds 100 columns
	-:889: WARNING: line length of 108 exceeds 100 columns
	-:892: WARNING: line length of 108 exceeds 100 columns
	-:893: WARNING: line length of 108 exceeds 100 columns
	-:896: WARNING: line length of 108 exceeds 100 columns
	-:897: WARNING: line length of 108 exceeds 100 columns
	-:898: WARNING: line length of 108 exceeds 100 columns
	-:899: WARNING: line length of 108 exceeds 100 columns
	-:911: WARNING: line length of 108 exceeds 100 columns
	-:912: WARNING: line length of 108 exceeds 100 columns
	-:915: WARNING: line length of 108 exceeds 100 columns
	-:916: WARNING: line length of 108 exceeds 100 columns
	-:1091: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0009-smiapp-Read-CCS-limit-values.patch:

    allmodconfig: return code #0:
	../drivers/media/i2c/smiapp/smiapp-core.c:161:12: warning: ‘ccs_get_limit’ defined but not used [-Wunused-function]

patches/0010-smiapp-Switch-to-CCS-limits.patch:

   checkpatch.pl:
	$ cat patches/0010-smiapp-Switch-to-CCS-limits.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:518: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?

patches/0014-smiapp-Use-CCS-registers.patch:

   checkpatch.pl:
	$ cat patches/0014-smiapp-Use-CCS-registers.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:133: CHECK: Lines should not end with a '['
	-:144: CHECK: Lines should not end with a '['
	-:256: CHECK: Logical continuations should be on the previous line
	-:469: CHECK: Lines should not end with a '('
	-:478: CHECK: Lines should not end with a '('
	-:489: CHECK: Lines should not end with a '('
	-:497: CHECK: Lines should not end with a '('
	-:505: CHECK: Lines should not end with a '('
	-:513: CHECK: Lines should not end with a '('

patches/0016-smiapp-Rename-register-access-functions.patch:

   checkpatch.pl:
	$ cat patches/0016-smiapp-Rename-register-access-functions.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:264: CHECK: Alignment should match open parenthesis
	-:269: CHECK: Alignment should match open parenthesis
	-:281: CHECK: Lines should not end with a '('

patches/0017-smiapp-Internal-rename-to-CCS.patch:

   checkpatch.pl:
	$ cat patches/0017-smiapp-Internal-rename-to-CCS.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:15: WARNING: added, moved or deleted file(s), does MAINTAINERS need updating?
	-:53: WARNING: It's generally not useful to have the filename in the file
	-:66: CHECK: Macro argument reuse 'dim' - possible side-effects?
	-:1615: WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'.
	-:1636: WARNING: Symbolic permissions 'S_IRUGO' are not preferred. Consider using octal permissions '0444'.
	-:2224: WARNING: It's generally not useful to have the filename in the file
	-:2380: WARNING: It's generally not useful to have the filename in the file
	-:2427: CHECK: Prefer using the BIT macro
	-:2443: CHECK: Macro argument reuse 'sensor' - possible side-effects?
	-:2443: CHECK: Macro argument reuse '_quirk' - possible side-effects?
	-:2443: CHECK: Macro argument '_quirk' may be better as '(_quirk)' to avoid precedence issues
	-:2449: CHECK: Macro argument reuse 'sensor' - possible side-effects?
	-:2449: CHECK: Macro argument '_quirk' may be better as '(_quirk)' to avoid precedence issues
	-:2659: WARNING: It's generally not useful to have the filename in the file
	-:2737: CHECK: Prefer kernel type 'u32' over 'uint32_t'
	-:2751: CHECK: Prefer using the BIT macro

patches/0019-smiapp-Rename-as-ccs.patch:

   checkpatch.pl:
	$ cat patches/0019-smiapp-Rename-as-ccs.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:157: WARNING: It's generally not useful to have the filename in the file
	-:190: WARNING: It's generally not useful to have the filename in the file
	-:210: WARNING: It's generally not useful to have the filename in the file
	-:230: WARNING: It's generally not useful to have the filename in the file

patches/0025-dt-bindings-nokia-smia-Convert-to-YAML.patch:

   checkpatch.pl:
	$ cat patches/0025-dt-bindings-nokia-smia-Convert-to-YAML.patch | formail -c | ./scripts/checkpatch.pl --terse --mailback --no-summary --strict
	-:24: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst
	-:93: WARNING: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst


Error #512 when building PDF docs