On Mon, 28 Apr 2008 17:11:51 +0530, "Madhusudhan Chikkature Rajashekar" <madhu.cr@xxxxxx> wrote: > Hi Felipe, > > Thanks for the comments. I will fix them and resend the patch. > > Please note my view on the below point. > >> > - pdev = platform_device_alloc("omap-bq2700-battery", id); >> > + pdev = platform_device_alloc("bq27000-bat", id); >> >> do you really need to change the name here? >> It seems that this change doesn't belong to this patch. > > I will make this as a separate patch.As BQ27000 chip is not omap specific, > I guess the above name is not correct. Hence I intend to make that change. Ok, works for me :-) I guess bq2700-battery is ok, just remove omap- from the previous name?? -- Best Regards, Felipe Balbi http://felipebalbi.com me@xxxxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html