Hi, Jonathan >-----Original Message----- >From: Jonathan Corbet [mailto:corbet@xxxxxxx] >Sent: Monday, 17 December, 2012 00:19 >To: Albert Wang >Cc: g.liakhovetski@xxxxxx; linux-media@xxxxxxxxxxxxxxx; Libin Yang >Subject: Re: [PATCH V3 07/15] [media] marvell-ccic: add SOF / EOF pair check for >marvell-ccic driver > >On Sat, 15 Dec 2012 17:57:56 +0800 >Albert Wang <twang13@xxxxxxxxxxx> wrote: > >> From: Libin Yang <lbyang@xxxxxxxxxxx> >> >> This patch adds the SOFx/EOFx pair check for marvell-ccic. >> >> When switching format, the last EOF may not arrive when stop streamning. >> And the EOF will be detected in the next start streaming. >> >> Must ensure clear the obsolete frame flags before every really start streaming. > >"obsolete" doesn't quite read right; it suggests that the flags only >apply to older hardware. I'd suggest "left over" or some such (in the >code comment too). Otherwise seems fine. > [Albert Wang] OK, we will change the "bad" word. :) >Acked-by: Jonathan Corbet <corbet@xxxxxxx> > >jon Thanks Albert Wang 86-21-61092656 -- 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
