* [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis
@ 2017-06-14 19:34 srishti sharma
2017-06-23 16:20 ` Greg KH
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-14 19:34 UTC (permalink / raw)
To: gregkh
Cc: stefan.wahren, eric, lee, f.fainelli, rjui, sbranden,
bcm-kernel-feedback-list, linux-rpi-kernel, linux-arm-kernel,
devel, linux-kernel, srishti sharma
Fixed alignment so that it matched open parenthesis.
Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
.../staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 18 +++++++++---------
1 file changed, 9 insertions(+), 9 deletions(-)
diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
index f484bb0..ce8ab36 100644
--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
+++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
@@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
int changed = 0;
@@ -185,7 +185,7 @@ static struct snd_kcontrol_new snd_bcm2835_ctl[] = {
};
static int snd_bcm2835_spdif_default_info(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_info *uinfo)
+ struct snd_ctl_elem_info *uinfo)
{
uinfo->type = SNDRV_CTL_ELEM_TYPE_IEC958;
uinfo->count = 1;
@@ -193,7 +193,7 @@ static int snd_bcm2835_spdif_default_info(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_default_get(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
int i;
@@ -210,7 +210,7 @@ static int snd_bcm2835_spdif_default_get(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_default_put(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
unsigned int val = 0;
@@ -230,7 +230,7 @@ static int snd_bcm2835_spdif_default_put(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_mask_info(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_info *uinfo)
+ struct snd_ctl_elem_info *uinfo)
{
uinfo->type = SNDRV_CTL_ELEM_TYPE_IEC958;
uinfo->count = 1;
@@ -238,7 +238,7 @@ static int snd_bcm2835_spdif_mask_info(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_mask_get(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
/*
* bcm2835 supports only consumer mode and sets all other format flags
@@ -249,7 +249,7 @@ static int snd_bcm2835_spdif_mask_get(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_stream_info(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_info *uinfo)
+ struct snd_ctl_elem_info *uinfo)
{
uinfo->type = SNDRV_CTL_ELEM_TYPE_IEC958;
uinfo->count = 1;
@@ -257,7 +257,7 @@ static int snd_bcm2835_spdif_stream_info(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_stream_get(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
int i;
@@ -274,7 +274,7 @@ static int snd_bcm2835_spdif_stream_get(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_spdif_stream_put(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
unsigned int val = 0;
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis
2017-06-14 19:34 [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis srishti sharma
@ 2017-06-23 16:20 ` Greg KH
2017-06-24 2:41 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-06-23 16:20 UTC (permalink / raw)
To: srishti sharma
Cc: stefan.wahren, eric, lee, f.fainelli, rjui, sbranden,
bcm-kernel-feedback-list, linux-rpi-kernel, linux-arm-kernel,
devel, linux-kernel
On Thu, Jun 15, 2017 at 01:04:55AM +0530, srishti sharma wrote:
> Fixed alignment so that it matched open parenthesis.
>
> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> ---
> .../staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 18 +++++++++---------
> 1 file changed, 9 insertions(+), 9 deletions(-)
What differs from your previous patch? Always properly version your
patch so we know what to take.
Please fix and resend.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis
2017-06-23 16:20 ` Greg KH
@ 2017-06-24 2:41 ` srishti sharma
2017-06-24 3:30 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-24 2:41 UTC (permalink / raw)
To: Greg KH
Cc: stefan.wahren, Eric Anholt, lee, Florian Fainelli, rjui,
sbranden, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel, devel, linux-kernel
On Fri, Jun 23, 2017 at 9:50 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Thu, Jun 15, 2017 at 01:04:55AM +0530, srishti sharma wrote:
>> Fixed alignment so that it matched open parenthesis.
>>
>> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> ---
>> .../staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 18 +++++++++---------
>> 1 file changed, 9 insertions(+), 9 deletions(-)
>
> What differs from your previous patch?
The previous patch had only one check fixed , this has all checks of
the type = parenthesis_alignment in the file bcm2835-ctl.c fixed .
Also there was a typo in the description of the previous patch so I
corrected that .
Always properly version your
> patch so we know what to take.
>
> Please fix and resend.
>
> thanks,
>
> greg k-h
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis
2017-06-24 2:41 ` srishti sharma
@ 2017-06-24 3:30 ` srishti sharma
0 siblings, 0 replies; 13+ messages in thread
From: srishti sharma @ 2017-06-24 3:30 UTC (permalink / raw)
To: Greg KH
Cc: stefan.wahren, Eric Anholt, lee, Florian Fainelli, rjui,
sbranden, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel, devel, linux-kernel
On Sat, Jun 24, 2017 at 8:11 AM, srishti sharma <srishtishar@gmail.com> wrote:
> On Fri, Jun 23, 2017 at 9:50 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> On Thu, Jun 15, 2017 at 01:04:55AM +0530, srishti sharma wrote:
>>> Fixed alignment so that it matched open parenthesis.
>>>
>>> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>>> ---
>>> .../staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 18 +++++++++---------
>>> 1 file changed, 9 insertions(+), 9 deletions(-)
>>
>> What differs from your previous patch?
>
> The previous patch had only one check fixed , this has all checks of
> the type = parenthesis_alignment in the file bcm2835-ctl.c fixed .
> Also there was a typo in the description of the previous patch so I
> corrected that .
>
> Always properly version your
>> patch so we know what to take.
>>
>> Please fix and resend.
I have re-sent version 2 .
>>
>> thanks,
>>
>> greg k-h
>
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-14 13:49 ` Greg KH
@ 2017-06-14 13:52 ` srishti sharma
0 siblings, 0 replies; 13+ messages in thread
From: srishti sharma @ 2017-06-14 13:52 UTC (permalink / raw)
To: Greg KH
Cc: Dan Carpenter, stefan.wahren, devel, Florian Fainelli, sbranden,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, rjui, linux-arm-kernel
On Wed, Jun 14, 2017 at 7:19 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Wed, Jun 14, 2017 at 07:11:01PM +0530, srishti sharma wrote:
>> On Wed, Jun 14, 2017 at 10:39 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> > On Wed, Jun 14, 2017 at 03:15:11AM +0530, srishti sharma wrote:
>> >> On Tue, Jun 13, 2017 at 8:17 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
>> >> > On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
>> >> >> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> >> >> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
>> >> >> >> Fixed alignment so that it matched open parenthesis.
>> >> >> >>
>> >> >> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> >> >> >> ---
>> >> >> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
>> >> >> >> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >> >> >>
>> >> >> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> >> index f484bb0..2148ed0 100644
>> >> >> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
>> >> >> >> }
>> >> >> >>
>> >> >> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
>> >> >> >> - struct snd_ctl_elem_value *ucontrol)
>> >> >> >> + struct snd_ctl_elem_value *ucontrol)
>> >> >> >> {
>> >> >> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> >> >> >> int changed = 0;
>> >> >> >
>> >> >> > This patch is corrupted and can not be applied :(
>> >> >>
>> >> >>
>> >> >> Why is this corrupted ?
>> >> >
>> >> > Try applying it with `git am`. There should be space characters at the
>> >> > start of these lines but your email client deleted them:
>> >> >
>> >> > struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> >> > int changed = 0;
>> >> >
>> >> > Read Documentation/process/email-clients.rst
>> >> >
>> >> > regards,
>> >> > dan carpenter
>> >>
>> >>
>> >> Hello,
>> >>
>> >> I tried applying it with ' git am ' and it was giving me this error:
>> >>
>> >> fatal: corrupt patch at line XX
>> >
>> > Exactly.
>> >
>> >> I think this was produced because of running ./scripts/cleanfile on the patch .
>> >
>> > Why would you do that? By doing that you corrupted the patch file :(
>>
>> After I run ./scritps/checkpatch.pl on the patch produced by git
>> format-patch it has trailing whitespaces . So I was using
>> ./scripts/cleanfile to fix that , I am not doing that anymore and am
>> also not getting the corrupt patch error .
>
> That means your code had trailing whitespaces, always fix that and NEVER
> hand-edit or modify a patch unless you really know what you are doing.
>
>> > Just use the proper output of 'git format-patch' and all will be fine.
>>
>> I have done this and am still getting these errors:
>>
>> error: patch failed:
>> drivers/staging/vc04_services/bcm2835-audio/bcm2835- ctl.c:105
>>
>> error: drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c:
>> patch does not apply
>>
>>
>> Why is it not applying ?
>
> That is up to you to figure out, we have no idea as to exactly what you
> are doing :)
I guess , I was applying the patch to the same repository I made the
change in :P , I am not getting any errors now .
>
> good luck!
>
> greg k-h
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-14 13:41 ` srishti sharma
@ 2017-06-14 13:49 ` Greg KH
2017-06-14 13:52 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-06-14 13:49 UTC (permalink / raw)
To: srishti sharma
Cc: Dan Carpenter, stefan.wahren, devel, Florian Fainelli, sbranden,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, rjui, linux-arm-kernel
On Wed, Jun 14, 2017 at 07:11:01PM +0530, srishti sharma wrote:
> On Wed, Jun 14, 2017 at 10:39 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Wed, Jun 14, 2017 at 03:15:11AM +0530, srishti sharma wrote:
> >> On Tue, Jun 13, 2017 at 8:17 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> >> > On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
> >> >> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >> >> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
> >> >> >> Fixed alignment so that it matched open parenthesis.
> >> >> >>
> >> >> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> >> >> >> ---
> >> >> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
> >> >> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >> >> >>
> >> >> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> >> index f484bb0..2148ed0 100644
> >> >> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
> >> >> >> }
> >> >> >>
> >> >> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
> >> >> >> - struct snd_ctl_elem_value *ucontrol)
> >> >> >> + struct snd_ctl_elem_value *ucontrol)
> >> >> >> {
> >> >> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> >> >> >> int changed = 0;
> >> >> >
> >> >> > This patch is corrupted and can not be applied :(
> >> >>
> >> >>
> >> >> Why is this corrupted ?
> >> >
> >> > Try applying it with `git am`. There should be space characters at the
> >> > start of these lines but your email client deleted them:
> >> >
> >> > struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> >> > int changed = 0;
> >> >
> >> > Read Documentation/process/email-clients.rst
> >> >
> >> > regards,
> >> > dan carpenter
> >>
> >>
> >> Hello,
> >>
> >> I tried applying it with ' git am ' and it was giving me this error:
> >>
> >> fatal: corrupt patch at line XX
> >
> > Exactly.
> >
> >> I think this was produced because of running ./scripts/cleanfile on the patch .
> >
> > Why would you do that? By doing that you corrupted the patch file :(
>
> After I run ./scritps/checkpatch.pl on the patch produced by git
> format-patch it has trailing whitespaces . So I was using
> ./scripts/cleanfile to fix that , I am not doing that anymore and am
> also not getting the corrupt patch error .
That means your code had trailing whitespaces, always fix that and NEVER
hand-edit or modify a patch unless you really know what you are doing.
> > Just use the proper output of 'git format-patch' and all will be fine.
>
> I have done this and am still getting these errors:
>
> error: patch failed:
> drivers/staging/vc04_services/bcm2835-audio/bcm2835- ctl.c:105
>
> error: drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c:
> patch does not apply
>
>
> Why is it not applying ?
That is up to you to figure out, we have no idea as to exactly what you
are doing :)
good luck!
greg k-h
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-14 5:09 ` Greg KH
@ 2017-06-14 13:41 ` srishti sharma
2017-06-14 13:49 ` Greg KH
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-14 13:41 UTC (permalink / raw)
To: Greg KH
Cc: Dan Carpenter, stefan.wahren, devel, Florian Fainelli, sbranden,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, rjui, linux-arm-kernel
On Wed, Jun 14, 2017 at 10:39 AM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Wed, Jun 14, 2017 at 03:15:11AM +0530, srishti sharma wrote:
>> On Tue, Jun 13, 2017 at 8:17 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
>> > On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
>> >> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> >> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
>> >> >> Fixed alignment so that it matched open parenthesis.
>> >> >>
>> >> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> >> >> ---
>> >> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
>> >> >> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >> >>
>> >> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> index f484bb0..2148ed0 100644
>> >> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
>> >> >> }
>> >> >>
>> >> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
>> >> >> - struct snd_ctl_elem_value *ucontrol)
>> >> >> + struct snd_ctl_elem_value *ucontrol)
>> >> >> {
>> >> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> >> >> int changed = 0;
>> >> >
>> >> > This patch is corrupted and can not be applied :(
>> >>
>> >>
>> >> Why is this corrupted ?
>> >
>> > Try applying it with `git am`. There should be space characters at the
>> > start of these lines but your email client deleted them:
>> >
>> > struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> > int changed = 0;
>> >
>> > Read Documentation/process/email-clients.rst
>> >
>> > regards,
>> > dan carpenter
>>
>>
>> Hello,
>>
>> I tried applying it with ' git am ' and it was giving me this error:
>>
>> fatal: corrupt patch at line XX
>
> Exactly.
>
>> I think this was produced because of running ./scripts/cleanfile on the patch .
>
> Why would you do that? By doing that you corrupted the patch file :(
After I run ./scritps/checkpatch.pl on the patch produced by git
format-patch it has trailing whitespaces . So I was using
./scripts/cleanfile to fix that , I am not doing that anymore and am
also not getting the corrupt patch error .
>
> Just use the proper output of 'git format-patch' and all will be fine.
I have done this and am still getting these errors:
error: patch failed:
drivers/staging/vc04_services/bcm2835-audio/bcm2835- ctl.c:105
error: drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c:
patch does not apply
Why is it not applying ?
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-13 21:45 ` srishti sharma
@ 2017-06-14 5:09 ` Greg KH
2017-06-14 13:41 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-06-14 5:09 UTC (permalink / raw)
To: srishti sharma
Cc: Dan Carpenter, stefan.wahren, devel, Florian Fainelli, sbranden,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, rjui, linux-arm-kernel
On Wed, Jun 14, 2017 at 03:15:11AM +0530, srishti sharma wrote:
> On Tue, Jun 13, 2017 at 8:17 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> > On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
> >> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> >> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
> >> >> Fixed alignment so that it matched open parenthesis.
> >> >>
> >> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> >> >> ---
> >> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
> >> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >> >>
> >> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> index f484bb0..2148ed0 100644
> >> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
> >> >> }
> >> >>
> >> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
> >> >> - struct snd_ctl_elem_value *ucontrol)
> >> >> + struct snd_ctl_elem_value *ucontrol)
> >> >> {
> >> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> >> >> int changed = 0;
> >> >
> >> > This patch is corrupted and can not be applied :(
> >>
> >>
> >> Why is this corrupted ?
> >
> > Try applying it with `git am`. There should be space characters at the
> > start of these lines but your email client deleted them:
> >
> > struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> > int changed = 0;
> >
> > Read Documentation/process/email-clients.rst
> >
> > regards,
> > dan carpenter
>
>
> Hello,
>
> I tried applying it with ' git am ' and it was giving me this error:
>
> fatal: corrupt patch at line XX
Exactly.
> I think this was produced because of running ./scripts/cleanfile on the patch .
Why would you do that? By doing that you corrupted the patch file :(
Just use the proper output of 'git format-patch' and all will be fine.
thanks,
greg k-h
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-13 14:47 ` Dan Carpenter
@ 2017-06-13 21:45 ` srishti sharma
2017-06-14 5:09 ` Greg KH
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-13 21:45 UTC (permalink / raw)
To: Dan Carpenter
Cc: Greg KH, stefan.wahren, devel, Florian Fainelli, sbranden, rjui,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
On Tue, Jun 13, 2017 at 8:17 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
>> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
>> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
>> >> Fixed alignment so that it matched open parenthesis.
>> >>
>> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> >> ---
>> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
>> >> 1 file changed, 1 insertion(+), 1 deletion(-)
>> >>
>> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> index f484bb0..2148ed0 100644
>> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
>> >> }
>> >>
>> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
>> >> - struct snd_ctl_elem_value *ucontrol)
>> >> + struct snd_ctl_elem_value *ucontrol)
>> >> {
>> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> >> int changed = 0;
>> >
>> > This patch is corrupted and can not be applied :(
>>
>>
>> Why is this corrupted ?
>
> Try applying it with `git am`. There should be space characters at the
> start of these lines but your email client deleted them:
>
> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> int changed = 0;
>
> Read Documentation/process/email-clients.rst
>
> regards,
> dan carpenter
Hello,
I tried applying it with ' git am ' and it was giving me this error:
fatal: corrupt patch at line XX
I think this was produced because of running ./scripts/cleanfile on the patch .
I was using this to fix trailing whitespaces . So, I used a different
method to remove them and didn't get that error.
But now when I run 'git am' I get a different error:
error: patch failed:
drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c:105
Why am I getting this error ?
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-13 14:37 ` srishti sharma
@ 2017-06-13 14:47 ` Dan Carpenter
2017-06-13 21:45 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: Dan Carpenter @ 2017-06-13 14:47 UTC (permalink / raw)
To: srishti sharma
Cc: Greg KH, stefan.wahren, devel, Florian Fainelli, sbranden, rjui,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
On Tue, Jun 13, 2017 at 08:07:14PM +0530, srishti sharma wrote:
> On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> > On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
> >> Fixed alignment so that it matched open parenthesis.
> >>
> >> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> >> ---
> >> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> index f484bb0..2148ed0 100644
> >> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> >> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
> >> }
> >>
> >> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
> >> - struct snd_ctl_elem_value *ucontrol)
> >> + struct snd_ctl_elem_value *ucontrol)
> >> {
> >> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> >> int changed = 0;
> >
> > This patch is corrupted and can not be applied :(
>
>
> Why is this corrupted ?
Try applying it with `git am`. There should be space characters at the
start of these lines but your email client deleted them:
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
int changed = 0;
Read Documentation/process/email-clients.rst
regards,
dan carpenter
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-13 13:00 ` Greg KH
@ 2017-06-13 14:37 ` srishti sharma
2017-06-13 14:47 ` Dan Carpenter
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-13 14:37 UTC (permalink / raw)
To: Greg KH
Cc: stefan.wahren, devel, Florian Fainelli, sbranden, rjui,
linux-kernel, Eric Anholt, bcm-kernel-feedback-list,
linux-rpi-kernel, linux-arm-kernel
On Tue, Jun 13, 2017 at 6:30 PM, Greg KH <gregkh@linuxfoundation.org> wrote:
> On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
>> Fixed alignment so that it matched open parenthesis.
>>
>> Signed-off-by: srishti sharma <srishtishar@gmail.com>
>> ---
>> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> index f484bb0..2148ed0 100644
>> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
>> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
>> }
>>
>> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
>> - struct snd_ctl_elem_value *ucontrol)
>> + struct snd_ctl_elem_value *ucontrol)
>> {
>> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
>> int changed = 0;
>
> This patch is corrupted and can not be applied :(
Why is this corrupted ?
Regards,
Srishti
^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
2017-06-09 21:07 srishti sharma
@ 2017-06-13 13:00 ` Greg KH
2017-06-13 14:37 ` srishti sharma
0 siblings, 1 reply; 13+ messages in thread
From: Greg KH @ 2017-06-13 13:00 UTC (permalink / raw)
To: srishti sharma
Cc: stefan.wahren, devel, f.fainelli, sbranden, rjui, linux-kernel,
eric, bcm-kernel-feedback-list, linux-rpi-kernel,
linux-arm-kernel
On Sat, Jun 10, 2017 at 02:37:22AM +0530, srishti sharma wrote:
> Fixed alignment so that it matched open parenthesis.
>
> Signed-off-by: srishti sharma <srishtishar@gmail.com>
> ---
> drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> index f484bb0..2148ed0 100644
> --- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> +++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
> @@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
> }
>
> static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
> - struct snd_ctl_elem_value *ucontrol)
> + struct snd_ctl_elem_value *ucontrol)
> {
> struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
> int changed = 0;
This patch is corrupted and can not be applied :(
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis.
@ 2017-06-09 21:07 srishti sharma
2017-06-13 13:00 ` Greg KH
0 siblings, 1 reply; 13+ messages in thread
From: srishti sharma @ 2017-06-09 21:07 UTC (permalink / raw)
To: gregkh
Cc: eric, stefan.wahren, f.fainelli, rjui, sbranden,
bcm-kernel-feedback-list, linux-rpi-kernel, linux-arm-kernel,
devel, linux-kernel, srishti sharma
Fixed alignment so that it matched open parenthesis.
Signed-off-by: srishti sharma <srishtishar@gmail.com>
---
drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
index f484bb0..2148ed0 100644
--- a/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
+++ b/drivers/staging/vc04_services/bcm2835-audio/bcm2835-ctl.c
@@ -105,7 +105,7 @@ static int snd_bcm2835_ctl_get(struct snd_kcontrol *kcontrol,
}
static int snd_bcm2835_ctl_put(struct snd_kcontrol *kcontrol,
- struct snd_ctl_elem_value *ucontrol)
+ struct snd_ctl_elem_value *ucontrol)
{
struct bcm2835_chip *chip = snd_kcontrol_chip(kcontrol);
int changed = 0;
--
2.7.4
^ permalink raw reply related [flat|nested] 13+ messages in thread
end of thread, other threads:[~2017-06-24 3:30 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-14 19:34 [PATCH] Staging: vc04_services: bcm2835-audio: bcm2835-ctl.c: Fixed alignment to match open parenthesis srishti sharma
2017-06-23 16:20 ` Greg KH
2017-06-24 2:41 ` srishti sharma
2017-06-24 3:30 ` srishti sharma
-- strict thread matches above, loose matches on Subject: below --
2017-06-09 21:07 srishti sharma
2017-06-13 13:00 ` Greg KH
2017-06-13 14:37 ` srishti sharma
2017-06-13 14:47 ` Dan Carpenter
2017-06-13 21:45 ` srishti sharma
2017-06-14 5:09 ` Greg KH
2017-06-14 13:41 ` srishti sharma
2017-06-14 13:49 ` Greg KH
2017-06-14 13:52 ` srishti sharma
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).