Hello.

On Tue, 2010-07-06 at 04:03 +0200, ext Mauro Carvalho Chehab wrote:
> Em 04-06-2010 07:34, Matti J. Aaltonen escreveu:
> > +
> > +#define V4L2_CID_FM_RX_BAND                        
> > (V4L2_CID_FM_TX_CLASS_BASE + 1)
> > +enum v4l2_fm_rx_band {
> > +   V4L2_FM_BAND_OTHER              = 0,
> > +   V4L2_FM_BAND_JAPAN              = 1,
> > +   V4L2_FM_BAND_OIRT               = 2
> > +};
> 
> 
> You don't need anything special to define the bandwidth. 
> VIDIOC_G_TUNER/VIDIOC_S_TUNER allows
> negotiating rangelow/rangehigh.

I don't quite get how you want this to be done... Are you saying that
this enum should not be there? At least the wl1273 chip actually
supports two bands so it would seem more straightforward to start by
defining the band and working from there than the other way round. 

B.R. 
Matti A.

> 
> > +
> >  /*
> >   * T U N I N G
> >   */
> > @@ -1377,7 +1389,8 @@ struct v4l2_hw_freq_seek {
> >     enum v4l2_tuner_type  type;
> >     __u32                 seek_upward;
> >     __u32                 wrap_around;
> > -   __u32                 reserved[8];
> > +   __u32                 spacing;
> > +   __u32                 reserved[7];
> >  };
> >  
> >  /*
> 
> I can't comment on your other API proposals, as you didn't send a patch 
> documenting it
> at the API spec.
> 
> Cheers,
> Mauro.


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to