On Wed, Jan 28, 2015 at 02:46:46PM +0800, Daniel Kurtz wrote: > Hi Dan, > > nit: this patch is not really for "drm/bridge". > > On Wed, Jan 28, 2015 at 2:43 PM, Dan Carpenter wrote: > > > > We were supposed to check "fmts" here instead of "formats". I suppose > > it eventually leads to a NULL dereference. > > > > Signed-off-by: Dan Carpenter > > > > Otherwise, good catch! > > Reviewed-by: Daniel Kurtz I've reworded the commit message slightly and applied this. Thanks. Thierry