Google Groups no longer supports new Usenet posts or subscriptions. Historical content remains viewable.
Dismiss

[PATCH] DAS1800: Fixing foo * bar should be foo *bar

1 view
Skip to first unread message

Joe Borg

unread,
Jan 11, 2014, 9:50:01 AM1/11/14
to
---
drivers/staging/comedi/drivers/das1800.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
index 1880038..8fb87df 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -462,7 +462,7 @@ static inline uint16_t munge_bipolar_sample(const struct comedi_device *dev,
return sample;
}

-static void munge_data(struct comedi_device *dev, uint16_t * array,
+static void munge_data(struct comedi_device *dev, uint16_t *array,
unsigned int num_elements)
{
unsigned int i;
--
1.8.5.2.309.ga25014b

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majo...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

Dan Carpenter

unread,
Jan 13, 2014, 3:20:01 AM1/13/14
to
No signed-off line. Just merge both patches since they do the same
thing and go to the same maintainers.

regards,
dan carpenter

Dan Carpenter

unread,
Jan 13, 2014, 4:30:02 AM1/13/14
to
On Mon, Jan 13, 2014 at 09:02:33AM +0000, Joe Borġ wrote:
> Apologies for missing that.
>
> Feel free to add signed-off-by Joe Borg <cyborg...@gmail.com> to the
> commit message.
>

Unless it's a bugfix for a security issue then we don't fix other
people's patches. You have to resend.

Joe Borg

unread,
Jan 13, 2014, 3:20:02 PM1/13/14
to
Signed-off-by Joe Borg <cyborg...@gmail.com>
---
drivers/staging/comedi/drivers/das1800.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
index 1880038..8fb87df 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -462,7 +462,7 @@ static inline uint16_t munge_bipolar_sample(const struct comedi_device *dev,
return sample;
}

-static void munge_data(struct comedi_device *dev, uint16_t * array,
+static void munge_data(struct comedi_device *dev, uint16_t *array,
unsigned int num_elements)
{
unsigned int i;
--
1.8.5.2

Greg KH

unread,
Jan 13, 2014, 4:10:03 PM1/13/14
to
On Mon, Jan 13, 2014 at 08:14:46PM +0000, Joe Borg wrote:
> Signed-off-by Joe Borg <cyborg...@gmail.com>

You need a changelog comment here, please.

And is that your "real" email address and name?

greg k-h

Greg KH

unread,
Jan 13, 2014, 4:40:04 PM1/13/14
to
On Mon, Jan 13, 2014 at 09:23:11PM +0000, Joe Borġ wrote:
> Ok, will do the change log.
>
> Yes, it's my real name and email... I know the email address is embarrassing,
> but genuine :).
>
>
>
> Regards,
> Joseph David Borġ

There's nothing wrong with using "Joe Borġ" as your signed-off-by: line,
as that's your real name :)

thanks,

Joe Borg

unread,
Jan 13, 2014, 4:50:03 PM1/13/14
to
Fixed an error flagged by checkpatch.pl, 'foo * bar should be foo *bar' in the parameters of function munge_data().

Signed-off-by Joe Borg <cyborg...@gmail.com>
---
drivers/staging/comedi/drivers/das1800.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
index 1880038..8fb87df 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -462,7 +462,7 @@ static inline uint16_t munge_bipolar_sample(const struct comedi_device *dev,
return sample;
}

-static void munge_data(struct comedi_device *dev, uint16_t * array,
+static void munge_data(struct comedi_device *dev, uint16_t *array,
unsigned int num_elements)
{
unsigned int i;
--
1.8.5.2

Greg KH

unread,
Jan 13, 2014, 5:00:03 PM1/13/14
to
On Mon, Jan 13, 2014 at 09:47:59PM +0000, Joe Borg wrote:
> Fixed an error flagged by checkpatch.pl, 'foo * bar should be foo *bar' in the parameters of function munge_data().

Please wrap your changelog comments at 72 colums, like git asks you to
when you type them in.

> Signed-off-by Joe Borg <cyborg...@gmail.com>

No "ġ"?

Joe Borg

unread,
Jan 13, 2014, 6:20:02 PM1/13/14
to
Fixed an error flagged by checkpatch.pl, 'foo * bar should be foo
*bar' in the parameters of function munge_data().

Signed-off-by Joe Borg <cyborg...@gmail.com>
---
drivers/staging/comedi/drivers/das1800.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
index 1880038..8fb87df 100644
--- a/drivers/staging/comedi/drivers/das1800.c
+++ b/drivers/staging/comedi/drivers/das1800.c
@@ -462,7 +462,7 @@ static inline uint16_t munge_bipolar_sample(const struct comedi_device *dev,
return sample;
}

-static void munge_data(struct comedi_device *dev, uint16_t * array,
+static void munge_data(struct comedi_device *dev, uint16_t *array,
unsigned int num_elements)
{
unsigned int i;
--
1.8.5.2

Greg KH

unread,
Jan 13, 2014, 7:00:02 PM1/13/14
to
On Mon, Jan 13, 2014 at 11:19:15PM +0000, Joe Borg wrote:
> Fixed an error flagged by checkpatch.pl, 'foo * bar should be foo
> *bar' in the parameters of function munge_data().

Ok, real minor nits here, but I'd like you to make them for future
patches as it makes my life easier, and I don't like to edit patches by
hand, as I don't scale very well.

Your subject should be:
[PATCH] staging: comedi: das1800: fix for * bar to be foo *bar

you need to add the driver subsystem, and specific area and driver. And
no kernel drivers are ALL CAPS, so that should never be used.

I've taken this patch as-is, but in the future please try to do this.

Oh, one more thing:

> Signed-off-by Joe Borg <cyborg...@gmail.com>

You forgot a ':' here :)

thanks,

greg k-h

Greg KH

unread,
Jan 13, 2014, 7:00:02 PM1/13/14
to
On Mon, Jan 13, 2014 at 11:19:15PM +0000, Joe Borg wrote:
> Fixed an error flagged by checkpatch.pl, 'foo * bar should be foo
> *bar' in the parameters of function munge_data().
>
> Signed-off-by: Joe Borg <cyborg...@gmail.com>
> ---
> drivers/staging/comedi/drivers/das1800.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/staging/comedi/drivers/das1800.c b/drivers/staging/comedi/drivers/das1800.c
> index 1880038..8fb87df 100644
> --- a/drivers/staging/comedi/drivers/das1800.c
> +++ b/drivers/staging/comedi/drivers/das1800.c
> @@ -462,7 +462,7 @@ static inline uint16_t munge_bipolar_sample(const struct comedi_device *dev,
> return sample;
> }
>
> -static void munge_data(struct comedi_device *dev, uint16_t * array,
> +static void munge_data(struct comedi_device *dev, uint16_t *array,
> unsigned int num_elements)

Ok, after all that work, it turns out I can't even apply this patch as
someone else did it already a while ago. You should be working off of
linux-next, or my staging.git tree, not Linus's tree, as that does not
contain all of the work we are doing on the next version of linux (hence
the -next).

So, care to try again with a different fix?

thanks,

greg k-h
0 new messages