On 06/05/2011 02:05 PM, Mauro Carvalho Chehab wrote: > This one breaks compilation: > > drivers/media/common/tuners/xc4000.c: In function ‘xc4000_set_analog_params’: > drivers/media/common/tuners/xc4000.c:1340: error: ‘type’ undeclared (first use in this function) > drivers/media/common/tuners/xc4000.c:1340: error: (Each undeclared identifier is reported only once > drivers/media/common/tuners/xc4000.c:1340: error: for each function it appears in.) > make[3]: ** [drivers/media/common/tuners/xc4000.o] Erro 1 > make[2]: ** [drivers/media/common/tuners] Erro 2 > make[1]: ** [drivers/media/common] Erro 2 > make: ** [drivers/media/] Erro 2 Yes, it depends on the 'unsigned int type = 0;' declaration that is in the xc4000_analog.patch posted later. Sorry for this mistake. Although perhaps it would have been a better idea to make all the changes to xc4000_set_analog_params() in the analog patch as well, since the function does not work correctly until that is applied. > I didn't made any review of them. Please let me know when you finish > submitting the patches for me to do a review at the resulting code. For now, I do not have more patches. I will wait for the modified code to appear on GIT, and use that as a base for further patches. > Ah, I'd appreciate if you could fix your emails. It takes me some > time to reformat the patches, as you're sending the patches as > attachments, but my email scripts aren't ready for patches with > multiple mime types. Patchwork might help, but it also got only 4 > patches from you (not sure if this is due to patchwork bugs or due to > the attachments). It also helps if you could add [PATCH] at the email > subject. I'm setting a backup process due to the constant patchwork > failures, but my alternative logic relies on having [PATCH] at the > subject logic, to move the patches into a separate mail directory. OK, I have changed the name in "from", and will send any later patches as plain text with [PATCH] in the subject. Should I re-send the previous patches in this format ? -- 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