driver version 0.1

PATCH New Omar Ramirez Luna 4 participants 9 messages Jan 11, 2010 – Jan 13, 2010

Share & tools

Omar Ramirez Luna

author

2010-01-11 17:41 UTC #0 lore raw

PATCH 1/1 driver version 0.1

Given the on going efforts to review bridge patches, a branch has been published with all of the comments received a few weeks ago, the current dspbridge which is meant to be 0.1 can be found at:

Branch: dspbridge @
http://dev.omapzoom.org/?p=tidspbridge/kernel-dspbridge.git;a=shortlog;h=refs/heads/dspbridge

Special thanks to:
- Felipe Contreras: for his comments and time during this sync up.
- Hiroshi Doyu and Ameya Palande for their time reviewing patches.
- And all who also have provided their comments.

Omar Ramirez Luna (1):
DSPBRIDGE: driver version 0.1

 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

CC Felipe Contreras <[email protected]>

CC Hiroshi Doyu <[email protected]>

CC Ameya Palande <[email protected]>

CC Nishant Menon <[email protected]>

Omar Ramirez Luna

author

2010-01-11 17:40 UTC #1 lore raw

PATCH 1/1 DSPBRIDGE: driver version 0.1

This is the official 0.1 dspbridge version.

CC Felipe Contreras <[email protected]>

CC Hiroshi Doyu <[email protected]>

CC Ameya Palande <[email protected]>

CC Nishant Menon <[email protected]>

Signed-off-by Omar Ramirez Luna <[email protected]>

---
 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

drivers/dsp/bridge/rmgr/drv_interface.c +2 -0

index c853854..efce35d 100644
@@ -101,6 +101,7 @@
 #define BRIDGE_NAME "C6410"
 /*  ----------------------------------- Globals */
 #define DRIVER_NAME  "DspBridge"
+#define DSPBRIDGE_VERSION	"0.1"
 s32 dsp_debug;
 
 struct platform_device *omap_dspbridge_dev;

@@ -177,6 +178,7 @@ MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0");
 
 MODULE_AUTHOR("Texas Instruments");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DSPBRIDGE_VERSION);
 
 static char *driver_name = DRIVER_NAME;

Signature · 1 line

1.6.2.4

Nishanth Menon

2010-01-11 18:30 UTC #2 lore raw

Re: [PATCH] DSPBRIDGE: driver version 0.1

Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:

Quoted 25 lines

This is the official 0.1 dspbridge version.
CC Felipe Contreras <[email protected]>
CC Hiroshi Doyu <[email protected]>
CC Ameya Palande <[email protected]>
CC Nishant Menon <[email protected]>
Signed-off-by Omar Ramirez Luna <[email protected]>
---
 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/dsp/bridge/rmgr/drv_interface.c +1 -0
index c853854..efce35d 100644
@@ -101,6 +101,7 @@
 #define BRIDGE_NAME "C6410"
 /*  ----------------------------------- Globals */
 #define DRIVER_NAME  "DspBridge"
+#define DSPBRIDGE_VERSION	"0.1"
 s32 dsp_debug;
 
 struct platform_device *omap_dspbridge_dev;

drivers/dsp/bridge/rmgr/drv_interface.c MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0"); · line 184

Quoted 7 lines

drivers/dsp/bridge/rmgr/drv_interface.c +1 -0

@@ -177,6 +178,7 @@ MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0");
 
 MODULE_AUTHOR("Texas Instruments");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DSPBRIDGE_VERSION);
 
 static char *driver_name = DRIVER_NAME;

is the IOCTL patch before or after the 0.1?

Signature · 2 lines

Regards,
Nishanth Menon

Ramirez Luna, Omar

author

2010-01-11 18:46 UTC #3 lore raw

RE: [PATCH] DSPBRIDGE: driver version 0.1

Quoted 29 lines

From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:
Quoted 26 lines
This is the official 0.1 dspbridge version.
CC Felipe Contreras <[email protected]>
CC Hiroshi Doyu <[email protected]>
CC Ameya Palande <[email protected]>
CC Nishant Menon <[email protected]>
Signed-off-by Omar Ramirez Luna <[email protected]>
---
 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/dsp/bridge/rmgr/drv_interface.c +1 -0
