At Thu, 10 May 2012 13:42:09 +0200,
Takashi Iwai wrote:
>
> At Thu, 10 May 2012 12:20:05 +0100,
> Alan Cox wrote:
> >
> > On Thu, 10 May 2012 09:10:16 +0200
> > Takashi Iwai <tiwai@xxxxxxx> wrote:
> >
> > > Add the support for audio clients to VGA-switcheroo for handling the
> > > HDMI audio controller together with VGA switching. The id of the
> > > audio controller should be given explicity at registration time unlike
> > > the video controller.
> >
> > It would I think be a lot cleaner and more future proof to pass a
> >
> > struct hdmi_audio_switch_ops
> >
> > or some similar named object with an array of function pointers ?
>
> That would be a good option, indeed.
>
> In my patch series, I just didn't want to break the existing API,
> so I kept the current style.
Dave, do you prefer the way with an ops struct as Alan suggested?
For example, I can make like
struct vga_switcheroo_client_ops {
void (*set_gpu_state)(struct pci_dev *dev, enum vga_switcheroo_state),
void (*reprobe)(struct pci_dev *dev),
bool (*can_switch)(struct pci_dev *dev));
};
and pass the pointer to vga_switcher_register_client() and
vga_switcher_register_audio_client().
If it's preferred, I'll work on it and resend you a pull request later.
BTW, I modified topic/vga-switcheroo branch of sound git tree again.
Now it contains only two commits I posted here. The rest commits for
HD-audio are found in topic/hda-switcheroo branch.
thanks,
Takashi
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel
