RE: [PATCH 1/6 v3] omap: musb: Adding hwmod data for ti81xx

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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>"

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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux