Re: [PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-23 Thread Hans Verkuil
On 11/21/17 16:27, Ville Syrjälä wrote: > On Mon, Nov 20, 2017 at 04:02:14PM +0100, Hans Verkuil wrote: >> On 11/20/2017 03:51 PM, Ville Syrjälä wrote: >>> On Mon, Nov 20, 2017 at 02:41:28PM +0100, Hans Verkuil wrote: From: Hans Verkuil Some devices (Windows Intel driver!) send a Ve

Re: [PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-21 Thread Ville Syrjälä
On Mon, Nov 20, 2017 at 04:02:14PM +0100, Hans Verkuil wrote: > On 11/20/2017 03:51 PM, Ville Syrjälä wrote: > > On Mon, Nov 20, 2017 at 02:41:28PM +0100, Hans Verkuil wrote: > >> From: Hans Verkuil > >> > >> Some devices (Windows Intel driver!) send a Vendor InfoFrame that > >> uses a payload len

Re: [PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-20 Thread Hans Verkuil
On 11/20/2017 03:51 PM, Ville Syrjälä wrote: > On Mon, Nov 20, 2017 at 02:41:28PM +0100, Hans Verkuil wrote: >> From: Hans Verkuil >> >> Some devices (Windows Intel driver!) send a Vendor InfoFrame that >> uses a payload length of 0x1b instead of the length of 5 or 6 >> that the unpack code expect

Re: [PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-20 Thread Ville Syrjälä
On Mon, Nov 20, 2017 at 02:41:28PM +0100, Hans Verkuil wrote: > From: Hans Verkuil > > Some devices (Windows Intel driver!) send a Vendor InfoFrame that > uses a payload length of 0x1b instead of the length of 5 or 6 > that the unpack code expects. The InfoFrame is padded with 0 by > the source.

[PATCH 1/2] drivers/video/hdmi: allow for larger-than-needed vendor IF

2017-11-20 Thread Hans Verkuil
From: Hans Verkuil Some devices (Windows Intel driver!) send a Vendor InfoFrame that uses a payload length of 0x1b instead of the length of 5 or 6 that the unpack code expects. The InfoFrame is padded with 0 by the source. The current code thinks anything other than 5 or 6 is an error, but large