Re: [PATCH] cx18: Fix warnings introduced during cleanup

2011-05-05 Thread Mauro Carvalho Chehab
Em 05-05-2011 09:42, Simon Farnsworth escreveu:
 I misused the ktime API, and failed to remove some traces of the
 in-kernel format conversion. Fix these, so the the driver builds
 without warnings.
 
 Signed-off-by: Simon Farnsworth simon.farnswo...@onelan.co.uk
 ---
 Mauro,
 
 You may want to squash this in with the cleanup patch itself - it's
 plain and simple oversight on my part (I should have seen the compiler
 warnings), and I should not have sent the cleanup patch to you without
 fixing these errors.
 
It will all depend on how much time I'll have during the next merge window.
I can't do it at the already-applied patches, as other people use it as basis,
and a rebase there would break its clones.

Mauro.
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] cx18: Fix warnings introduced during cleanup

2011-05-05 Thread Simon Farnsworth
On Thursday 5 May 2011, Mauro Carvalho Chehab mche...@infradead.org wrote:
 Em 05-05-2011 09:42, Simon Farnsworth escreveu:
  I misused the ktime API, and failed to remove some traces of the
  in-kernel format conversion. Fix these, so the the driver builds
  without warnings.
  
  Signed-off-by: Simon Farnsworth simon.farnswo...@onelan.co.uk
  ---
  Mauro,
  
  You may want to squash this in with the cleanup patch itself - it's
  plain and simple oversight on my part (I should have seen the compiler
  warnings), and I should not have sent the cleanup patch to you without
  fixing these errors.
 
 It will all depend on how much time I'll have during the next merge window.
 I can't do it at the already-applied patches, as other people use it as
 basis, and a rebase there would break its clones.
 
 Mauro.

No problem; if it ends up as a separate patch in the tree, it's not going to 
hurt, as the version with warnings functions adequately enough to not break 
bisection.
-- 
Simon Farnsworth
Software Engineer
ONELAN Limited
http://www.onelan.com/
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html