index c853854..efce35d 100644
@@ -101,6 +101,7 @@
 #define BRIDGE_NAME "C6410"
 /*  ----------------------------------- Globals */
 #define DRIVER_NAME  "DspBridge"
+#define DSPBRIDGE_VERSION	"0.1"
 s32 dsp_debug;
 
 struct platform_device *omap_dspbridge_dev;
@@ -177,6 +178,7 @@ MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0");
 

Quoted 7 lines

drivers/dsp/bridge/rmgr/drv_interface.c MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0"); · line 184
Quoted 6 lines
drivers/dsp/bridge/rmgr/drv_interface.c +1 -0
 MODULE_AUTHOR("Texas Instruments");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DSPBRIDGE_VERSION);
 
 static char *driver_name = DRIVER_NAME;
 
is the IOCTL patch before or after the 0.1?

IOCTL patch is after 0.1

I'll be resending v2 of IOCTL patch but should be applied after this patch.

- omar

Nishanth Menon

2010-01-11 18:55 UTC #4 lore raw

Re: [PATCH] DSPBRIDGE: driver version 0.1

Ramirez Luna, Omar had written, on 01/11/2010 12:46 PM, the following:

Quoted 33 lines

Quoted 33 lines
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:
Quoted 30 lines
This is the official 0.1 dspbridge version.
CC Felipe Contreras <[email protected]>
CC Hiroshi Doyu <[email protected]>
CC Ameya Palande <[email protected]>
CC Nishant Menon <[email protected]>
Signed-off-by Omar Ramirez Luna <[email protected]>
---
 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/dsp/bridge/rmgr/drv_interface.c +2 -0
index c853854..efce35d 100644
@@ -101,6 +101,7 @@
 #define BRIDGE_NAME "C6410"
 /*  ----------------------------------- Globals */
 #define DRIVER_NAME  "DspBridge"
+#define DSPBRIDGE_VERSION	"0.1"
 s32 dsp_debug;
 
 struct platform_device *omap_dspbridge_dev;
@@ -177,6 +178,7 @@ MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0");
 
 MODULE_AUTHOR("Texas Instruments");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DSPBRIDGE_VERSION);
 

Quoted 7 lines

Quoted 3 lines
drivers/dsp/bridge/rmgr/drv_interface.c MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0"); · line 184
Quoted 2 lines
drivers/dsp/bridge/rmgr/drv_interface.c +0 -0
 static char *driver_name = DRIVER_NAME;
 
is the IOCTL patch before or after the 0.1?
IOCTL patch is after 0.1
I'll be resending v2 of IOCTL patch but should be applied after this patch.

sounds reasonable. I wonder what happened to
http://marc.info/?l=linux-omap&m=125729526105560&w=2

Signature · 2 lines

Regards,
Nishanth Menon

Ramirez Luna, Omar

author

2010-01-11 19:28 UTC #5 lore raw

RE: [PATCH] DSPBRIDGE: driver version 0.1

Quoted 40 lines

From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 12:46 PM, the following:
Quoted 37 lines
Quoted 36 lines
From: Menon, Nishanth
Ramirez Luna, Omar had written, on 01/11/2010 11:50 AM, the following:
Quoted 25 lines
This is the official 0.1 dspbridge version.
CC Felipe Contreras <[email protected]>
CC Hiroshi Doyu <[email protected]>
CC Ameya Palande <[email protected]>
CC Nishant Menon <[email protected]>
Signed-off-by Omar Ramirez Luna <[email protected]>
---
 drivers/dsp/bridge/rmgr/drv_interface.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)
drivers/dsp/bridge/rmgr/drv_interface.c +1 -0
index c853854..efce35d 100644
@@ -101,6 +101,7 @@
 #define BRIDGE_NAME "C6410"
 /*  ----------------------------------- Globals */
 #define DRIVER_NAME  "DspBridge"
+#define DSPBRIDGE_VERSION	"0.1"
 s32 dsp_debug;
 
 struct platform_device *omap_dspbridge_dev;
