Re: [PATCH 00/11] tda998x updates

On Tue, Sep 29, 2015 at 07:32:16PM +0100, Russell King - ARM Linux wrote:

Hi,

Here's my currently queued TDA998x development work for 4.4.

As there have been no comments against this series, I'll send David a pull request later today.

• Remove some useless NULL checks here variables can't be NULL.
• Return IRQ_HANDLED only if we handled the IRQ, otherwise return IRQ_NONE. This avoids locking the system up if the IRQ gets stuck.
• Re-implement a previous patch "Fix EDID read timeout on HDMI connect" to be much more reliable: an attempt to read the EDID may come in while we're delaying the HPD detect event, violating the critical pause.
• Use Linux types rather than C99 types.
• Handle all outstanding interrupts, rather than only the first interrupt that we discover pending.
• Use more helpers from drivers/video/hdmi.c - this removes our own infoframe checksumming code.
• Remove DRM slave encoder support (which I think no one is using anymore.) This allows other tidy-ups, removing the abstractions to allow slave encoder support to co-exist with component support.

drivers/gpu/drm/i2c/tda998x_drv.c | 487 +++++++++++++++----------------------- 1 file changed, 185 insertions(+), 302 deletions(-)

-- FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up according to speedtest.net.