RE: [PATCH v4 0/2] Mem-to-mem device framework

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Pawel Osciak [mailto:p.osciak@xxxxxxxxxxx]
> Sent: Monday, April 19, 2010 6:00 PM
> To: linux-media@xxxxxxxxxxxxxxx
> Cc: p.osciak@xxxxxxxxxxx; m.szyprowski@xxxxxxxxxxx;
> kyungmin.park@xxxxxxxxxxx; Hiremath, Vaibhav
> Subject: [PATCH v4 0/2] Mem-to-mem device framework
> 
> Hello,
> 
> this is the fourth version of the mem-to-mem device framework.
> 
> Changes in v4:
> - v4l2_m2m_poll() now also reports POLLOUT | POLLWRNORM when an output
>   buffer is ready to be dequeued
> - more cleaning up, addressing most of the comments to v3
> 
> Vaibhav: your clean-up patch didn't apply after my changes. I incorporated
> most
> of your clean-up changes. If you prefer it to be separate, we will have
> to prepare another one somehow. 
[Hiremath, Vaibhav] No need to create separate patch for this, it's ok as long as you included all the required changes.

You can add "Tested-By" Or "Reviewed-By" in your patch series, that should be ok.

I will take a final look to this patch and respond.

> Also, sorry, but I cannot agree with
> changing
> unsigned types into u32, I do not see any reason to use fixed-width types
> there.
> 
[Hiremath, Vaibhav] As I mentioned there no strict rule for this, it was learning from my first patch.

Thanks,
Vaibhav
> This series contains:
> [PATCH v4 1/2] v4l: Add memory-to-memory device helper framework for
> videobuf.
> [PATCH v4 2/2] v4l: Add a mem-to-mem videobuf framework test device.
> 
> Best regards
> --
> Pawel Osciak
> Linux Platform Group
> Samsung Poland R&D Center
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux