Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1586070 > unrolled thread

[PATCH] staging: xgifb: function prototype argument should have an identifier name

Started byArushi Singhal <arushisinghal19971997@gmail.com>
First post2017-02-22 12:20 +0100
Last post2017-02-25 07:40 +0100
Articles 4 — 3 participants

Back to article view | Back to linux.kernel

Contents

  [PATCH] staging: xgifb: function prototype argument should have an  identifier name Arushi Singhal <arushisinghal19971997@gmail.com> - 2017-02-22 12:20 +0100
    Re: [PATCH] staging: xgifb: function prototype argument should have  an identifier name Greg Kroah-Hartman <gregkh@linuxfoundation.org> - 2017-02-24 17:40 +0100
      Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype  argument should have an identifier name Julia Lawall <julia.lawall@lip6.fr> - 2017-02-24 17:50 +0100
        Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype  argument should have an identifier name Julia Lawall <julia.lawall@lip6.fr> - 2017-02-25 07:40 +0100

#1586070 — [PATCH] staging: xgifb: function prototype argument should have an identifier name

FromArushi Singhal <arushisinghal19971997@gmail.com>
Date2017-02-22 12:20 +0100
Subject[PATCH] staging: xgifb: function prototype argument should have an identifier name
Message-ID<tdD7I-R8-7@gated-at.bofh.it>
function prototype arguments like 'struct vb_device_info *','unsigned
long' etc. should have an identifier name.

Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
---
Changes in v3:
  - By mistake one irrelevant line was added which is removed in this
    patch.
  - write the changes done in previous version in correct format.
  - make the commit subject more relevant and accurate.
---
 drivers/staging/xgifb/vb_setmode.h | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/drivers/staging/xgifb/vb_setmode.h b/drivers/staging/xgifb/vb_setmode.h
index 6f082a7a5a4a..c6317ab00474 100644
--- a/drivers/staging/xgifb/vb_setmode.h
+++ b/drivers/staging/xgifb/vb_setmode.h
@@ -1,14 +1,14 @@
 #ifndef _VBSETMODE_
 #define _VBSETMODE_
 
-void InitTo330Pointer(unsigned char, struct vb_device_info *);
-void XGI_UnLockCRT2(struct vb_device_info *);
-void XGI_LockCRT2(struct vb_device_info *);
-void XGI_DisplayOff(struct xgifb_video_info *,
-		    struct xgi_hw_device_info *,
-		    struct vb_device_info *);
-void XGI_GetVBType(struct vb_device_info *);
-void XGI_SenseCRT1(struct vb_device_info *);
+void InitTo330Pointer(unsigned char ChipType, struct vb_device_info *pVBInfo);
+void XGI_UnLockCRT2(struct vb_device_info *pVBInfo);
+void XGI_LockCRT2(struct vb_device_info *pVBInfo);
+void XGI_DisplayOff(struct xgifb_video_info *xgifb_info,
+		    struct xgi_hw_device_info *pXGIHWDE,
+		    struct vb_device_info *pVBInfo);
+void XGI_GetVBType(struct vb_device_info *pVBInfo);
+void XGI_SenseCRT1(struct vb_device_info *pVBInfo);
 unsigned char XGISetModeNew(struct xgifb_video_info *xgifb_info,
 			    struct xgi_hw_device_info *HwDeviceExtension,
 			    unsigned short ModeNo);
@@ -18,6 +18,6 @@ unsigned char XGI_SearchModeID(unsigned short ModeNo,
 unsigned short XGI_GetRatePtrCRT2(struct xgi_hw_device_info *pXGIHWDE,
 				  unsigned short ModeNo,
 				  unsigned short ModeIdIndex,
-				  struct vb_device_info *);
+				  struct vb_device_info *pVBInfo);
 
 #endif
-- 
2.11.0

[toc] | [next] | [standalone]

#1587780 — Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name

FromGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Date2017-02-24 17:40 +0100
SubjectRe: [PATCH] staging: xgifb: function prototype argument should have an identifier name
Message-ID<ter4u-2Kh-17@gated-at.bofh.it>
In reply to#1586070
On Wed, Feb 22, 2017 at 04:49:19PM +0530, Arushi Singhal wrote:
> function prototype arguments like 'struct vb_device_info *','unsigned
> long' etc. should have an identifier name.
> 
> Signed-off-by: Arushi Singhal <arushisinghal19971997@gmail.com>
> ---
> Changes in v3:
>   - By mistake one irrelevant line was added which is removed in this
>     patch.
>   - write the changes done in previous version in correct format.
>   - make the commit subject more relevant and accurate.

This patch doesn't apply to my tree at all :(

sorry,

greg k-h

[toc] | [prev] | [next] | [standalone]

