{"push_id":1528288327,"size":404,"distinct_size":404,"ref":"refs/heads/utilite/dt","head":"d6fdd81062661ed6ac2d14d0f22f99ce1f09d24d","before":"9460146adda93f69ecacc23002ebbc5c7c20939b","commits":[{"sha":"9447082ae666fbf3adbe9c9152117daa31a8b737","author":{"name":"Sakari Ailus","email":"20e5d1d02c2f1dd1e848b2d45f5d9cb636f7f9d6@linux.intel.com"},"message":"[media] smiapp: Implement power-on and power-off sequences without runtime PM\n\nPower on the sensor when the module is loaded and power it off when it is\nremoved.\n\nSigned-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/9447082ae666fbf3adbe9c9152117daa31a8b737"},{"sha":"4bfb934b0067b7f6a24470682c5f7254fd4d8282","author":{"name":"Sakari Ailus","email":"20e5d1d02c2f1dd1e848b2d45f5d9cb636f7f9d6@linux.intel.com"},"message":"[media] smiapp: Make suspend and resume functions __maybe_unused\n\nThe smiapp_suspend() and smiapp_resume() functions will end up being unused\nif CONFIG_PM is enabled but CONFIG_PM_SLEEP is disabled, causing a\ncompiler warning from both of the function definitions. Fix this by\nmarking the functions with __maybe_unused.\n\nSuggested-by: Arnd Bergmann <arnd@arndb.de>\nSigned-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/4bfb934b0067b7f6a24470682c5f7254fd4d8282"},{"sha":"f3854973f196baad5be6b62d8f5ea24b0346b63f","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: fix report_current_latency\n\nIn the (very) small print of the REPORT_CURRENT_LATENCY message there is a\nline that says that the last byte of the message (audio out delay) is only\npresent if the 'audio out compensated' value is 3.\n\nI missed this, and so if this message was sent with a total length of 6 (i.e.\nwithout the audio out delay byte), then it was rejected by the framework\nsince a minimum length of 7 was expected.\n\nFix this minimum length check and update the wrappers in cec-funcs.h to do\nthe right thing based on the message length.\n\nSigned-off-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/f3854973f196baad5be6b62d8f5ea24b0346b63f"},{"sha":"120476123646ba3619c90db7bcbc6f8eea53c990","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: when canceling a message, don't overwrite old status info\n\nWhen a pending message was canceled (e.g. due to a timeout), then the\nold tx_status info was overwritten instead of ORed. The same happened\nwith the tx_error_cnt field. So just modify them instead of overwriting\nthem.\n\nSigned-off-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/120476123646ba3619c90db7bcbc6f8eea53c990"},{"sha":"a24f56d47930492c94ef6875bf45adf7607ca1a4","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: CEC_MSG_GIVE_FEATURES should abort for CEC version < 2\n\nThis is a 2.0 only message, so it should return Feature Abort if the\nadapter is configured for CEC version 1.4.\n\nRight now it does nothing, which means that the sender will time out.\n\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/a24f56d47930492c94ef6875bf45adf7607ca1a4"},{"sha":"7af26f889eb67db272021a939f7d4a57e96dd961","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: update log_addr[] before finishing configuration\n\nThe loop that sets the unused logical addresses to INVALID should be\ndone before 'configured' is set to true. This ensures that cec_log_addrs\nis consistent before it will be used.\n\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/7af26f889eb67db272021a939f7d4a57e96dd961"},{"sha":"52bc30fda9622f492427d484bd4dd8ee42cc4667","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: replace cec_report_features by cec_fill_msg_report_features\n\nThe fill function just fills in the cec_msg struct, it doesn't transmit\nthe message. This is now done explicitly.\n\nThis makes it possible to switch to transmitting this message with adap->lock\nheld.\n\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/52bc30fda9622f492427d484bd4dd8ee42cc4667"},{"sha":"d3d64bc7408f1ff0b0ff8354056e2a48eda5886d","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: move cec_report_phys_addr into cec_config_thread_func\n\nIt's only a small function and this makes it easier to switch to\ntransmitting the message with adap->lock held in the next patch.\n\nSigned-off-by: Hans Verkuil <hansverk@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/d3d64bc7408f1ff0b0ff8354056e2a48eda5886d"},{"sha":"f60f35609f89ef4fee73776bc1ef697923251995","author":{"name":"Hans Verkuil","email":"bba902086d13d09b24d7a3f060bbc824fae5d6d0@cisco.com"},"message":"[media] cec: fix race between configuring and unconfiguring\n\nThis race was discovered by running cec-compliance -A with the cec module debug\nparameter set to 2: suddenly the test would fail.\n\nIt turns out that this happens when the test configures the adapter in\nnon-blocking mode, then it waits for the CEC_EVENT_STATE_CHANGE event and once\nthe event is received it unconfigures the adapter.\n\nWhat happened was that the unconfigure was executed while the configure was\nstill transmitting the Report Features and Report Physical Address messages.\nThis messed up the internal state of the cec_adapter.\n\nThe fix is to transmit those messages with the adap->lock mutex held (this will\njust queue them up in the internal transmit queue, and not actually transmit\nanything yet). Only unlock the mutex once everything is done. The main thread\nwill dequeue the messages from the internal transmit queue and transmit them\none by one, unless an unconfigure was done, and in that case any messages are\njust dropped.\n\nSigned-off-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/f60f35609f89ef4fee73776bc1ef697923251995"},{"sha":"78ccbf9ff89bd7a20d36be039cb3eab71081648c","author":{"name":"Christoph Hellwig","email":"ed9f47d6dfaca04b245b5f2816a0d3e9462a0b83@lst.de"},"message":"[media] media/cobalt: use pci_irq_allocate_vectors\n\nSimply the interrupt setup by using the new PCI layer helpers.\n\nDespite using pci_enable_msi_range, this driver was only requesting a\nsingle MSI vector anyway.\n\nSigned-off-by: Christoph Hellwig <hch@lst.de>\nSigned-off-by: Hans Verkuil <hans.verkuil@cisco.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/78ccbf9ff89bd7a20d36be039cb3eab71081648c"},{"sha":"48775cb73c2e26b7ca9d679875a6e570c8b8e124","author":{"name":"Max Kellermann","email":"d0bf04c4eeeb6d33cd5a0a197a096f6ee6b9c1c8@gmail.com"},"message":"[media] pctv452e: move buffer to heap, no mutex\n\ncommit 73d5c5c864f4 (\"[media] pctv452e: don't do DMA on stack\") caused\na NULL pointer dereference which occurs when dvb_usb_init()\ncalls dvb_usb_device_power_ctrl() for the first time, before the\nfrontend has been attached. It also caused a recursive deadlock because\ntt3650_ci_msg_locked() has already locked the mutex.\n\nSo, partially revert it, but move the buffer to the heap\n(DMA capable), not to the stack (may not be DMA capable).\nInstead of sharing one buffer which needs mutex protection,\ndo a new heap allocation for each call.\n\nFixes: commit 73d5c5c864f4 (\"[media] pctv452e: don't do DMA on stack\")\n\nCc: stable@vger.kernel.org # For Kernel 4.9\nSigned-off-by: Max Kellermann <max.kellermann@gmail.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/48775cb73c2e26b7ca9d679875a6e570c8b8e124"},{"sha":"aff808e813fc2d311137754165cf53d4ee6ddcc2","author":{"name":"Laurent Pinchart","email":"8b14580067f63df365ab75f40300acb6346a24eb@ideasonboard.com"},"message":"[media] v4l: tvp5150: Reset device at probe time, not in get/set format handlers\n\nThe tvp5150 doesn't support format setting through the subdev pad API\nand thus implements the set format handler as a get format operation.\nThe single handler, tvp5150_fill_fmt(), resets the device by calling\ntvp5150_reset(). This causes malfunction as the device can be reset at\nwill, possibly from userspace when the subdev userspace API is enabled.\n\nThe reset call was added in commit ec2c4f3f93cb (\"[media] media:\ntvp5150: Add mbus_fmt callbacks\"), probably as an attempt to set the\ndevice to a known state before detecting the current TV standard.\nHowever, the get format handler doesn't access the hardware to get the\nTV standard since commit 963ddc63e20d (\"[media] media: tvp5150: Add\ncropping support\"). There is thus no need to reset the device when\ngetting the format.\n\nHowever, removing the tvp5150_reset() from the get/set format handlers\nresults in the function not being called at all if the bridge driver\ndoesn't use the .reset() operation. The operation is nowadays abused and\nshouldn't be used, so shouldn't expect bridge drivers to call it. To\nmake sure the device is properly initialize, move the reset call from\nthe format handlers to the probe function.\n\nCc: stable@vger.kernel.org # For Kernel 4.5 and upper\nSigned-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/aff808e813fc2d311137754165cf53d4ee6ddcc2"},{"sha":"b4b2de386bbb6589d81596999d4a924928dc119b","author":{"name":"Laurent Pinchart","email":"8b14580067f63df365ab75f40300acb6346a24eb@ideasonboard.com"},"message":"[media] v4l: tvp5150: Fix comment regarding output pin muxing\n\nThe FID/GLCO/VLK/HVLK and INTREQ/GPCL/VBLK pins are muxed differently\ndepending on whether the input is an S-Video or composite signal. The\ncomment that explains the logic doesn't reflect the code. It appears\nthat the comment is incorrect, as disabling the output data bus in\ncomposite mode makes no sense. Update the comment to match the code.\n\nWhile at it define macros for the MISC_CTL register bits, the code is\ntoo confusing with numerical values.\n\nCc: stable@vger.kernel.org # For Kernel\t4.5 and\tupper\nSigned-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/b4b2de386bbb6589d81596999d4a924928dc119b"},{"sha":"79d6205a3f741c9fb89cfc47dfa0eddb1526726d","author":{"name":"Laurent Pinchart","email":"8b14580067f63df365ab75f40300acb6346a24eb@ideasonboard.com"},"message":"[media] v4l: tvp5150: Don't override output pinmuxing at stream on/off time\n\nThe s_stream() handler incorrectly writes the whole MISC_CTL register to\nenable or disable the outputs, overriding the output pinmuxing\nconfiguration. Fix it to only touch the output enable bits.\n\nThe CONF_SHARED_PIN register is also written by the same function,\nresulting in muxing the INTREQ signal instead of the VBLK/GPCL signal on\nthe INTREQ/GPCL/VBLK pin. As the driver doesn't support interrupts this\nis obviously incorrect, and breaks operation on other devices. Fix it by\nremoving the write.\n\nCc: stable@vger.kernel.org # For Kernel\t4.5 and\tupper\nSigned-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/79d6205a3f741c9fb89cfc47dfa0eddb1526726d"},{"sha":"4dd19196c5539c377beaa9850fac30c18318c7a1","author":{"name":"Arnd Bergmann","email":"72a0f6d3b182607e6d5c44b93e1abb7b23ee18f3@arndb.de"},"message":"[media] dvb: avoid warning in dvb_net\n\nWith gcc-5 or higher on x86, we can get a bogus warning in the\ndvb-net code:\n\ndrivers/media/dvb-core/dvb_net.c: In function 'dvb_net_ule':\narch/x86/include/asm/string_32.h:78:22: error: '*((void *)&dest_addr+4)' may be used uninitialized in this function [-Werror=maybe-uninitialized]\n\nThe problem here is that gcc doesn't track all of the conditions\nto prove it can't end up copying uninitialized data.\nThis changes the logic around so we zero out the destination\naddress earlier when we determine that it is not set here.\nThis allows the compiler to figure it out.\n\nSigned-off-by: Arnd Bergmann <arnd@arndb.de>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/4dd19196c5539c377beaa9850fac30c18318c7a1"},{"sha":"c739c0a7c3c2472d7562b8f802cdce44d2597c8b","author":{"name":"Arnd Bergmann","email":"72a0f6d3b182607e6d5c44b93e1abb7b23ee18f3@arndb.de"},"message":"[media] s5k4ecgx: select CRC32 helper\n\nA rare randconfig build failure shows up in this driver when\nthe CRC32 helper is not there:\n\ndrivers/media/built-in.o: In function `s5k4ecgx_s_power':\ns5k4ecgx.c:(.text+0x9eb4): undefined reference to `crc32_le'\n\nThis adds the 'select' that all other users of this function have.\n\nFixes: 8b99312b7214 (\"[media] Add v4l2 subdev driver for S5K4ECGX sensor\")\n\nSigned-off-by: Arnd Bergmann <arnd@arndb.de>\nSigned-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/c739c0a7c3c2472d7562b8f802cdce44d2597c8b"},{"sha":"0e0694ff1a7791274946b7f51bae692da0001a08","author":{"name":"Mauro Carvalho Chehab","email":"ea19850a825a0b2eb1de0168fb2a06f5170bd9e2@s-opensource.com"},"message":"Merge branch 'patchwork' into v4l_for_linus\n\n* patchwork:\n  [media] s5k4ecgx: select CRC32 helper\n  [media] dvb: avoid warning in dvb_net\n  [media] v4l: tvp5150: Don't override output pinmuxing at stream on/off time\n  [media] v4l: tvp5150: Fix comment regarding output pin muxing\n  [media] v4l: tvp5150: Reset device at probe time, not in get/set format handlers\n  [media] pctv452e: move buffer to heap, no mutex\n  [media] media/cobalt: use pci_irq_allocate_vectors\n  [media] cec: fix race between configuring and unconfiguring\n  [media] cec: move cec_report_phys_addr into cec_config_thread_func\n  [media] cec: replace cec_report_features by cec_fill_msg_report_features\n  [media] cec: update log_addr[] before finishing configuration\n  [media] cec: CEC_MSG_GIVE_FEATURES should abort for CEC version < 2\n  [media] cec: when canceling a message, don't overwrite old status info\n  [media] cec: fix report_current_latency\n  [media] smiapp: Make suspend and resume functions __maybe_unused\n  [media] smiapp: Implement power-on and power-off sequences without runtime PM","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/0e0694ff1a7791274946b7f51bae692da0001a08"},{"sha":"4ea33ef0f9e95b69db9131d7afd98563713e81b0","author":{"name":"Sven Eckelmann","email":"9d878b831e8ad202268d6003fc179621a34463a2@narfation.org"},"message":"batman-adv: Decrease hardif refcnt on fragmentation send error\n\nAn error before the hardif is found has to free the skb. But every error\nafter that has to free the skb + put the hard interface.\n\nFixes: 8def0be82dd1 (\"batman-adv: Consume skb in batadv_frag_send_packet\")\nSigned-off-by: Sven Eckelmann <sven@narfation.org>\nSigned-off-by: Simon Wunderlich <sw@simonwunderlich.de>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/4ea33ef0f9e95b69db9131d7afd98563713e81b0"},{"sha":"fd7c99142d77dc4a851879a66715abf12a3193fb","author":{"name":"Dave Martin","email":"efb8cc9d02b9c95845d8a14436d3a7276e72ae4e@arm.com"},"message":"tile/ptrace: Preserve previous registers for short regset write\n\nEnsure that if userspace supplies insufficient data to\nPTRACE_SETREGSET to fill all the registers, the thread's old\nregisters are preserved.\n\nCc: stable@vger.kernel.org\nSigned-off-by: Dave Martin <Dave.Martin@arm.com>\nSigned-off-by: Chris Metcalf <cmetcalf@mellanox.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/fd7c99142d77dc4a851879a66715abf12a3193fb"},{"sha":"3e70c5d6ea510e38f612d07fa0fd7487277b7087","author":{"name":"Nicolas Iooss","email":"c0d64d4e47db0f189cbd8dcd56b59d380e758d07@m4x.org"},"message":"drm/i915/gvt: verify functions types in new_mmio_info()\n\nThe current prototype of new_mmio_info() uses void* for parameters read\nand write, which are functions with precise calling conventions\n(argument types and return type). Write down these conventions in\nnew_mmio_info() definition.\n\nThis has been reported by the following warnings when clang is used to\nbuild the kernel:\n\n    drivers/gpu/drm/i915/gvt/handlers.c:124:21: error: pointer type\n    mismatch ('void *' and 'int (*)(struct intel_vgpu *, unsigned int,\n    void *, unsigned int)') [-Werror,-Wpointer-type-mismatch]\n            info->read = read ? read : intel_vgpu_default_mmio_read;\n                              ^ ~~~~   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n    drivers/gpu/drm/i915/gvt/handlers.c:125:23: error: pointer type\n    mismatch ('void *' and 'int (*)(struct intel_vgpu *, unsigned int,\n    void *, unsigned int)') [-Werror,-Wpointer-type-mismatch]\n            info->write = write ? write : intel_vgpu_default_mmio_write;\n                                ^ ~~~~~   ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n\nThis allows the compiler to detect that sbi_ctl_mmio_write() returns a\n\"bool\" value instead of an expected \"int\" one. Fix this.\n\nSigned-off-by: Nicolas Iooss <nicolas.iooss_linux@m4x.org>\nSigned-off-by: Zhenyu Wang <zhenyuw@linux.intel.com>","distinct":true,"url":"https://api.github.com/repos/vcspth/linux/commits/3e70c5d6ea510e38f612d07fa0fd7487277b7087"}]}
{"actor":{"display_login":"vcspth"}}