Re: [PATCH 0/6] DVB-T2 API updates, documentation and accompanying small fixes

[Mauro Carvalho Chehab <mchehab@redhat.com>]

Em 08-05-2011 12:51, Steve Kerrison escreveu:
> Hi Mauro, Antti, Andreas,
> 
> I hope this patch set is formed appropriately - it is my first patch
> submission direct to the linux-media group.

They look sane on my eyes. I'll comment patch by patch were needed. Non-commented
patches mean that they're ok.

PS.: as I'm traveling abroad this week, I may eventually not apply the patch series this
week.
> 
> Following the pull of Antti's work on support for the cxd2820r and PCTV
> nanoStick T2 290e, this patch set implements Andreas' modifications to the API
> to give provisional DVB-T2 support and the removal of a workaround for this
> in the cxd2820r module.
> 
> In addition, there are some minor fixes to compiler warnings as a result
> of the expanded enums. I cannot test these myself but they treat unrecognized
> values as *_AUTO and I can't see where a problem would be created.
> 
> I have updated the documentation a little. If I've done the right thing then
> I guess there is incentive there for me continue to expand DVB related
> elements of the API docs.
> 
> This patch set has been tested by me on two systems, with one running a MythTV
> backend utilising a long-supported DVB tuner. MythTV works fine with the old
> tuner and the nanoStick T2 290e works in VLC. I've yet to test the 290e in
> MythTV - I was more intent on making sure the patches hadn't broken userland
> or older devices.
> 
> Feedback, testing  and discussion of where to go next is welcomed!
> 
> Regards,
> Steve Kerrison.
>