[alsa-devel] [PATCH v2 1/4] ALSA: hda - add flags and routines to get devices selection info for DP1.2 MST
    Lin, Mengdong 
    mengdong.lin at intel.com
       
    Tue Aug 27 05:16:50 CEST 2013
    
    
  
Hi Takashi,
> -----Original Message-----
> From: Takashi Iwai [mailto:tiwai at suse.de]
> Sent: Monday, August 26, 2013 8:44 PM
> >
> > +/* display pin's device list entry */ struct  hda_devlist_entry {
> > +	unsigned int present_detect:1;
> > +	unsigned int eld_valid:1;
> > +	unsigned int inactive:1;
> > +};
> 
> Can't it be simply an unsigned int containing the raw value?
> You defined the bit flag in the above, and copying the bit field from/to the bit
> flag looks redundant.
> 
How about using "u8"? 
Since each device entry is only 4 bits in the response to "Get Device List Entry" Control.
Thanks
Mengdong
    
    
More information about the Alsa-devel
mailing list