Linux kernel stable tree (mirror) https://git.radix-linux.su/kernel/linux.git/atom?h=linux-4.20.y 2018-10-04T00:32:14+00:00 2018-10-04T00:32:14+00:00 Dave Airlie airlied@redhat.com 2018-10-04T00:28:27+00:00 urn:sha1:d8938c981f58ee344687b7910a611ac345960045 two tda9950 fixes. Signed-off-by: Dave Airlie <airlied@redhat.com> From: Russell King <rmk@armlinux.org.uk> Link: https://patchwork.freedesktop.org/patch/msgid/20181001162948.GA9508@rmk-PC.armlinux.org.uk 2018-09-20T16:13:23+00:00 Hans Verkuil hverkuil@xs4all.nl 2018-08-27T12:28:50+00:00 urn:sha1:e0dccce1193f87597548d0db6ecc942fb92c04cd The CEC_TX_STATUS_MAX_RETRIES should be set for errors only to prevent the CEC framework from retrying the transmit. If the transmit was successful, then don't set this flag. Found by running 'cec-compliance -A' on a beaglebone box. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> 2018-09-20T16:13:23+00:00 Colin Ian King colin.king@canonical.com 2018-05-27T21:42:55+00:00 urn:sha1:d98627d1360d55e3b28f702caca8b6342c4a4e45 Currently the check to see if the timeout has reached zero is incorrect and the check is instead checking if the timeout is non-zero and not zero, hence it will break out of the loop on the first iteration and the msleep is never executed. Fix this by breaking from the loop when timeout is zero. Detected by CoverityScan, CID#1469404 ("Logically Dead Code") Fixes: f0316f93897c ("drm/i2c: tda9950: add CEC driver") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> 2018-08-17T17:40:09+00:00 Linus Torvalds torvalds@linux-foundation.org 2018-08-17T17:40:09+00:00 urn:sha1:84f5685230c6aeb427947a2299dcb54c77863df0 Pull VLA removal leftovers from Kees Cook: - bus/imx-weim: Use maximum register count to avoid VLA - drm/i2c/tda9950: Use maximum CEC message size to avoid VLA * tag 'vla-leftovers-v4.19-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/kees/linux: bus: imx-weim: Remove VLA usage drm/i2c: tda9950: Remove VLA usage 2018-08-13T20:40:52+00:00 Kees Cook keescook@chromium.org 2018-06-20T04:38:31+00:00 urn:sha1:699112f5e831c088ff6aeea594d23ebecf6bd806 In the quest to remove all stack VLA usage from the kernel[1], this sets the buffer to maximum size and adds a sanity check. [1] https://lkml.kernel.org/r/CA+55aFzCG-zNmZwX4A2FQpadafLfEzK6CC=qPXydAacU1RqZWA@mail.gmail.com Cc: David Airlie <airlied@linux.ie> Cc: Hans Verkuil <hans.verkuil@cisco.com> Cc: Russell King <rmk+kernel@armlinux.org.uk> Cc: dri-devel@lists.freedesktop.org Signed-off-by: Kees Cook <keescook@chromium.org> 2018-08-07T19:52:22+00:00 Dave Airlie airlied@redhat.com 2018-08-07T19:52:15+00:00 urn:sha1:a7ccc5a43b829a96b4c62c0de71fc8c6fbd17b8a From: Russell King <rmk@armlinux.org.uk> As per the patches posted, discussed and tested by Peter Rosin, this converts TDA998x to a bridge driver, while still allowing Armada and TI LCDC to continue using it as they always have done. It also gets rid of the private .fill_modes function, and tweaks the TMDS divider calculation to be more correct to the available information. [airlied: fixed two conflicts] Signed-off-by: Dave Airlie <airlied@redhat.com> Link: https://patchwork.freedesktop.org/patch/msgid/20180802093421.GA29670@rmk-PC.armlinux.org.uk 2018-08-07T09:32:03+00:00 Russell King rmk+kernel@armlinux.org.uk 2018-08-02T09:27:15+00:00 urn:sha1:926a299c42e38bbe8bb909eae0146e676b66afe4 The serializer PLL divider is a power-of-two divider, so our calculation which assumes that it's a numerical divider is incorrect. Replace it with one that results in a power-of-two divider value instead. Tested with all supported modes with a Samsung S24C750. Tested-by: Peter Rosin <peda@axentia.se> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> 2018-08-07T09:32:03+00:00 Russell King rmk+kernel@armlinux.org.uk 2018-08-02T09:27:15+00:00 urn:sha1:a3d335f5de7cdd714737a9f8bdcc7b205b0f8d5e We can achieve the same effect via the get_modes() method, rather than wrapping the fill_modes helper. Tested-by: Peter Rosin <peda@axentia.se> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> 2018-08-07T09:32:03+00:00 Russell King rmk+kernel@armlinux.org.uk 2018-08-02T09:27:15+00:00 urn:sha1:b073a70ecd37bc6e7c25a47315cd8d65e7b313fd Move the mode_valid() implementation to the bridge instead of the connector, as we're checking the bridge's capabilities. Tested-by: Peter Rosin <peda@axentia.se> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk> 2018-08-07T09:32:03+00:00 Russell King rmk+kernel@armlinux.org.uk 2018-08-02T09:25:19+00:00 urn:sha1:5a03f5346fedc8b5f1a8601aa46a0930ba56a7ae Register the bridge outside of the component helper as we have drivers that wish to use the tda998x without its encoder. Tested-by: Peter Rosin <peda@axentia.se> Signed-off-by: Russell King <rmk+kernel@armlinux.org.uk>