Re: [PATCH 0/4 v6] i.MX31: dmaengine and framebuffer drivers

From: Guennadi Liakhovetski
Date: Fri Jan 16 2009 - 05:10:17 EST


On Mon, 5 Jan 2009, Dan Williams wrote:

> On Fri, Dec 26, 2008 at 10:11 AM, Guennadi Liakhovetski
> <g.liakhovetski@xxxxxx> wrote:
> > Hi,
> >
> > This is version 6 of dmaengine and framebuffer drivers for i.MX31.
> >
>
> Tha drivers/dma/ bits look ok to me minus the new warnings:
>
> drivers/dma/ipu/ipu_irq.c: In function 'ipu_irq_fn':
> drivers/dma/ipu/ipu_irq.c:328: warning: 'irq' may be used
> uninitialized in this function
> drivers/dma/ipu/ipu_irq.c: In function 'ipu_irq_err':
> drivers/dma/ipu/ipu_irq.c:290: warning: 'irq' may be used
> uninitialized in this function
>
> I can take patches 1 and 2 through the async_tx tree, or if you would
> rather, add my Acked-by to those and take 1-4 through the MXC tree.
> I'll be sending the async_tx pull request for 2.6.29 in the next few
> days.

Ok, let's do it this way: Sascha has acked version 8 of patches 2 and 3.
And there seems to be no active framebuffer maintainer ATM. So, maybe you
could take patches 1-3 into your tree, so they are exposed to the public,
people have easier access to them in case they want to test (I had several
requests), and you update them in case you change async-tx API again:-) It
would be good to try and push them in for 2.6.29 - these are new drivers,
so, might still be possible to convince Andrew / Linus, if not, you could
keep them in your tree (and in next) until 2.6.30, then we'd add platform
bindings and the camera driver. What do you think?

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/