patch submission process

patch submission process

[Mathieu Lacage]

One month ago, I followed arnd's suggestion and posted these two patches:
http://marc.info/?l=linux-arch&m=127763437119458&w=2
http://marc.info/?l=linux-arch&m=127719544028714&w=2

and I would like to know if they were rejected because I screwed up
something in the submission process or just forgotten or maybe merged
in a tree I did not hear about.

Is there a special process to follow to monitor the state of a patch
submission ?

Mathieu
-- 
Mathieu Lacage <mathieu.lacage@gmail.com>

Re: patch submission process

[Arnd Bergmann]

On Wednesday 28 July 2010, Mathieu Lacage wrote:
> One month ago, I followed arnd's suggestion and posted these two patches:
> http://marc.info/?l=linux-arch&m=127763437119458&w=2
> http://marc.info/?l=linux-arch&m=127719544028714&w=2
> 
> and I would like to know if they were rejected because I screwed up
> something in the submission process or just forgotten or maybe merged
> in a tree I did not hear about.
> 
> Is there a special process to follow to monitor the state of a patch
> submission ?

I'm sorry for being so unorganized on this, I should really have added
the patches into my git tree on git.kernel.org.

The best replacement for a process is to keep bugging the maintainer
until the patches show up in the linux-next kernel. I still have
you patch emails in my incoming mail folder.

	Arnd

[PATCH] Add "Line In" input jack constants

[David Henningsson]

