[PATCH] Adds Sony Move Navigation controller.

[PATCH] Adds Sony Move Navigation controller.

[Alex Gal]

Signed-off-by: Alex <a.gal@miip.ca>
---
 plugins/sixaxis.c       | 7 +++++++
 profiles/input/server.c | 4 ++++
 2 files changed, 11 insertions(+)

diff --git a/plugins/sixaxis.c b/plugins/sixaxis.c
index b09404a..6f7893e 100644
--- a/plugins/sixaxis.c
+++ b/plugins/sixaxis.c
@@ -60,6 +60,13 @@ static const struct {
 		.pid = 0x0268,
 		.version = 0x0000,
 	},
+	{
+		.name = "PLAYSTATION(R)3 Controller", /* Move Navigation */
+		.source = 0x0002,
+		.vid = 0x054c,
+		.pid = 0x042f,
+		.version = 0x0000,
+	},
 };
 
 struct leds_data {
diff --git a/profiles/input/server.c b/profiles/input/server.c
index 307db96..5463311 100644
--- a/profiles/input/server.c
+++ b/profiles/input/server.c
@@ -140,6 +140,10 @@ static bool dev_is_sixaxis(const bdaddr_t *src, const bdaddr_t *dst)
 	if (vid == 0x054c && pid == 0x05c4)
 		return true;
 
+    /* Move Navigation */
+	if (vid == 0x054c && pid == 0x042f)
+		return true;
+
 	return false;
 }
 
-- 
2.1.0

Re: [PATCH] Adds Sony Move Navigation controller.

[Szymon Janc]

Hi Alex,

Thanks for patch, some small comments from my side.

On Tuesday 07 of October 2014 12:04:37 Alex Gal wrote:
> Signed-off-by: Alex <a.gal@miip.ca>

We don't use SOB in userspace so this is not needed.

Also I'd prefer if you could put plugin change into separate patch.
And prefix them appropriately ie. "sixaxis: " for plugin and "input: "
for input change.

> ---
>  plugins/sixaxis.c       | 7 +++++++
>  profiles/input/server.c | 4 ++++
>  2 files changed, 11 insertions(+)
> 
> diff --git a/plugins/sixaxis.c b/plugins/sixaxis.c
> index b09404a..6f7893e 100644
> --- a/plugins/sixaxis.c
> +++ b/plugins/sixaxis.c
> @@ -60,6 +60,13 @@ static const struct {
>  		.pid = 0x0268,
>  		.version = 0x0000,
>  	},
> +	{
> +		.name = "PLAYSTATION(R)3 Controller", /* Move Navigation */

Does Move Navigation use same name as DS3?

> +		.source = 0x0002,
> +		.vid = 0x054c,
> +		.pid = 0x042f,
> +		.version = 0x0000,
> +	},
>  };
>  
>  struct leds_data {
> diff --git a/profiles/input/server.c b/profiles/input/server.c
> index 307db96..5463311 100644
> --- a/profiles/input/server.c
> +++ b/profiles/input/server.c
> @@ -140,6 +140,10 @@ static bool dev_is_sixaxis(const bdaddr_t *src, const bdaddr_t *dst)
>  	if (vid == 0x054c && pid == 0x05c4)
>  		return true;
>  
> +    /* Move Navigation */
> +	if (vid == 0x054c && pid == 0x042f)
> +		return true;
> +
>  	return false;
>  }
>  
> 

-- 
Best regards, 
Szymon Janc

Re: [PATCH] Adds Sony Move Navigation controller.

[Alex Gal]

Hi Szymon,

On Fri, Oct 10, 2014 at 6:07 AM, Szymon Janc <szymon.janc@tieto.com> wrote:

> Thanks for patch, some small comments from my side.

No problem, I will resend the patches later this week.

By the way, you were right about the Move Navigation clones since they
try to connect with a different mac address. I will try to see what
can be done about them.

Thank you,

Alex