Hello. On 02-12-2010 20:27, anoop pa wrote:
-obj-$(CONFIG_IRQ_MSP_CIC) += msp_irq_cic.o +obj-$(CONFIG_IRQ_MSP_CIC) += msp_irq_cic.o msp_irq_per.o
What does this change have to do with the rest of the patch?
This change is required for next patch in this. series.Is this potentially wrong .
It's not wrong -- it's just that one patch needs to address one issue. Or at least you should describe all your changed in the changelog.
Do I want to move this to next patch?
I don't know what you want. :-)
+ set_vi_handler (MIPS_CPU_IPI_CALL_IRQ, ipi_call_dispatch);
Spaces between the function name and ( are not allowed -- run your patch thru scripts/checkpatch.pl.
Not sure what went wrong. I had checked it before sending .
linux.git$ ./scripts/checkpatch.pl 0001-VSMP-support-for-msp71xx-family-of-platforms.patch total: 0 errors, 0 warnings, 84 lines checked
0001-VSMP-support-for-msp71xx-family-of-platforms.patch has no obvious style problems and is ready for submission.
I'm not sure -- perhaps checkpatch.pl has stopped complaing about those spaces... but it ceratainly did in the past.
Regards, Anoop
WBR, Sergei
