{"push_id":2545213592,"size":217,"distinct_size":1,"ref":"refs/heads/mediatree/master-ddb-ioctl-v1","head":"5548678e8ab2eecaef37b13f6cff3feefa64f83e","before":"a888e23ac3e36ade162905b4b6f3818036a642c6","commits":[{"sha":"42a182282ea2426d56b2d63be634ee419194c45c","author":{"name":"Mauro Carvalho Chehab","email":"ad86ba2154032c9f55743a190faa2459a9d61d42@s-opensource.com"},"message":"media: si470x: fix a typo at the Makefile causing build issues\n\nInstead of +=, the rule had :=, with actually disables build\nof everything else.\n\nFixes: 58757984ca3c (\"media: si470x: allow build both USB and I2C at the same time\")\nReported-by: Daniel Scheller <d.scheller.oss@gmail.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/42a182282ea2426d56b2d63be634ee419194c45c"},{"sha":"fef6cc6b3618279f243a5d046a1c86a1df7311f6","author":{"name":"Mauro Carvalho Chehab","email":"ad86ba2154032c9f55743a190faa2459a9d61d42@s-opensource.com"},"message":"media: v4l2-compat-ioctl32: fix several __user annotations\n\nSmatch report several issues with bad __user annotations:\n\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:447:21: warning: incorrect type in argument 1 (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:447:21:    expected void [noderef] <asn:1>*uptr\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:447:21:    got void *<noident>\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:621:21: warning: incorrect type in argument 1 (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:621:21:    expected void const volatile [noderef] <asn:1>*<noident>\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:621:21:    got struct v4l2_plane [noderef] <asn:1>**<noident>\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:693:13: warning: incorrect type in argument 1 (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:693:13:    expected void [noderef] <asn:1>*uptr\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:693:13:    got void *[assigned] base\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:871:13: warning: incorrect type in assignment (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:871:13:    expected struct v4l2_ext_control [noderef] <asn:1>*kcontrols\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:871:13:    got struct v4l2_ext_control *<noident>\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:957:13: warning: incorrect type in assignment (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:957:13:    expected unsigned char [usertype] *__pu_val\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:957:13:    got void [noderef] <asn:1>*\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:973:13: warning: incorrect type in argument 1 (different address spaces)\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:973:13:    expected void [noderef] <asn:1>*uptr\n  drivers/media/v4l2-core/v4l2-compat-ioctl32.c:973:13:    got void *[assigned] edid\n\nFix them.\n\nReviewed-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/fef6cc6b3618279f243a5d046a1c86a1df7311f6"},{"sha":"6dd0394f5fcd51ce6892627eefbadcf42441e94d","author":{"name":"Mauro Carvalho Chehab","email":"ad86ba2154032c9f55743a190faa2459a9d61d42@s-opensource.com"},"message":"media: v4l2-compat-ioctl32: better name userspace pointers\n\nIn the past, \"up\" were an acronym for \"user pointer\" and \"kp\" for\n\"kernel pointer\". However, since commit a1dfb4c48cc1 (\"media:\nv4l2-compat-ioctl32.c: refactor compat ioctl32 logic\"), both\nare now __user pointers.\n\nSo, the usage of \"kp\" is really misleading there. So, rename\nboth to just \"p32\" and \"p64\" everywhere it occurs, in order to\nmake peace with this file's namespace.\n\nThere are two exceptions to \"up/kp\" nomenclature: at\nalloc_userspace() and at do_video_ioctl().\n\nThere, a new userspace pointer were allocated, in order to store\nthe 64 bits version of the ioctl. Those were called as \"up_native\",\nwith is, IMHO, an even worse name, as \"native\" could mislead of\nbeing the arguments that were filled from userspace. I almost\nrenamed it to just \"p64\", but, after thinking more about that,\nit sounded better to call it as \"new_p64\", as this makes clearer\nthat this is the data structure that was allocated inside this\nfile in order to be used to pass/retrieve data when calling the\n64-bit ready file->f_op->unlocked_ioctl() function.\n\nSuggested-by: Sakari Ailus <sakari.ailus@linux.intel.com>\nSuggested-by: Hans Verkuil <hverkuil@xs4all.nl>\nReviewed-by: Hans Verkuil <hans.verkuil@cisco.com>\nAcked-by: Sakari Ailus <sakari.ailus@linux.intel.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/6dd0394f5fcd51ce6892627eefbadcf42441e94d"},{"sha":"09c033b66596a9c38f61875c80c674c4fed61b86","author":{"name":"Mauro Carvalho Chehab","email":"ad86ba2154032c9f55743a190faa2459a9d61d42@s-opensource.com"},"message":"media: v4l2-compat-ioctl32: simplify casts\n\nMaking the cast right for get_user/put_user is not trivial, as\nit needs to ensure that the types are the correct ones.\n\nImprove it by using macros.\n\nTested with vivid with:\n\t$ sudo modprobe vivid no_error_inj=1\n\t$ v4l2-compliance-32bits -a -s10 >32bits && v4l2-compliance-64bits -a -s10 > 64bits && diff -U0 32bits 64bits\n\t--- 32bits\t2018-04-17 11:18:29.141240772 -0300\n\t+++ 64bits\t2018-04-17 11:18:40.635282341 -0300\n\t@@ -1 +1 @@\n\t-v4l2-compliance SHA   : bc71e4a67c6fbc5940062843bc41e7c8679634ce, 32 bits\n\t+v4l2-compliance SHA   : bc71e4a67c6fbc5940062843bc41e7c8679634ce, 64 bits\n\nUsing the latest version of v4l-utils with this patch applied:\n\thttps://patchwork.linuxtv.org/patch/48746/\n\nReviewed-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/09c033b66596a9c38f61875c80c674c4fed61b86"},{"sha":"1d338b86e17d87215cf57b1ad1d13b2afe582d33","author":{"name":"Mauro Carvalho Chehab","email":"ad86ba2154032c9f55743a190faa2459a9d61d42@s-opensource.com"},"message":"media: v4l2-compat-ioctl32: better document the code\n\nThis file does a lot of non-trivial struff. Document it using\nkernel-doc markups where needed and improve the comments inside\ndo_video_ioctl().\n\nReviewed-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/1d338b86e17d87215cf57b1ad1d13b2afe582d33"},{"sha":"b7e1e6859fbf60519fd82d7120cee106a6019512","author":{"name":"Suman Anna","email":"68636d6b0a4dfe1213ee9d7712e75a8cc339119f@ti.com"},"message":"media: omap3isp: fix unbalanced dma_iommu_mapping\n\nThe OMAP3 ISP driver manages its MMU mappings through the IOMMU-aware\nARM DMA backend. The current code creates a dma_iommu_mapping and\nattaches this to the ISP device, but never detaches the mapping in\neither the probe failure paths or the driver remove path resulting\nin an unbalanced mapping refcount and a memory leak. Fix this properly.\n\nReported-by: Pavel Machek <pavel@ucw.cz>\nSigned-off-by: Suman Anna <s-anna@ti.com>\nTested-by: Pavel Machek <pavel@ucw.cz>\nReviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\nSigned-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/b7e1e6859fbf60519fd82d7120cee106a6019512"},{"sha":"29e61d6ef061b012d320327af7dbb3990e75be45","author":{"name":"Kai-Heng Feng","email":"06af28c154da900e58fef391e9e9415561e1d8ae@canonical.com"},"message":"media: cx231xx: Add support for AverMedia DVD EZMaker 7\n\nUser reports AverMedia DVD EZMaker 7 can be driven by VIDEO_GRABBER.\nAdd the device to the id_table to make it work.\n\nBugLink: https://bugs.launchpad.net/bugs/1620762\n\nCc: stable@vger.kernel.org\nSigned-off-by: Kai-Heng Feng <kai.heng.feng@canonical.com>\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/29e61d6ef061b012d320327af7dbb3990e75be45"},{"sha":"0fd6e1217303c98550de3552c65837c3f56729da","author":{"name":"Colin Ian King","email":"d100aa70785e45ee8deab668147d941941915c21@canonical.com"},"message":"media: cec: set ev rather than v with CEC_PIN_EVENT_FL_DROPPED bit\n\nSetting v with the CEC_PIN_EVENT_FL_DROPPED is incorrect, instead\nev should be set with this bit. Fix this.\n\nDetected by CoverityScan, CID#1467974 (\"Extra high-order bits\")\n\nFixes: 6ec1cbf6b125 (\"media: cec: improve CEC pin event handling\")\n\nSigned-off-by: Colin Ian King <colin.king@canonical.com>\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/0fd6e1217303c98550de3552c65837c3f56729da"},{"sha":"536f0f55384cfcc25d1f8f3cfd4a44048dca3b46","author":{"name":"Koji Matsuoka","email":"db42c32d091ec1f7514297abc6313f329b726b2a@renesas.com"},"message":"media: rcar-vin: Fix image alignment for setting pre clipping\n\nIn Video Pixel/Line Pre-Clip Register, the setting value can be\nset in 1 line unit, but it can only be specified as a multiple of\n4 by v4l_bound_align_image function().\nSo correct that it can be specified in 1 line unit with this patch.\n\nSigned-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com>\nSigned-off-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>\nAcked-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/536f0f55384cfcc25d1f8f3cfd4a44048dca3b46"},{"sha":"ed8c34d7ec35b0a265f1b4d9ed6ab75e9f6facd7","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: report receiver and transmitter type on device register\n\nOn the raspberry pi, we might have two lirc devices; one for sending and\none for receiving. This change makes it much more apparent which one\nis which.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/ed8c34d7ec35b0a265f1b4d9ed6ab75e9f6facd7"},{"sha":"a86d6df84ae6eb1fd1ca6cbd03f16637674a6af8","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: set timeout to smallest value required by enabled protocols\n\nThe longer the IR timeout, the longer the rc device waits until delivering\nthe trailing space. So, by reducing this timeout, we reduce the delay for\nthe last scancode to be delivered.\n\nNote that the lirc daemon disables all protocols, in which case we revert\nback to the default value.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/a86d6df84ae6eb1fd1ca6cbd03f16637674a6af8"},{"sha":"95d1544eb643847e05df06c3de252609593c9073","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: add ioctl to get the current timeout\n\nSince the kernel now modifies the timeout, make it possible to retrieve\nthe current value.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/95d1544eb643847e05df06c3de252609593c9073"},{"sha":"284922562b8170a030fb130ead98224f7211d1ef","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: per-protocol repeat period and minimum keyup timer\n\nEach IR protocol has its own repeat period. We can minimise the keyup\ntimer to be the protocol period + IR timeout. This makes keys less\n\"sticky\" and makes IR more reactive and nicer to use.\n\nThis feature was previously attempted in commit d57ea877af38 (\"media: rc:\nper-protocol repeat period\"), but that did not take the IR timeout into\naccount, and had to be reverted.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/284922562b8170a030fb130ead98224f7211d1ef"},{"sha":"c421c62a4a08aba220ad7176ef4aa4b0ced9480a","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mce_kbd decoder: low timeout values cause double keydowns\n\nThe mce keyboard repeats pressed keys every 100ms. If the IR timeout\nis set to less than that, we send key up events before the repeat\narrives, so we have key up/key down for each IR repeat.\n\nThe keyboard ends any sequence with a 0 scancode, in which case all keys\nare cleared so there is no need to run the timeout timer: it only exists\nfor the case that the final 0 was not received.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/c421c62a4a08aba220ad7176ef4aa4b0ced9480a"},{"sha":"539327608dbedf242131fc6f1b8bcca712952c45","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mce_kbd protocol encodes two scancodes\n\nIf two keys are pressed, then both keys are encoded in the scancode. This\nmakes the mce keyboard more responsive.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/539327608dbedf242131fc6f1b8bcca712952c45"},{"sha":"63039c29f7a4ce8a8bd165173840543c0098d7b0","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mce_kbd decoder: fix stuck keys\n\nThe MCE Remote sends a 0 scancode when keys are released. If this is not\nreceived or decoded, then keys can get \"stuck\"; the keyup event is not\nsent since the input_sync() is missing from the timeout handler.\n\nCc: stable@vger.kernel.org\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/63039c29f7a4ce8a8bd165173840543c0098d7b0"},{"sha":"cb5bd0575c41ac34c62a7784da7232d1c90e6eca","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mce_kbd decoder: remove superfluous call to input_sync\n\nThere is nothing to sync in this code path.\n\nReported-by: Matthias Reichl <hias@horus.com>\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/cb5bd0575c41ac34c62a7784da7232d1c90e6eca"},{"sha":"53a62800efb2f0ebc1972cab242461ae8c57bfd4","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mce_kbd decoder: fix race condition\n\nThe MCE keyboard sends both key down and key up events. We have a timeout\nhandler mce_kbd_rx_timeout() in case the keyup event is never received;\nhowever, this may race with new key down events from occurring.\n\nThe race is that key down scancode arrives and key down events are\ngenerated. The timeout handler races this and generates key up events\nstraight afterwards. Since the keyboard generates scancodes every 100ms,\nmost likely the keys will be repeated 100ms later, and now we have new\nkey down events and the user sees duplicate key presses.\n\nReported-by: Matthias Reichl <hias@horus.com>\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/53a62800efb2f0ebc1972cab242461ae8c57bfd4"},{"sha":"91352b572786a49f26a12e8b1b7571bc3df64bab","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mceusb: IR of length 0 means IR timeout, not reset\n\nThe last usb packet with IR data will end with 0x80 (MCE_IRDATA_TRAILER).\nIf we reset the decoder state at this point, IR decoding can fail.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/91352b572786a49f26a12e8b1b7571bc3df64bab"},{"sha":"877f1a7cee3f5dd7a08bdabb52948f155c8f267e","author":{"name":"Sean Young","email":"d7e19930cc1f42c2d0781f4d9e6f1fe5891bf9cf@mess.org"},"message":"media: rc: mceusb: allow the timeout to be configurable\n\nmceusb devices have a default timeout of 100ms, but this can be changed.\n\nSigned-off-by: Sean Young <sean@mess.org>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":false,"url":"https://api.github.com/repos/herrnst/dddvb-linux-kernel/commits/877f1a7cee3f5dd7a08bdabb52948f155c8f267e"}]}
{"actor":{"display_login":"herrnst"}}