Hi,
> > > On Tue, Oct 11, 2011 at 02:13:07PM +0530, Gupta, Ajay Kumar wrote:
> > > > Hi,
> > > > > On Mon, Oct 10, 2011 at 10:40:55AM +0530, Gupta, Ajay Kumar
> wrote:
> > > > > > Hi,
> > > > > > > On Mon, Sep 26, 2011 at 03:21:07PM +0530, Gupta, Ajay Kumar
> > > wrote:
> > > > > > > > > To: linux-omap@xxxxxxxxxxxxxxx
> > > > > > > > > Cc: linux-usb@xxxxxxxxxxxxxxx; Balbi, Felipe;
> > > > > > > > > tony@xxxxxxxxxxx; B, Ravi; Cousson, Benoit; Gupta, Ajay
> > > > > > > > > Kumar
> > > > > > > > > Subject: [PATCH 1/6 v3] omap: musb: Adding hwmod data
> > > > > > > > > for ti81xx
> > > > > > > >
> > > > > > > > Felipe,
> > > > > > > > I am planning to send patch 1/6 along with next revision
> > > > > > > > of ti81xx hwmod baseport patch from Hemant so that all
> > > > > > > > hwmod data gets reviewed at one place.
> > > > > > > >
> > > > > > > > So now only rest 5 (2/6 to 6/6) patches needs to be queued
> > > > > > > > for
> > > > > usb.
> > > > > > > >
> > > > > > > > I hope this will be fine with you. Let me know if you want
> > > > > > > > me to post Same 5 patches separately.
> > > > > > >
> > > > > > > I guess it's better to queue them separately. I will only
> > > > > > > take whatever lies in drivers/usb/musb
> > > > > >
> > > > > > Ok fine so usb hwmod patch will come from Hemant in his hwmod
> > > > > > baseport Series. I will resubmit the one patch fixing your
> > > comment.
> > > > >
> > > > > ok cool, tks.
> > > > >
> > > > > For you platform_data stuff (if you have any), you can add a
> > > > > header to include/linux/platform_data/musb-dsps.h
> > > >
> > > > I need to access "struct omap_musb_board_data" and so would add
> > > > "#include <plat/usb.h>" in new file drivers/usb/musb/musb_dsps.h.
> > > >
> > > > Any comment ?
> > >
> > > It would be nice to move that platform_data from <plat> to
> > > <linux/platform_data>
> >
> > Yes, correct. But that would need change at below files.
> >
> > arch/arm/mach-omap2/board-4430sdp.c:static struct omap_musb_board_data
> > musb_board_data = { arch/arm/mach-omap2/board-am3517evm.c:static
> > struct omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-omap3evm.c:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-rx51.c:static struct omap_musb_board_data
> > musb_board_data = { arch/arm/mach-omap2/board-omap4panda.c:static
> > struct omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-ti8168evm.c:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-omap4pcm049.c:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/usb-musb.c:static void usb_musb_mux_init(struct
> > omap_musb_board_data *board_data)
> > arch/arm/mach-omap2/board-ti8148evm.c:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-am335xevm.c:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-ti8148evm.c.orig:static struct
> > omap_musb_board_data musb_board_data = {
> > arch/arm/mach-omap2/board-omap3evm.c.orig:static struct
> > omap_musb_board_data musb_board_data = {
> > drivers/usb/musb/am35x.c:
> > drivers/usb/musb/musb_dsps.c
> > drivers/usb/musb/omap2430.c:
> >
> > I think this can be done in a separate patch and let the ti81xx 6/6
> > Patch to remain same.
> >
> > The separate patch will replace "#include <plat/usb.h>" to "#include
> > <platform_data/usb.h>"
>
> cool, that can be done with a sed script ;-)
Ok fine then I will send the refreshed 5 patch set for ti81xx and 3 patch set
for am335x correcting your comments.
Thanks,
Ajay
>
> --
> balbi
--
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
