* Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-07 23:30 Chris Bergeron
2007-05-08 0:25 ` Satyam Sharma
0 siblings, 1 reply; 16+ messages in thread
From: Chris Bergeron @ 2007-05-07 23:30 UTC (permalink / raw)
To: linux-kernel
On trying to build a 2.6.21.1 kernel using the configuration at
http://pcburn.com/files/kernel/breaks.2.6.21.1.config I'm getting the
following error with Networking Support -> Bluetooth Subsystem Support
-> HIDP protocol support set to built-in:
CC init/version.o
LD init/built-in.o
LD .tmp_vmlinux1
net/built-in.o: In function `hidp_add_connection':
(.text+0x8bb08): undefined reference to `hid_ff_init'
make: *** [.tmp_vmlinux1] Error 1
Works with that option turned off. Also, the GCC version I'm using is
the current Debian Etch release: gcc (GCC) 4.1.2 20061115 (prerelease)
(Debian 4.1.1-21). I also tried it with gcc (GCC) 3.4.6 (Debian
3.4.6-5) for the same result.
I don't need to build that, so I simply unset it. If this is at all
useful and you need something else just CC: me in the reply.
Thanks,
-- Chris
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-07 23:30 Getting make net/built-in.o Error with 2.6.21.1 Build Chris Bergeron
@ 2007-05-08 0:25 ` Satyam Sharma
0 siblings, 0 replies; 16+ messages in thread
From: Satyam Sharma @ 2007-05-08 0:25 UTC (permalink / raw)
To: chris; +Cc: linux-kernel, marcel, bluez-devel, netdev
On 5/8/07, Chris Bergeron <chris@pcburn.com> wrote:
> On trying to build a 2.6.21.1 kernel using the configuration at
> http://pcburn.com/files/kernel/breaks.2.6.21.1.config I'm getting the
> following error with Networking Support -> Bluetooth Subsystem Support
> -> HIDP protocol support set to built-in:
>
> CC init/version.o
> LD init/built-in.o
> LD .tmp_vmlinux1
> net/built-in.o: In function `hidp_add_connection':
> (.text+0x8bb08): undefined reference to `hid_ff_init'
> make: *** [.tmp_vmlinux1] Error 1
You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
patch below solves it. To test, you can just "make oldconfig" on the
same .config that broke above and then proceed to make bzImage.
Satyam
---
Make CONFIG_BT_HIDP depend on CONFIG_USB_HID and CONFIG_HID_FF.
Signed-off-by: Satyam Sharma <ssatyam@cse.iitk.ac.in>
---
net/bluetooth/hidp/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff -ruNp a/net/bluetooth/hidp/Kconfig b/net/bluetooth/hidp/Kconfig
--- a/net/bluetooth/hidp/Kconfig 2007-04-26 08:38:32.000000000 +0530
+++ b/net/bluetooth/hidp/Kconfig 2007-05-08 05:40:58.000000000 +0530
@@ -1,6 +1,6 @@
config BT_HIDP
tristate "HIDP protocol support"
- depends on BT && BT_L2CAP && INPUT
+ depends on BT && BT_L2CAP && INPUT && USB_HID && HID_FF
select HID
help
HIDP (Human Interface Device Protocol) is a transport layer
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 0:25 ` Satyam Sharma
0 siblings, 0 replies; 16+ messages in thread
From: Satyam Sharma @ 2007-05-08 0:25 UTC (permalink / raw)
To: chris; +Cc: netdev, marcel, linux-kernel, bluez-devel
On 5/8/07, Chris Bergeron <chris@pcburn.com> wrote:
> On trying to build a 2.6.21.1 kernel using the configuration at
> http://pcburn.com/files/kernel/breaks.2.6.21.1.config I'm getting the
> following error with Networking Support -> Bluetooth Subsystem Support
> -> HIDP protocol support set to built-in:
>
> CC init/version.o
> LD init/built-in.o
> LD .tmp_vmlinux1
> net/built-in.o: In function `hidp_add_connection':
> (.text+0x8bb08): undefined reference to `hid_ff_init'
> make: *** [.tmp_vmlinux1] Error 1
You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
patch below solves it. To test, you can just "make oldconfig" on the
same .config that broke above and then proceed to make bzImage.
Satyam
---
Make CONFIG_BT_HIDP depend on CONFIG_USB_HID and CONFIG_HID_FF.
Signed-off-by: Satyam Sharma <ssatyam@cse.iitk.ac.in>
---
net/bluetooth/hidp/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
---
diff -ruNp a/net/bluetooth/hidp/Kconfig b/net/bluetooth/hidp/Kconfig
--- a/net/bluetooth/hidp/Kconfig 2007-04-26 08:38:32.000000000 +0530
+++ b/net/bluetooth/hidp/Kconfig 2007-05-08 05:40:58.000000000 +0530
@@ -1,6 +1,6 @@
config BT_HIDP
tristate "HIDP protocol support"
- depends on BT && BT_L2CAP && INPUT
+ depends on BT && BT_L2CAP && INPUT && USB_HID && HID_FF
select HID
help
HIDP (Human Interface Device Protocol) is a transport layer
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 0:25 ` Satyam Sharma
@ 2007-05-08 15:49 ` Jiri Kosina
-1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 15:49 UTC (permalink / raw)
To: Satyam Sharma; +Cc: chris, linux-kernel, Marcel Holtmann, bluez-devel, netdev
On Tue, 8 May 2007, Satyam Sharma wrote:
> > CC init/version.o
> > LD init/built-in.o
> > LD .tmp_vmlinux1
> > net/built-in.o: In function `hidp_add_connection':
> > (.text+0x8bb08): undefined reference to `hid_ff_init'
> > make: *** [.tmp_vmlinux1] Error 1
> You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
> patch below solves it. To test, you can just "make oldconfig" on the
> same .config that broke above and then proceed to make bzImage.
I'd say we need a different fix here. First, the question is whether there
are any force-feedback devices, supported by USBHID force-feedback layer,
which have a bluetooth version?
If there are none of them, we could just drop the FF initialization
completely for now.
On the other hand if there are such devices, just calling hid_ff_init() is
presently not enough anyway, as the force feedback drivers for HID devices
are currently USB-transport specific.
--
Jiri Kosina
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 15:49 ` Jiri Kosina
0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 15:49 UTC (permalink / raw)
To: Satyam Sharma; +Cc: bluez-devel, chris, Marcel Holtmann, linux-kernel, netdev
On Tue, 8 May 2007, Satyam Sharma wrote:
> > CC init/version.o
> > LD init/built-in.o
> > LD .tmp_vmlinux1
> > net/built-in.o: In function `hidp_add_connection':
> > (.text+0x8bb08): undefined reference to `hid_ff_init'
> > make: *** [.tmp_vmlinux1] Error 1
> You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
> patch below solves it. To test, you can just "make oldconfig" on the
> same .config that broke above and then proceed to make bzImage.
I'd say we need a different fix here. First, the question is whether there
are any force-feedback devices, supported by USBHID force-feedback layer,
which have a bluetooth version?
If there are none of them, we could just drop the FF initialization
completely for now.
On the other hand if there are such devices, just calling hid_ff_init() is
presently not enough anyway, as the force feedback drivers for HID devices
are currently USB-transport specific.
--
Jiri Kosina
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 15:49 ` Jiri Kosina
@ 2007-05-08 16:05 ` Satyam Sharma
-1 siblings, 0 replies; 16+ messages in thread
From: Satyam Sharma @ 2007-05-08 16:05 UTC (permalink / raw)
To: Jiri Kosina; +Cc: chris, linux-kernel, Marcel Holtmann, bluez-devel, netdev
Hi Jiri,
On 5/8/07, Jiri Kosina <jikos@jikos.cz> wrote:
> On Tue, 8 May 2007, Satyam Sharma wrote:
>
> > > CC init/version.o
> > > LD init/built-in.o
> > > LD .tmp_vmlinux1
> > > net/built-in.o: In function `hidp_add_connection':
> > > (.text+0x8bb08): undefined reference to `hid_ff_init'
> > > make: *** [.tmp_vmlinux1] Error 1
> > You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
> > patch below solves it. To test, you can just "make oldconfig" on the
> > same .config that broke above and then proceed to make bzImage.
>
> I'd say we need a different fix here. First, the question is whether there
> are any force-feedback devices, supported by USBHID force-feedback layer,
> which have a bluetooth version?
>
> If there are none of them, we could just drop the FF initialization
> completely for now.
>
> On the other hand if there are such devices, just calling hid_ff_init() is
> presently not enough anyway, as the force feedback drivers for HID devices
> are currently USB-transport specific.
Sure, my aim here was to only solve the _build breakage_ by fixing the
Kconfig for this module (that used code from another kernel module
without listing it in its dependencies). If, as you say, the real
solution is that we should actually be taking out the offending call
to the other module itself, then please go ahead -- I don't know much
about the Bluetooth / HIDP subsytem anyway.
Thanks,
Satyam
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 16:05 ` Satyam Sharma
0 siblings, 0 replies; 16+ messages in thread
From: Satyam Sharma @ 2007-05-08 16:05 UTC (permalink / raw)
To: Jiri Kosina; +Cc: bluez-devel, chris, Marcel Holtmann, linux-kernel, netdev
Hi Jiri,
On 5/8/07, Jiri Kosina <jikos@jikos.cz> wrote:
> On Tue, 8 May 2007, Satyam Sharma wrote:
>
> > > CC init/version.o
> > > LD init/built-in.o
> > > LD .tmp_vmlinux1
> > > net/built-in.o: In function `hidp_add_connection':
> > > (.text+0x8bb08): undefined reference to `hid_ff_init'
> > > make: *** [.tmp_vmlinux1] Error 1
> > You've got CONFIG_BT_HIDP=y there but CONFIG_USB_HID=m. The trivial
> > patch below solves it. To test, you can just "make oldconfig" on the
> > same .config that broke above and then proceed to make bzImage.
>
> I'd say we need a different fix here. First, the question is whether there
> are any force-feedback devices, supported by USBHID force-feedback layer,
> which have a bluetooth version?
>
> If there are none of them, we could just drop the FF initialization
> completely for now.
>
> On the other hand if there are such devices, just calling hid_ff_init() is
> presently not enough anyway, as the force feedback drivers for HID devices
> are currently USB-transport specific.
Sure, my aim here was to only solve the _build breakage_ by fixing the
Kconfig for this module (that used code from another kernel module
without listing it in its dependencies). If, as you say, the real
solution is that we should actually be taking out the offending call
to the other module itself, then please go ahead -- I don't know much
about the Bluetooth / HIDP subsytem anyway.
Thanks,
Satyam
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 16:05 ` Satyam Sharma
@ 2007-05-08 16:23 ` Jiri Kosina
-1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 16:23 UTC (permalink / raw)
To: Satyam Sharma, Marcel Holtmann; +Cc: chris, linux-kernel, bluez-devel, netdev
On Tue, 8 May 2007, Satyam Sharma wrote:
> Sure, my aim here was to only solve the _build breakage_ by fixing the
> Kconfig for this module (that used code from another kernel module
> without listing it in its dependencies). If, as you say, the real
> solution is that we should actually be taking out the offending call to
> the other module itself, then please go ahead -- I don't know much about
> the Bluetooth / HIDP subsytem anyway.
Converting the hid-ff drivers to be also transport-independent is on my
TODO list, but it didn't happen yet.
Marcel - are you aware of any devices currently supported by USB HID
force-feedback code, which have a bluetooth version, please?
I'd propose the patch below, until I make the usbhid force-feedback code
transport independent. Thanks.
From: Jiri Kosina <jkosina@suse.cz>
[Bluetooth] HIDP - don't initialize force feedback
The current implementation of force feedback for HID devices is
USB-transport only and therefore calling hid_ff_init() from hidp code is
not going to work (plus it creates unwanted dependency of hidp on usbhid).
Remove the hid_ff_init() until either the hid-ff is made
transport-independent, or at least support for bluetooth transport is
added.
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c
index d342e89..3e77e81 100644
--- a/net/bluetooth/hidp/core.c
+++ b/net/bluetooth/hidp/core.c
@@ -737,10 +737,8 @@ static inline void hidp_setup_hid(struct
list_for_each_entry(report, &hid->report_enum[HID_FEATURE_REPORT].report_list, list)
hidp_send_report(session, report);
- if (hidinput_connect(hid) == 0) {
+ if (hidinput_connect(hid) == 0)
hid->claimed |= HID_CLAIMED_INPUT;
- hid_ff_init(hid);
- }
}
int hidp_add_connection(struct hidp_connadd_req *req, struct socket *ctrl_sock, struct socket *intr_sock)
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 16:23 ` Jiri Kosina
0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 16:23 UTC (permalink / raw)
To: Satyam Sharma, Marcel Holtmann; +Cc: netdev, chris, bluez-devel, linux-kernel
On Tue, 8 May 2007, Satyam Sharma wrote:
> Sure, my aim here was to only solve the _build breakage_ by fixing the
> Kconfig for this module (that used code from another kernel module
> without listing it in its dependencies). If, as you say, the real
> solution is that we should actually be taking out the offending call to
> the other module itself, then please go ahead -- I don't know much about
> the Bluetooth / HIDP subsytem anyway.
Converting the hid-ff drivers to be also transport-independent is on my
TODO list, but it didn't happen yet.
Marcel - are you aware of any devices currently supported by USB HID
force-feedback code, which have a bluetooth version, please?
I'd propose the patch below, until I make the usbhid force-feedback code
transport independent. Thanks.
From: Jiri Kosina <jkosina@suse.cz>
[Bluetooth] HIDP - don't initialize force feedback
The current implementation of force feedback for HID devices is
USB-transport only and therefore calling hid_ff_init() from hidp code is
not going to work (plus it creates unwanted dependency of hidp on usbhid).
Remove the hid_ff_init() until either the hid-ff is made
transport-independent, or at least support for bluetooth transport is
added.
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c
index d342e89..3e77e81 100644
--- a/net/bluetooth/hidp/core.c
+++ b/net/bluetooth/hidp/core.c
@@ -737,10 +737,8 @@ static inline void hidp_setup_hid(struct
list_for_each_entry(report, &hid->report_enum[HID_FEATURE_REPORT].report_list, list)
hidp_send_report(session, report);
- if (hidinput_connect(hid) == 0) {
+ if (hidinput_connect(hid) == 0)
hid->claimed |= HID_CLAIMED_INPUT;
- hid_ff_init(hid);
- }
}
int hidp_add_connection(struct hidp_connadd_req *req, struct socket *ctrl_sock, struct socket *intr_sock)
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply related [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 16:23 ` Jiri Kosina
@ 2007-05-08 16:48 ` Marcel Holtmann
-1 siblings, 0 replies; 16+ messages in thread
From: Marcel Holtmann @ 2007-05-08 16:48 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Satyam Sharma, chris, linux-kernel, bluez-devel, netdev
Hi Jiri,
> > Sure, my aim here was to only solve the _build breakage_ by fixing the
> > Kconfig for this module (that used code from another kernel module
> > without listing it in its dependencies). If, as you say, the real
> > solution is that we should actually be taking out the offending call to
> > the other module itself, then please go ahead -- I don't know much about
> > the Bluetooth / HIDP subsytem anyway.
>
> Converting the hid-ff drivers to be also transport-independent is on my
> TODO list, but it didn't happen yet.
>
> Marcel - are you aware of any devices currently supported by USB HID
> force-feedback code, which have a bluetooth version, please?
I haven't looked at all details for the PS3 controller, but that might
be the first one. In theory they can and at some point they will enter
the market.
> I'd propose the patch below, until I make the usbhid force-feedback code
> transport independent. Thanks.
>
>
>
> From: Jiri Kosina <jkosina@suse.cz>
>
> [Bluetooth] HIDP - don't initialize force feedback
>
> The current implementation of force feedback for HID devices is
> USB-transport only and therefore calling hid_ff_init() from hidp code is
> not going to work (plus it creates unwanted dependency of hidp on usbhid).
> Remove the hid_ff_init() until either the hid-ff is made
> transport-independent, or at least support for bluetooth transport is
> added.
>
> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Under the condition that you remember to put it back once a generic FF
exists.
Regards
Marcel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Bluez-devel] Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 16:48 ` Marcel Holtmann
0 siblings, 0 replies; 16+ messages in thread
From: Marcel Holtmann @ 2007-05-08 16:48 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Satyam Sharma, chris, bluez-devel, linux-kernel, netdev
Hi Jiri,
> > Sure, my aim here was to only solve the _build breakage_ by fixing the
> > Kconfig for this module (that used code from another kernel module
> > without listing it in its dependencies). If, as you say, the real
> > solution is that we should actually be taking out the offending call to
> > the other module itself, then please go ahead -- I don't know much about
> > the Bluetooth / HIDP subsytem anyway.
>
> Converting the hid-ff drivers to be also transport-independent is on my
> TODO list, but it didn't happen yet.
>
> Marcel - are you aware of any devices currently supported by USB HID
> force-feedback code, which have a bluetooth version, please?
I haven't looked at all details for the PS3 controller, but that might
be the first one. In theory they can and at some point they will enter
the market.
> I'd propose the patch below, until I make the usbhid force-feedback code
> transport independent. Thanks.
>
>
>
> From: Jiri Kosina <jkosina@suse.cz>
>
> [Bluetooth] HIDP - don't initialize force feedback
>
> The current implementation of force feedback for HID devices is
> USB-transport only and therefore calling hid_ff_init() from hidp code is
> not going to work (plus it creates unwanted dependency of hidp on usbhid).
> Remove the hid_ff_init() until either the hid-ff is made
> transport-independent, or at least support for bluetooth transport is
> added.
>
> Signed-off-by: Jiri Kosina <jkosina@suse.cz>
Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
Under the condition that you remember to put it back once a generic FF
exists.
Regards
Marcel
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 16:48 ` [Bluez-devel] " Marcel Holtmann
@ 2007-05-08 17:47 ` Jiri Kosina
-1 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 17:47 UTC (permalink / raw)
To: Marcel Holtmann; +Cc: Satyam Sharma, chris, linux-kernel, bluez-devel, netdev
On Tue, 8 May 2007, Marcel Holtmann wrote:
> > Marcel - are you aware of any devices currently supported by USB HID
> > force-feedback code, which have a bluetooth version, please?
> I haven't looked at all details for the PS3 controller, but that might
> be the first one. In theory they can and at some point they will enter
> the market.
You are right, PS3 controller is going to be shipped in both variants. On
the other hand it is perfectly possible that we will need special
force-feedback driver for it anyway.
BTW when talking about this - we already have PS3 quirk present in usb hid
(extra control URB is required to make it operational), probably something
similar will be needed for BT version too.
> > From: Jiri Kosina <jkosina@suse.cz>
> > [Bluetooth] HIDP - don't initialize force feedback
> > The current implementation of force feedback for HID devices is
> > USB-transport only and therefore calling hid_ff_init() from hidp code is
> > not going to work (plus it creates unwanted dependency of hidp on usbhid).
> > Remove the hid_ff_init() until either the hid-ff is made
> > transport-independent, or at least support for bluetooth transport is
> > added.
> > Signed-off-by: Jiri Kosina <jkosina@suse.cz>
> Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
> Under the condition that you remember to put it back once a generic FF
> exists.
Sure. I will take this through my tree then, thanks.
--
Jiri Kosina
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 17:47 ` Jiri Kosina
0 siblings, 0 replies; 16+ messages in thread
From: Jiri Kosina @ 2007-05-08 17:47 UTC (permalink / raw)
To: Marcel Holtmann; +Cc: Satyam Sharma, chris, bluez-devel, linux-kernel, netdev
On Tue, 8 May 2007, Marcel Holtmann wrote:
> > Marcel - are you aware of any devices currently supported by USB HID
> > force-feedback code, which have a bluetooth version, please?
> I haven't looked at all details for the PS3 controller, but that might
> be the first one. In theory they can and at some point they will enter
> the market.
You are right, PS3 controller is going to be shipped in both variants. On
the other hand it is perfectly possible that we will need special
force-feedback driver for it anyway.
BTW when talking about this - we already have PS3 quirk present in usb hid
(extra control URB is required to make it operational), probably something
similar will be needed for BT version too.
> > From: Jiri Kosina <jkosina@suse.cz>
> > [Bluetooth] HIDP - don't initialize force feedback
> > The current implementation of force feedback for HID devices is
> > USB-transport only and therefore calling hid_ff_init() from hidp code is
> > not going to work (plus it creates unwanted dependency of hidp on usbhid).
> > Remove the hid_ff_init() until either the hid-ff is made
> > transport-independent, or at least support for bluetooth transport is
> > added.
> > Signed-off-by: Jiri Kosina <jkosina@suse.cz>
> Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
> Under the condition that you remember to put it back once a generic FF
> exists.
Sure. I will take this through my tree then, thanks.
--
Jiri Kosina
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
2007-05-08 17:47 ` Jiri Kosina
(?)
@ 2007-05-08 17:50 ` Marcel Holtmann
-1 siblings, 0 replies; 16+ messages in thread
From: Marcel Holtmann @ 2007-05-08 17:50 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Satyam Sharma, chris, linux-kernel, bluez-devel, netdev
Hi Jiri,
> > > Marcel - are you aware of any devices currently supported by USB HID
> > > force-feedback code, which have a bluetooth version, please?
> > I haven't looked at all details for the PS3 controller, but that might
> > be the first one. In theory they can and at some point they will enter
> > the market.
>
> You are right, PS3 controller is going to be shipped in both variants. On
> the other hand it is perfectly possible that we will need special
> force-feedback driver for it anyway.
>
> BTW when talking about this - we already have PS3 quirk present in usb hid
> (extra control URB is required to make it operational), probably something
> similar will be needed for BT version too.
I know, but we do that in user space before we hand over the control to
the kernel. It is simpler, because it is on the control channel and that
part is not nicely integrated into the HID subsystem to be used by the
transport layers right now. At least as far as I can tell.
Regards
Marcel
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 17:50 ` Marcel Holtmann
0 siblings, 0 replies; 16+ messages in thread
From: Marcel Holtmann @ 2007-05-08 17:50 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Satyam Sharma, chris, bluez-devel, linux-kernel, netdev
Hi Jiri,
> > > Marcel - are you aware of any devices currently supported by USB HID
> > > force-feedback code, which have a bluetooth version, please?
> > I haven't looked at all details for the PS3 controller, but that might
> > be the first one. In theory they can and at some point they will enter
> > the market.
>
> You are right, PS3 controller is going to be shipped in both variants. On
> the other hand it is perfectly possible that we will need special
> force-feedback driver for it anyway.
>
> BTW when talking about this - we already have PS3 quirk present in usb hid
> (extra control URB is required to make it operational), probably something
> similar will be needed for BT version too.
I know, but we do that in user space before we hand over the control to
the kernel. It is simpler, because it is on the control channel and that
part is not nicely integrated into the HID subsystem to be used by the
transport layers right now. At least as far as I can tell.
Regards
Marcel
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
^ permalink raw reply [flat|nested] 16+ messages in thread
* Re: [Bluez-devel] Getting make net/built-in.o Error with 2.6.21.1 Build
@ 2007-05-08 17:50 ` Marcel Holtmann
0 siblings, 0 replies; 16+ messages in thread
From: Marcel Holtmann @ 2007-05-08 17:50 UTC (permalink / raw)
To: Jiri Kosina; +Cc: Satyam Sharma, chris, bluez-devel, linux-kernel, netdev
Hi Jiri,
> > > Marcel - are you aware of any devices currently supported by USB HID
> > > force-feedback code, which have a bluetooth version, please?
> > I haven't looked at all details for the PS3 controller, but that might
> > be the first one. In theory they can and at some point they will enter
> > the market.
>
> You are right, PS3 controller is going to be shipped in both variants. On
> the other hand it is perfectly possible that we will need special
> force-feedback driver for it anyway.
>
> BTW when talking about this - we already have PS3 quirk present in usb hid
> (extra control URB is required to make it operational), probably something
> similar will be needed for BT version too.
I know, but we do that in user space before we hand over the control to
the kernel. It is simpler, because it is on the control channel and that
part is not nicely integrated into the HID subsystem to be used by the
transport layers right now. At least as far as I can tell.
Regards
Marcel
-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel
^ permalink raw reply [flat|nested] 16+ messages in thread
end of thread, other threads:[~2007-05-08 17:50 UTC | newest]
Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-05-07 23:30 Getting make net/built-in.o Error with 2.6.21.1 Build Chris Bergeron
2007-05-08 0:25 ` Satyam Sharma
2007-05-08 0:25 ` Satyam Sharma
2007-05-08 15:49 ` Jiri Kosina
2007-05-08 15:49 ` Jiri Kosina
2007-05-08 16:05 ` Satyam Sharma
2007-05-08 16:05 ` Satyam Sharma
2007-05-08 16:23 ` Jiri Kosina
2007-05-08 16:23 ` Jiri Kosina
2007-05-08 16:48 ` Marcel Holtmann
2007-05-08 16:48 ` [Bluez-devel] " Marcel Holtmann
2007-05-08 17:47 ` Jiri Kosina
2007-05-08 17:47 ` Jiri Kosina
2007-05-08 17:50 ` Marcel Holtmann
2007-05-08 17:50 ` [Bluez-devel] " Marcel Holtmann
2007-05-08 17:50 ` Marcel Holtmann
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.