drivers/dsp/bridge/rmgr/drv_interface.c MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0"); · line 184
Quoted 7 lines
drivers/dsp/bridge/rmgr/drv_interface.c +1 -0
@@ -177,6 +178,7 @@ MODULE_PARM_DESC(tc_wordswapon, "TC Word Swap Option. default = 0");
 
 MODULE_AUTHOR("Texas Instruments");
 MODULE_LICENSE("GPL");
+MODULE_VERSION(DSPBRIDGE_VERSION);
 
 static char *driver_name = DRIVER_NAME;
 
is the IOCTL patch before or after the 0.1?

Quoted 5 lines

Quoted 3 lines
IOCTL patch is after 0.1
I'll be resending v2 of IOCTL patch but should be applied after this patch.
sounds reasonable. I wonder what happened to
http://marc.info/?l=linux-omap&m=125729526105560&w=2

To the change: currently is on hold given the amount of changes but scheduled for this month (hopefully ;))

To the request: I have a patch queue to be applied after 0.1 patch and v2 of ioctl, the patches have already been sent to this mailing list but changes didn't fit prior to 0.1 alignment

Regards,

omar

Weng, Wending

2010-01-11 19:03 UTC #6 lore raw

PM: Device usb1 failed to suspend: error -16

In reply to Nishanth Menon

Hi All,

I tried to suspend the system, and got the following messages:

PM: Syncing filesystems ... done.
Freezing user space processes ... (elapsed 0.00 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. Suspending console(s) (use no_console_suspend to debug) musb_bus_suspend 2261: trying to suspend as a_host while active pm_op(): usb_dev_suspend+0x0/0x8 returns -16
PM: Device usb1 failed to suspend: error -16
PM: Some devices failed to suspend
Restarting tasks ... Segmentation fault
done.

I use linux-omap-2.6.git, should I use the branch linux-omap-pm? Thanks.

Wending Weng
Software Engineer at Rheinmetall

Gupta, Ajay Kumar

2010-01-12 11:40 UTC #7 lore raw

RE: Device usb1 failed to suspend: error -16

Quoted 11 lines

-----Original Message-----
From: [email protected] [mailto:linux-omap- [email protected]] On Behalf Of Weng, Wending
Sent: Tuesday, January 12, 2010 12:34 AM
To: linux-omap
Subject: PM: Device usb1 failed to suspend: error -16
Hi All,
I tried to suspend the system, and got the following messages:

Quoted 8 lines

PM: Syncing filesystems ... done.
Freezing user space processes ... (elapsed 0.00 seconds) done. Freezing remaining freezable tasks ... (elapsed 0.00 seconds) done. Suspending console(s) (use no_console_suspend to debug) musb_bus_suspend 2261: trying to suspend as a_host while active pm_op(): usb_dev_suspend+0x0/0x8 returns -16
PM: Device usb1 failed to suspend: error -16
PM: Some devices failed to suspend

You need to apply MUSB and EHCI patches to fix this.

MUSB PM patch is already submitted (at [1]) and is in queue. I plan to submit OMAP EHCI PM patch after some cleanups.

[1] http://marc.info/?l=linux-usb&m=126200057420865&w=2

-Ajay

Quoted 11 lines

Restarting tasks ... Segmentation fault
done.
I use linux-omap-2.6.git, should I use the branch linux-omap-pm? Thanks.
Wending Weng
Software Engineer at Rheinmetall
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

Ramirez Luna, Omar

author Applied

2010-01-13 23:55 UTC #8 lore raw

PATCH 1/1 DSPBRIDGE: driver version 0.1

In reply to Omar Ramirez Luna

Quoted 4 lines

From: Ramirez Luna, Omar
This is the official 0.1 dspbridge version.

Quoted 9 lines

Signed-off-by Omar Ramirez Luna <[email protected]>
CC Felipe Contreras <[email protected]>
CC Hiroshi Doyu <[email protected]>
CC Ameya Palande <[email protected]>
CC Nishant Menon <[email protected]>
---
drivers/dsp/bridge/rmgr/drv_interface.c | 2 ++
1 files changed, 2 insertions(+), 0 deletions(-)

Pushed to dspbridge

- omar