#1587788 — Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name

FromJulia Lawall <julia.lawall@lip6.fr>
Date2017-02-24 17:50 +0100
SubjectRe: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name
Message-ID<tere9-2Ov-15@gated-at.bofh.it>
In reply to#1587780

[Multipart message — attachments visible in raw view] — view raw

On Fri, 24 Feb 2017, Arushi Singhal wrote:

>
>
> On Fri, Feb 24, 2017 at 10:01 PM, Greg Kroah-Hartman
> <gregkh@linuxfoundation.org> wrote:
>       On Wed, Feb 22, 2017 at 04:49:19PM +0530, Arushi Singhal wrote:
>       > function prototype arguments like 'struct vb_device_info
>       *','unsigned
>       > long' etc. should have an identifier name.
>       >
>       > Signed-off-by: Arushi Singhal
>       <arushisinghal19971997@gmail.com>
>       > ---
>       > Changes in v3:
>       >   - By mistake one irrelevant line was added which is removed
>       in this
>       >     patch.
>       >   - write the changes done in previous version in correct
>       format.
>       >   - make the commit subject more relevant and accurate.
>
>       This patch doesn't apply to my tree at all :(
>
> Why this is so?

Maybe someone else made a change on the same lines as yours.  Check that
your tree is up to date.

julia

> thanks
> Arushi
>       sorry,
>
>       greg k-h
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF_sihk6_UhgLzUH6
> FVSFMu02Vadp1tjatQft-UJ14SJeA%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
>

[toc] | [prev] | [next] | [standalone]

#1588074 — Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name

FromJulia Lawall <julia.lawall@lip6.fr>
Date2017-02-25 07:40 +0100
SubjectRe: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name
Message-ID<teEbo-3P4-7@gated-at.bofh.it>
In reply to#1587788

[Multipart message — attachments visible in raw view] — view raw

On Sat, 25 Feb 2017, Arushi Singhal wrote:

>
>
> On Fri, Feb 24, 2017 at 10:18 PM, Julia Lawall <julia.lawall@lip6.fr> wrote:
>
>
>       On Fri, 24 Feb 2017, Arushi Singhal wrote:
>
>       >
>       >
>       > On Fri, Feb 24, 2017 at 10:01 PM, Greg Kroah-Hartman
>       > <gregkh@linuxfoundation.org> wrote:
>       >       On Wed, Feb 22, 2017 at 04:49:19PM +0530, Arushi Singhal
>       wrote:
>       >       > function prototype arguments like 'struct
>       vb_device_info
>       >       *','unsigned
>       >       > long' etc. should have an identifier name.
>       >       >
>       >       > Signed-off-by: Arushi Singhal
>       >       <arushisinghal19971997@gmail.com>
>       >       > ---
>       >       > Changes in v3:
>       >       >   - By mistake one irrelevant line was added which is
>       removed
>       >       in this
>       >       >     patch.
>       >       >   - write the changes done in previous version in
>       correct
>       >       format.
>       >       >   - make the commit subject more relevant and
>       accurate.
>       >
>       >       This patch doesn't apply to my tree at all :(
>       >
>       > Why this is so?
>
>       Maybe someone else made a change on the same lines as yours. 
>       Check that
>       your tree is up to date.
>
> Is all the changes being done by someone before?

I don't know.  Just update your tree and try to apply your patch again.
Or look at the code that your patch should apply to.  It should be easy to
see what is wrong.

julia

>
>       julia
>
>       > thanks
>       > Arushi
>       >       sorry,
>       >
>       >       greg k-h
>       >
>       >
> > --
> > You received this message because you are subscribed to the Google
> Groups
> > "outreachy-kernel" group.
> > To unsubscribe from this group and stop receiving emails from it,
> send an
> > email to outreachy-kernel+unsubscribe@googlegroups.com.
> > To post to this group, send email to
> outreachy-kernel@googlegroups.com.
> > To view this discussion on the webvisithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF_sihk6_Uhg
> LzUH6
> > FVSFMu02Vadp1tjatQft-UJ14SJeA%40mail.gmail.com.
> > For more options, visit https://groups.google.com/d/optout.
> >
> >
>
>
> --
> You received this message because you are subscribed to the Google Groups
> "outreachy-kernel" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to outreachy-kernel+unsubscribe@googlegroups.com.
> To post to this group, send email to outreachy-kernel@googlegroups.com.
> To view this discussion on the web visithttps://groups.google.com/d/msgid/outreachy-kernel/CA%2BXqjF91bd_er%2BidvUB
> PZCTYFHaxF6Mgf1oQ6vTz9gmk6ggC6Q%40mail.gmail.com.
> For more options, visit https://groups.google.com/d/optout.
>
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web