Re: [oe] [meta-handheld][PATCH] zaurusd: refresh patches (using devtool)

2018-04-12 Thread Denys Dmytriyenko 
On Thu, Apr 12, 2018 at 11:23:39AM +0200, Andrea Adami wrote:
> On Thu, Apr 12, 2018 at 10:59 AM, Koen Kooi  
> wrote:
> > Op 10-04-18 om 23:01 schreef Andrea Adami:
> >> Silence warnings about ingnored context.
> >>
> >> Signed-off-by: Andrea Adami 
> >
> >
> >> --- a/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
> >> +++ b/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
> >> @@ -1,17 +1,17 @@
> >> -From 85b94bed1feab70bc529a59fb80c26da825b9abe Mon Sep 17 00:00:00 2001
> >> +From 668a7bc50ca8f36df5a0eae73a5fa251aada9cc4 Mon Sep 17 00:00:00 2001
> >>  From: Dmitry Eremin-Solenikov 
> >>  Date: Fri, 5 Aug 2011 16:00:10 +0400
> >>  Subject: [PATCH] zaurusd: add support for collie
> >> -Upstream-Status: Pending
> > Was that change intended?
> 
> No, not really.
> Let say I have spotted it and planned to read the devtool code...
> 
> I think it is tolerable in this context, because unfortunately not all
> patches have Upstream-Status.
> (I am wondering if there is still Upstream for zaurusd...)
> 
> So I have left the result unchanged., planning to update the
> Upstream-Status soon.

Related, but not exactly the same:
https://bugzilla.yoctoproject.org/show_bug.cgi?id=12674

Basically, in your case Upstream-status was above the empty line that 
separates patch header and patch description. Upstream-status should be 
in the patch description or it will be lost by devtool...

-- 
Denys
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-handheld][PATCH] zaurusd: refresh patches (using devtool)

2018-04-12 Thread Andrea Adami 
On Thu, Apr 12, 2018 at 10:59 AM, Koen Kooi  wrote:
> Op 10-04-18 om 23:01 schreef Andrea Adami:
>> Silence warnings about ingnored context.
>>
>> Signed-off-by: Andrea Adami 
>
>
>> --- a/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
>> +++ b/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
>> @@ -1,17 +1,17 @@
>> -From 85b94bed1feab70bc529a59fb80c26da825b9abe Mon Sep 17 00:00:00 2001
>> +From 668a7bc50ca8f36df5a0eae73a5fa251aada9cc4 Mon Sep 17 00:00:00 2001
>>  From: Dmitry Eremin-Solenikov 
>>  Date: Fri, 5 Aug 2011 16:00:10 +0400
>>  Subject: [PATCH] zaurusd: add support for collie
>> -Upstream-Status: Pending
> Was that change intended?

No, not really.
Let say I have spotted it and planned to read the devtool code...

I think it is tolerable in this context, because unfortunately not all
patches have Upstream-Status.
(I am wondering if there is still Upstream for zaurusd...)

So I have left the result unchanged., planning to update the
Upstream-Status soon.

Cheers
Andrea

>
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

Re: [oe] [meta-handheld][PATCH] zaurusd: refresh patches (using devtool)

2018-04-12 Thread Koen Kooi 
Op 10-04-18 om 23:01 schreef Andrea Adami:
> Silence warnings about ingnored context.
> 
> Signed-off-by: Andrea Adami 


> --- a/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
> +++ b/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
> @@ -1,17 +1,17 @@
> -From 85b94bed1feab70bc529a59fb80c26da825b9abe Mon Sep 17 00:00:00 2001
> +From 668a7bc50ca8f36df5a0eae73a5fa251aada9cc4 Mon Sep 17 00:00:00 2001
>  From: Dmitry Eremin-Solenikov 
>  Date: Fri, 5 Aug 2011 16:00:10 +0400
>  Subject: [PATCH] zaurusd: add support for collie
> -Upstream-Status: Pending
Was that change intended?

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel

[oe] [meta-handheld][PATCH] zaurusd: refresh patches (using devtool)

2018-04-10 Thread Andrea Adami 
Silence warnings about ingnored context.

Signed-off-by: Andrea Adami 
---
 .../0001-zaurusd-add-support-for-collie.patch  |  16 +-
 recipes-bsp/zaurusd/zaurusd/add-poodle.patch   |  14 +-
 .../zaurusd/zaurusd/alsa-6000x-default.state.patch | 166 +++--
 .../zaurusd/zaurusd/alsa-cxx00-default.state.patch |  61 
 .../zaurusd/zaurusd/avoid-rotated-server.patch | 106 +++--
 .../zaurusd/zaurusd/disable-alsa-handling.patch|  16 +-
 recipes-bsp/zaurusd/zaurusd/scripts_pidof.patch|  10 +-
 .../use-ts-symlink-instead-of-hardcoding.diff  | 111 ++
 8 files changed, 300 insertions(+), 200 deletions(-)

