[PATCH 1/9] Includes new structures and defined MACRO

[Ma Ling]

Hi ajax

based on your suggestion, I will fix them you mentioned in next
version.

Thanks a lot!
Ma Ling

On Thu, 2009-02-12 at 03:19 +0800, Adam Jackson wrote:
> On Wed, 2009-02-11 at 17:12 +0800, Ma Ling wrote:
> 
> > +struct cea_audio_blk {
> > +  Uchar descs[3];
> > +};
> 
> I think you have this wrong.  The block is up to 31 bytes long, based on
> the length field.  Each descriptor is three bytes, so there can be up to
> 10.
Yes, this is a bug, I will fix!
> 
> > +struct cea_vendor_blk {
> > +  unsigned char ieee_id[3];
> > +  Uchar  portB:4;
> > +  Uchar  portA:4;
> > +  Uchar  portD:4;
> > +  Uchar  portC:4;
> > +  struct vendor_hdmi hdmi;
> > +};
> 
> I don't see the port bits mentioned in the base CEA spec, at all.  Where
> are you getting this from?
Oh, I saw it in 8.3.2 HDMI Vendor-Specific Data Block (HDMI VSDB) of HDMISpecification13a.pdf
> 
> - ajax