Re: [cdi-devel] [PATCH] CDI.usb, mempool

[Kevin Wolf <kevin@xxxxxxxxxx>]

On Fri, Dec 11, 2009 at 07:24:42PM +1000, Matthew Iselin wrote:
> On Thu, Dec 10, 2009 at 5:50 AM,  <max@xxxxxxxxxx> wrote:
> > From: Max Reitz <max@xxxxxxxxxx>
> >
> > * Split the CDI.usb function "cdi_usb_driver_init_and_register"
> >  into an init and a register function, because a driver using the
> >  new driver registering method (via CDI_DRIVER macros) may not
> >  register itself
> > + mempool as CDI header (has been used in the old USB driver)
> >
> > Signed-off-by: Max Reitz <max@xxxxxxxxxx>
> 
> Minor nitpick: at some point you may want to document the mempool header.
> 
> Acked-by: Matthew Iselin <matthew@xxxxxxxxxxxxxx>

This is actually not a minor nitpick, but a good reason to reject a
patch right away. I'll consider the patch as an exception only because
this interface is going to be changed anyway when we introduce cdi_mem_area
as discussed in the audio/IDC thread.

Kevin