diff --git 
a/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch 
b/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
index 36ded13..0d242f5 100644
--- a/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
+++ b/recipes-bsp/zaurusd/zaurusd/0001-zaurusd-add-support-for-collie.patch
@@ -1,17 +1,17 @@
-From 85b94bed1feab70bc529a59fb80c26da825b9abe Mon Sep 17 00:00:00 2001
+From 668a7bc50ca8f36df5a0eae73a5fa251aada9cc4 Mon Sep 17 00:00:00 2001
 From: Dmitry Eremin-Solenikov 
 Date: Fri, 5 Aug 2011 16:00:10 +0400
 Subject: [PATCH] zaurusd: add support for collie
-Upstream-Status: Pending
 
 Note: alsa mixer support is disabled, as collie doesn't support sound (yet).
 
 Signed-off-by: Dmitry Eremin-Solenikov 
+
 ---
- config/mach-config/mach-collie-functions |4 
- config/mach-config/mach-collie-vars.in   |3 +++
- configure.ac |1 +
- 3 files changed, 8 insertions(+), 0 deletions(-)
+ config/mach-config/mach-collie-functions | 4 
+ config/mach-config/mach-collie-vars.in   | 3 +++
+ configure.ac | 1 +
+ 3 files changed, 8 insertions(+)
  create mode 100644 config/mach-config/mach-collie-functions
  create mode 100644 config/mach-config/mach-collie-vars.in
 
@@ -35,7 +35,7 @@ index 000..ec1bfb8
 +
 +ALSA_MIXER_INIT=@appconfdir@/alsa/5500-default.state
 diff --git a/configure.ac b/configure.ac
-index 78ea2ae..5160fab 100644
+index 36f858c..8ee281b 100644
 --- a/configure.ac
 +++ b/configure.ac
 @@ -65,6 +65,7 @@ AC_CONFIG_FILES([
@@ -46,5 +46,3 @@ index 78ea2ae..5160fab 100644
config/mach-config/mach-corgi-vars
config/mach-config/mach-husky-vars
config/mach-config/mach-poodle-vars
---
-1.7.2.5
diff --git a/recipes-bsp/zaurusd/zaurusd/add-poodle.patch 
b/recipes-bsp/zaurusd/zaurusd/add-poodle.patch
index 3793faf..6698aeb 100644
--- a/recipes-bsp/zaurusd/zaurusd/add-poodle.patch
+++ b/recipes-bsp/zaurusd/zaurusd/add-poodle.patch
@@ -1,5 +1,15 @@
 zaurusd/config/mach-config/mach-poodle-vars.in (revision 418)
-+++ zaurusd/config/mach-config/mach-poodle-vars.in (working copy)
+From 9edcb004b09cef8e01598482935fc6ba2a6beb81 Mon Sep 17 00:00:00 2001
+From: Paul Eggleton 
+Date: Sun, 24 Jul 2011 19:12:32 +0100
+
+---
+ config/mach-config/mach-poodle-vars.in | 8 +++-
+ 1 file changed, 7 insertions(+), 1 deletion(-)
+
+diff --git a/config/mach-config/mach-poodle-vars.in 
b/config/mach-config/mach-poodle-vars.in
+index 949460a..cbb1478 100644
+--- a/config/mach-config/mach-poodle-vars.in
 b/config/mach-config/mach-poodle-vars.in
 @@ -1,3 +1,9 @@
  MACHINE=poodle
  
diff --git a/recipes-bsp/zaurusd/zaurusd/alsa-6000x-default.state.patch 
b/recipes-bsp/zaurusd/zaurusd/alsa-6000x-default.state.patch
index 29c98f7..0409100 100644
--- a/recipes-bsp/zaurusd/zaurusd/alsa-6000x-default.state.patch
+++ b/recipes-bsp/zaurusd/zaurusd/alsa-6000x-default.state.patch
@@ -1,6 +1,16 @@
 zaurusd/config/alsa/6000x-default.state2009-04-25 22:57:30.0 
+0100
-+++ zaurusd/config/alsa/6000x-default.state2009-05-14 10:41:29.0 
+0100
-@@ -6,8 +6,8 @@
+From dfd71335507681ec984195265ed5866e677d9bb6 Mon Sep 17 00:00:00 2001
+From: Paul Eggleton 
+Date: Sun, 24 Jul 2011 19:12:32 +0100
+
+---
+ config/alsa/6000x-default.state | 192 +---
+ 1 file changed, 101 insertions(+), 91 deletions(-)
+
+diff --git a/config/alsa/6000x-default.state b/config/alsa/6000x-default.state
+index d2bf5e2..c38265d 100644
+--- a/config/alsa/6000x-default.state
 b/config/alsa/6000x-default.state
+@@ -6,8 +6,8 @@ state.Tosa {
comment.range '0 - 31'
iface MIXER
name 'Speaker Playback Volume'
@@ -11,7 +21,7 @@
}
control.2 {
comment.access 'read write'
-@@ -24,8 +24,8 @@
+@@ -24,8 +24,8 @@ state.Tosa {
comment.range '0 - 31'
iface MIXER
name 'Headphone Playback Volume'
@@ -22,7 +32,7 @@
}
control.4 {
comment.access 'read write'
-@@ -37,13 +37,23 @@
+@@ -37,13 +37,23 @@ state.Tosa {
}
control.5 {