[-- Attachment #1: Type: text/plain, Size: 207 bytes --]

Hi,

Do you think this is reasonable to apply to 3.2? If so I'll continue 
with writing some patches that use this new constant, of course.

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

[-- Attachment #2: 0001-Add-Line-In-input-jack-constants.patch --]
[-- Type: text/x-patch, Size: 2357 bytes --]

>From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
From: David Henningsson <david.henningsson@canonical.com>
Date: Wed, 5 Oct 2011 15:53:25 +0200
Subject: [PATCH] Add "Line In" input jack constants

Similar to Line Out, these constants form the base for future
patches enabling input jack reporting for Line in jacks.

Signed-off-by: David Henningsson <david.henningsson@canonical.com>
---
 include/linux/input.h     |    1 +
 include/sound/jack.h      |    1 +
 sound/core/jack.c         |    1 +
 sound/pci/hda/hda_codec.c |    2 ++
 4 files changed, 5 insertions(+), 0 deletions(-)

diff --git a/include/linux/input.h b/include/linux/input.h
index a637e78..a514fb8 100644
--- a/include/linux/input.h
+++ b/include/linux/input.h
@@ -814,6 +814,7 @@ struct input_keymap_entry {
 #define SW_KEYPAD_SLIDE		0x0a  /* set = keypad slide out */
 #define SW_FRONT_PROXIMITY	0x0b  /* set = front proximity sensor active */
 #define SW_ROTATE_LOCK		0x0c  /* set = rotate locked/disabled */
+#define SW_LINEIN_INSERT	0x0d  /* set = inserted */
 #define SW_MAX			0x0f
 #define SW_CNT			(SW_MAX+1)
 
diff --git a/include/sound/jack.h b/include/sound/jack.h
index c140fc7..63c7907 100644
--- a/include/sound/jack.h
+++ b/include/sound/jack.h
@@ -42,6 +42,7 @@ enum snd_jack_types {
 	SND_JACK_MECHANICAL	= 0x0008, /* If detected separately */
 	SND_JACK_VIDEOOUT	= 0x0010,
 	SND_JACK_AVOUT		= SND_JACK_LINEOUT | SND_JACK_VIDEOOUT,
+	SND_JACK_LINEIN		= 0x0020,
 
 	/* Kept separate from switches to facilitate implementation */
 	SND_JACK_BTN_0		= 0x4000,
diff --git a/sound/core/jack.c b/sound/core/jack.c
index 53b53e9..240a3e1 100644
--- a/sound/core/jack.c
+++ b/sound/core/jack.c
@@ -30,6 +30,7 @@ static int jack_switch_types[] = {
 	SW_LINEOUT_INSERT,
 	SW_JACK_PHYSICAL_INSERT,
 	SW_VIDEOOUT_INSERT,
+	SW_LINEIN_INSERT,
 };
 
 static int snd_jack_dev_free(struct snd_device *device)
diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
index e3db196..8b046a1 100644
--- a/sound/pci/hda/hda_codec.c
+++ b/sound/pci/hda/hda_codec.c
@@ -5264,6 +5264,8 @@ static const char *get_jack_default_name(struct hda_codec *codec, hda_nid_t nid,
 		return "Mic";
 	case SND_JACK_LINEOUT:
 		return "Line-out";
+	case SND_JACK_LINEIN:
+		return "Line-in";
 	case SND_JACK_HEADSET:
 		return "Headset";
 	case SND_JACK_VIDEOOUT:
-- 
1.7.5.4


[-- Attachment #3: Type: text/plain, Size: 0 bytes --]

Re: [PATCH] Add "Line In" input jack constants

[Takashi Iwai]

At Wed, 05 Oct 2011 16:01:22 +0200,
David Henningsson wrote:
> 
> Hi,
> 
> Do you think this is reasonable to apply to 3.2? If so I'll continue 
> with writing some patches that use this new constant, of course.

Looks OK to me.  I can take it in unless Mark has any objection.


thanks,

Takashi

> -- 
> David Henningsson, Canonical Ltd.
> http://launchpad.net/~diwic
> [2 0001-Add-Line-In-input-jack-constants.patch <text/x-patch (7bit)>]
> >From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
> From: David Henningsson <david.henningsson@canonical.com>
> Date: Wed, 5 Oct 2011 15:53:25 +0200
> Subject: [PATCH] Add "Line In" input jack constants
> 
> Similar to Line Out, these constants form the base for future
> patches enabling input jack reporting for Line in jacks.
> 
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> ---
>  include/linux/input.h     |    1 +
>  include/sound/jack.h      |    1 +
>  sound/core/jack.c         |    1 +
>  sound/pci/hda/hda_codec.c |    2 ++
>  4 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/include/linux/input.h b/include/linux/input.h
> index a637e78..a514fb8 100644
> --- a/include/linux/input.h
> +++ b/include/linux/input.h
> @@ -814,6 +814,7 @@ struct input_keymap_entry {
>  #define SW_KEYPAD_SLIDE		0x0a  /* set = keypad slide out */
>  #define SW_FRONT_PROXIMITY	0x0b  /* set = front proximity sensor active */
>  #define SW_ROTATE_LOCK		0x0c  /* set = rotate locked/disabled */
> +#define SW_LINEIN_INSERT	0x0d  /* set = inserted */
>  #define SW_MAX			0x0f
>  #define SW_CNT			(SW_MAX+1)
>  
> diff --git a/include/sound/jack.h b/include/sound/jack.h
> index c140fc7..63c7907 100644
> --- a/include/sound/jack.h
> +++ b/include/sound/jack.h
> @@ -42,6 +42,7 @@ enum snd_jack_types {
>  	SND_JACK_MECHANICAL	= 0x0008, /* If detected separately */
>  	SND_JACK_VIDEOOUT	= 0x0010,
>  	SND_JACK_AVOUT		= SND_JACK_LINEOUT | SND_JACK_VIDEOOUT,
> +	SND_JACK_LINEIN		= 0x0020,
>  
>  	/* Kept separate from switches to facilitate implementation */
>  	SND_JACK_BTN_0		= 0x4000,
> diff --git a/sound/core/jack.c b/sound/core/jack.c
> index 53b53e9..240a3e1 100644
> --- a/sound/core/jack.c
> +++ b/sound/core/jack.c
> @@ -30,6 +30,7 @@ static int jack_switch_types[] = {
>  	SW_LINEOUT_INSERT,
>  	SW_JACK_PHYSICAL_INSERT,
>  	SW_VIDEOOUT_INSERT,
> +	SW_LINEIN_INSERT,
>  };
>  
>  static int snd_jack_dev_free(struct snd_device *device)
> diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
> index e3db196..8b046a1 100644
> --- a/sound/pci/hda/hda_codec.c
> +++ b/sound/pci/hda/hda_codec.c
> @@ -5264,6 +5264,8 @@ static const char *get_jack_default_name(struct hda_codec *codec, hda_nid_t nid,
>  		return "Mic";
>  	case SND_JACK_LINEOUT:
>  		return "Line-out";
> +	case SND_JACK_LINEIN:
> +		return "Line-in";
>  	case SND_JACK_HEADSET:
>  		return "Headset";
>  	case SND_JACK_VIDEOOUT:
> -- 
> 1.7.5.4
> 

Re: [PATCH] Add "Line In" input jack constants

[Mark Brown]

On Wed, Oct 05, 2011 at 04:01:22PM +0200, David Henningsson wrote:
> Hi,
> 
> Do you think this is reasonable to apply to 3.2? If so I'll continue
> with writing some patches that use this new constant, of course.
> 
> -- 
> David Henningsson, Canonical Ltd.
> http://launchpad.net/~diwic

> From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
> From: David Henningsson <david.henningsson@canonical.com>
> Date: Wed, 5 Oct 2011 15:53:25 +0200
> Subject: [PATCH] Add "Line In" input jack constants
> 
> Similar to Line Out, these constants form the base for future
> patches enabling input jack reporting for Line in jacks.

Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

but please follow the patch submission process in SubmittingPatches - in
particular, don't send your patch as an attachment.

Re: [PATCH] Add "Line In" input jack constants

[Takashi Iwai]

At Wed, 05 Oct 2011 16:01:22 +0200,
David Henningsson wrote:
> 
> Hi,
> 
> Do you think this is reasonable to apply to 3.2? If so I'll continue 
> with writing some patches that use this new constant, of course.

Oh, we should add Dmitry to Cc, since the patch touchs linux/input.h.
Dmitry, is it OK to add SW_LINEIN_INSERT as below?


thanks,

Takashi

> 
> -- 
> David Henningsson, Canonical Ltd.
> http://launchpad.net/~diwic
> [2 0001-Add-Line-In-input-jack-constants.patch <text/x-patch (7bit)>]
> >From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
> From: David Henningsson <david.henningsson@canonical.com>
> Date: Wed, 5 Oct 2011 15:53:25 +0200
> Subject: [PATCH] Add "Line In" input jack constants
> 
> Similar to Line Out, these constants form the base for future
> patches enabling input jack reporting for Line in jacks.
> 
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> ---
>  include/linux/input.h     |    1 +
>  include/sound/jack.h      |    1 +
>  sound/core/jack.c         |    1 +
>  sound/pci/hda/hda_codec.c |    2 ++
>  4 files changed, 5 insertions(+), 0 deletions(-)
> 
> diff --git a/include/linux/input.h b/include/linux/input.h
> index a637e78..a514fb8 100644
> --- a/include/linux/input.h
> +++ b/include/linux/input.h
> @@ -814,6 +814,7 @@ struct input_keymap_entry {
>  #define SW_KEYPAD_SLIDE		0x0a  /* set = keypad slide out */
>  #define SW_FRONT_PROXIMITY	0x0b  /* set = front proximity sensor active */
>  #define SW_ROTATE_LOCK		0x0c  /* set = rotate locked/disabled */
> +#define SW_LINEIN_INSERT	0x0d  /* set = inserted */
>  #define SW_MAX			0x0f
>  #define SW_CNT			(SW_MAX+1)
>  
> diff --git a/include/sound/jack.h b/include/sound/jack.h
> index c140fc7..63c7907 100644
> --- a/include/sound/jack.h
> +++ b/include/sound/jack.h
> @@ -42,6 +42,7 @@ enum snd_jack_types {
>  	SND_JACK_MECHANICAL	= 0x0008, /* If detected separately */
>  	SND_JACK_VIDEOOUT	= 0x0010,
>  	SND_JACK_AVOUT		= SND_JACK_LINEOUT | SND_JACK_VIDEOOUT,
> +	SND_JACK_LINEIN		= 0x0020,
>  
>  	/* Kept separate from switches to facilitate implementation */
>  	SND_JACK_BTN_0		= 0x4000,
> diff --git a/sound/core/jack.c b/sound/core/jack.c
> index 53b53e9..240a3e1 100644
> --- a/sound/core/jack.c
> +++ b/sound/core/jack.c
> @@ -30,6 +30,7 @@ static int jack_switch_types[] = {
>  	SW_LINEOUT_INSERT,
>  	SW_JACK_PHYSICAL_INSERT,
>  	SW_VIDEOOUT_INSERT,
> +	SW_LINEIN_INSERT,
>  };
>  
>  static int snd_jack_dev_free(struct snd_device *device)
> diff --git a/sound/pci/hda/hda_codec.c b/sound/pci/hda/hda_codec.c
> index e3db196..8b046a1 100644
> --- a/sound/pci/hda/hda_codec.c
> +++ b/sound/pci/hda/hda_codec.c
> @@ -5264,6 +5264,8 @@ static const char *get_jack_default_name(struct hda_codec *codec, hda_nid_t nid,
>  		return "Mic";
>  	case SND_JACK_LINEOUT:
>  		return "Line-out";
> +	case SND_JACK_LINEIN:
> +		return "Line-in";
>  	case SND_JACK_HEADSET:
>  		return "Headset";
>  	case SND_JACK_VIDEOOUT:
> -- 
> 1.7.5.4
> 

Re: [PATCH] Add "Line In" input jack constants

[Dmitry Torokhov]

On Wed, Oct 05, 2011 at 05:55:17PM +0200, Takashi Iwai wrote:
> At Wed, 05 Oct 2011 16:01:22 +0200,
> David Henningsson wrote:
> > 
> > Hi,
> > 
> > Do you think this is reasonable to apply to 3.2? If so I'll continue 
> > with writing some patches that use this new constant, of course.
> 
> Oh, we should add Dmitry to Cc, since the patch touchs linux/input.h.
> Dmitry, is it OK to add SW_LINEIN_INSERT as below?
> 

Yeah, that should be OK.

Thanks.

-- 
Dmitry

Re: [PATCH] Add "Line In" input jack constants

[Takashi Iwai]

At Wed, 5 Oct 2011 16:07:56 +0100,
Mark Brown wrote:
> 
> On Wed, Oct 05, 2011 at 04:01:22PM +0200, David Henningsson wrote:
> > Hi,
> > 
> > Do you think this is reasonable to apply to 3.2? If so I'll continue
> > with writing some patches that use this new constant, of course.
> > 
> > -- 
> > David Henningsson, Canonical Ltd.
> > http://launchpad.net/~diwic
> 
> > From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
> > From: David Henningsson <david.henningsson@canonical.com>
> > Date: Wed, 5 Oct 2011 15:53:25 +0200
> > Subject: [PATCH] Add "Line In" input jack constants
> > 
> > Similar to Line Out, these constants form the base for future
> > patches enabling input jack reporting for Line in jacks.
> 
> Acked-by: Mark Brown <broonie@opensource.wolfsonmicro.com>

OK, applied now.  Thanks.


Takashi

Patch submission process (was: [PATCH] Add "Line In" input jack constants)

[David Henningsson]

On 10/05/2011 05:07 PM, Mark Brown wrote:
> but please follow the patch submission process in SubmittingPatches - in
> particular, don't send your patch as an attachment.

I find this hard to fulfil at times, in particular when I want to add 
something to the email that I don't want to be a part of the commit 
message. Is this possible with git send-email, and without sending the 
patch as an attachment?

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

Re: [PATCH] Add "Line In" input jack constants

[David Henningsson]

On 10/06/2011 08:23 AM, Takashi Iwai wrote:
> At Wed, 5 Oct 2011 16:07:56 +0100,
> Mark Brown wrote:
>>
>> On Wed, Oct 05, 2011 at 04:01:22PM +0200, David Henningsson wrote:
>>> Hi,
>>>
>>> Do you think this is reasonable to apply to 3.2? If so I'll continue
>>> with writing some patches that use this new constant, of course.
>>>
>>> --
>>> David Henningsson, Canonical Ltd.
>>> http://launchpad.net/~diwic
>>
>>>  From 356e80c4e2f5d9d1c85b5eee93d2932f7637b338 Mon Sep 17 00:00:00 2001
>>> From: David Henningsson<david.henningsson@canonical.com>
>>> Date: Wed, 5 Oct 2011 15:53:25 +0200
>>> Subject: [PATCH] Add "Line In" input jack constants
>>>
>>> Similar to Line Out, these constants form the base for future
>>> patches enabling input jack reporting for Line in jacks.
>>
>> Acked-by: Mark Brown<broonie@opensource.wolfsonmicro.com>
>
> OK, applied now.  Thanks.

Danke schön! And thanks for taking the effort of contacting the relevant 
people.

-- 
David Henningsson, Canonical Ltd.
http://launchpad.net/~diwic

Re: Patch submission process

[Clemens Ladisch]

On 10/06/2011 08:46 AM, David Henningsson wrote:
> On 10/05/2011 05:07 PM, Mark Brown wrote:
> > ... don't send your patch as an attachment.
>
> I find this hard to fulfil at times, in particular when I want to add
> something to the email that I don't want to be a part of the commit
> message. Is this possible with git send-email, and without sending the
> patch as an attachment?

Use "git send-email --annotate", and delimit the something from the
commit message with a scissors line (man git-am).


Regards,
Clemens

Re: Patch submission process (was: [PATCH] Add "Line In" input jack constants)

[Mark Brown]

On Thu, Oct 06, 2011 at 08:46:22AM +0200, David Henningsson wrote:
> On 10/05/2011 05:07 PM, Mark Brown wrote:
> >but please follow the patch submission process in SubmittingPatches - in
> >particular, don't send your patch as an attachment.

> I find this hard to fulfil at times, in particular when I want to
> add something to the email that I don't want to be a part of the
> commit message. Is this possible with git send-email, and without
> sending the patch as an attachment?

The relevant section of SubmittingPatches is:

|  - A marker line containing simply "---".
|
|   - Any additional comments not suitable for the changelog.
|
|   - The actual patch (diff output).

You can edit the patch after running format-patch, or use --annotate
with send-mail.