Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-13 Thread Sharma, Shashank
On 7/13/2017 6:33 PM, Ville Syrjälä wrote: If the mode requires pixel repeat to meet the minimum clock requirement, then we can't just not do pixel repeat. That would violate the spec in other ways, and IIRC we couldn't even generate a low enough clock for it. The YCBCR420 modes I am seeing on s

Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-13 Thread Ville Syrjälä
On Thu, Jul 13, 2017 at 06:19:01PM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 7/13/2017 6:05 PM, Ville Syrjälä wrote: > > On Thu, Jul 13, 2017 at 10:37:53AM +0530, Sharma, Shashank wrote: > >> Regards > >> > >> Shashank > >> > >> > >> On 7/12/2017 10:47 PM, Ville Syrjälä wrot

Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-13 Thread Sharma, Shashank
Regards Shashank On 7/13/2017 6:05 PM, Ville Syrjälä wrote: On Thu, Jul 13, 2017 at 10:37:53AM +0530, Sharma, Shashank wrote: Regards Shashank On 7/12/2017 10:47 PM, Ville Syrjälä wrote: On Mon, Jul 10, 2017 at 04:48:36PM +0530, Shashank Sharma wrote: A source must set output colorspace

Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-13 Thread Ville Syrjälä
On Thu, Jul 13, 2017 at 10:37:53AM +0530, Sharma, Shashank wrote: > Regards > > Shashank > > > On 7/12/2017 10:47 PM, Ville Syrjälä wrote: > > On Mon, Jul 10, 2017 at 04:48:36PM +0530, Shashank Sharma wrote: > >> A source must set output colorspace information in AVI > >> infoframes, so that the

Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-12 Thread Sharma, Shashank
Regards Shashank On 7/12/2017 10:47 PM, Ville Syrjälä wrote: On Mon, Jul 10, 2017 at 04:48:36PM +0530, Shashank Sharma wrote: A source must set output colorspace information in AVI infoframes, so that the sink can decode upcoming frames accordingly. This patch adds a function to add the outp

Re: [PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-12 Thread Ville Syrjälä
On Mon, Jul 10, 2017 at 04:48:36PM +0530, Shashank Sharma wrote: > A source must set output colorspace information in AVI > infoframes, so that the sink can decode upcoming frames > accordingly. > > This patch adds a function to add the output colorspace > information in the AVI infoframes. > > V

[PATCH 08/20] drm: set output colorspace in AVI infoframe

2017-07-10 Thread Shashank Sharma
A source must set output colorspace information in AVI infoframes, so that the sink can decode upcoming frames accordingly. This patch adds a function to add the output colorspace information in the AVI infoframes. V2: Rebase V3: Rebase V4: Rebase V5: Rebase V6: Made patch independent of HDMI out