From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH v5 4/9] video: Add generic HDMI infoframe helpers Date: Fri, 22 Feb 2013 18:01:22 +0200 Message-ID: <20130222160122.GP4469@intel.com> References: <1361516611-2517-1-git-send-email-thierry.reding@avionic-design.de> <1361516611-2517-5-git-send-email-thierry.reding@avionic-design.de> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id C0642E60CF for ; Fri, 22 Feb 2013 08:01:26 -0800 (PST) Content-Disposition: inline In-Reply-To: <1361516611-2517-5-git-send-email-thierry.reding@avionic-design.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Thierry Reding Cc: Paulo Zanoni , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org On Fri, Feb 22, 2013 at 08:03:26AM +0100, Thierry Reding wrote: > +enum hdmi_audio_coding_type { > + HDMI_AUDIO_CODING_TYPE_STREAM, > + HDMI_AUDIO_CODING_TYPE_PCM, > + HDMI_AUDIO_CODING_TYPE_AC3, > + HDMI_AUDIO_CODING_TYPE_MPEG1, > + HDMI_AUDIO_CODING_TYPE_MP3, > + HDMI_AUDIO_CODING_TYPE_MPEG2, > + HDMI_AUDIO_CODING_TYPE_AAC_LC, > + HDMI_AUDIO_CODING_TYPE_DTS, > + HDMI_AUDIO_CODING_TYPE_ATRAC, > + HDMI_AUDIO_CODING_TYPE_DSD, > + HDMI_AUDIO_CODING_TYPE_EAC3, > + HDMI_AUDIO_CODING_TYPE_DTS_HD, > + HDMI_AUDIO_CODING_TYPE_MLP, > + HDMI_AUDIO_CODING_TYPE_DST, > + HDMI_AUDIO_CODING_TYPE_WMA_PRO, + HDMI_AUDIO_CODING_TYPE_EXT or something like it? > +}; > + > +enum hdmi_audio_sample_size { > + HDMI_AUDIO_SAMPLE_SIZE_STREAM, > + HDMI_AUDIO_SAMPLE_SIZE_16, > + HDMI_AUDIO_SAMPLE_SIZE_20, > + HDMI_AUDIO_SAMPLE_SIZE_24, > +}; > + > +enum hdmi_audio_sample_frequency { > + HDMI_AUDIO_SAMPLE_FREQUENCY_STREAM, > + HDMI_AUDIO_SAMPLE_FREQUENCY_32000, > + HDMI_AUDIO_SAMPLE_FREQUENCY_44100, > + HDMI_AUDIO_SAMPLE_FREQUENCY_48000, > + HDMI_AUDIO_SAMPLE_FREQUENCY_88200, > + HDMI_AUDIO_SAMPLE_FREQUENCY_96000, > + HDMI_AUDIO_SAMPLE_FREQUENCY_176400, > + HDMI_AUDIO_SAMPLE_FREQUENCY_192000, > +}; > + > +enum hdmi_audio_coding_type_ext { > + HDMI_AUDIO_CODING_TYPE_EXT_STREAM, The spec is a bit confusing w.r.t. this value, but I think this is correct. Any other value is illegal when CT!=3D15, and when CT=3D=3D15, CXT=3D0 means that look at CT (infinite loop anyone?), or stream header. Since the infinite loop option doesn't make sense, looking at the stream header is the only option. Although I think it would be generally safer to avoid that specific combination. I'm sure some "clever" guy would actually implement the infinite loop instead. > + HDMI_AUDIO_CODING_TYPE_EXT_HE_AAC, > + HDMI_AUDIO_CODING_TYPE_EXT_HE_AAC_V2, > + HDMI_AUDIO_CODING_TYPE_EXT_MPEG_SURROUND, > +}; > + -- = Ville Syrj=E4l=E4 Intel OTC