linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] staging: xgifb: function prototype argument should have an identifier name
@ 2017-02-22 11:19 Arushi Singhal
  2017-02-24 16:31 ` Greg Kroah-Hartman
  0 siblings, 1 reply; 4+ messages in thread
From: Arushi Singhal @ 2017-02-22 11:19 UTC (permalink / raw)
  To: arnaud.patard; +Cc: Greg Kroah-Hartman, devel, linux-kernel, outreachy-kernel

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

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name
  2017-02-22 11:19 [PATCH] staging: xgifb: function prototype argument should have an identifier name Arushi Singhal
@ 2017-02-24 16:31 ` Greg Kroah-Hartman
       [not found]   ` <CA+XqjF_sihk6_UhgLzUH6FVSFMu02Vadp1tjatQft-UJ14SJeA@mail.gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Greg Kroah-Hartman @ 2017-02-24 16:31 UTC (permalink / raw)
  To: Arushi Singhal; +Cc: arnaud.patard, devel, linux-kernel, outreachy-kernel

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name
       [not found]   ` <CA+XqjF_sihk6_UhgLzUH6FVSFMu02Vadp1tjatQft-UJ14SJeA@mail.gmail.com>
@ 2017-02-24 16:48     ` Julia Lawall
       [not found]       ` <CA+XqjF91bd_er+idvUBPZCTYFHaxF6Mgf1oQ6vTz9gmk6ggC6Q@mail.gmail.com>
  0 siblings, 1 reply; 4+ messages in thread
From: Julia Lawall @ 2017-02-24 16:48 UTC (permalink / raw)
  To: Arushi Singhal
  Cc: Greg Kroah-Hartman, arnaud.patard, devel, linux-kernel, outreachy-kernel

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



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.
>
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [Outreachy kernel] Re: [PATCH] staging: xgifb: function prototype argument should have an identifier name
       [not found]       ` <CA+XqjF91bd_er+idvUBPZCTYFHaxF6Mgf1oQ6vTz9gmk6ggC6Q@mail.gmail.com>
@ 2017-02-25  6:34         ` Julia Lawall
  0 siblings, 0 replies; 4+ messages in thread
From: Julia Lawall @ 2017-02-25  6:34 UTC (permalink / raw)
  To: Arushi Singhal
  Cc: Julia Lawall, Greg Kroah-Hartman, arnaud.patard, devel,
	linux-kernel, outreachy-kernel

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



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.
>
>

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-02-25  6:34 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-02-22 11:19 [PATCH] staging: xgifb: function prototype argument should have an identifier name Arushi Singhal
2017-02-24 16:31 ` Greg Kroah-Hartman
     [not found]   ` <CA+XqjF_sihk6_UhgLzUH6FVSFMu02Vadp1tjatQft-UJ14SJeA@mail.gmail.com>
2017-02-24 16:48     ` [Outreachy kernel] " Julia Lawall
     [not found]       ` <CA+XqjF91bd_er+idvUBPZCTYFHaxF6Mgf1oQ6vTz9gmk6ggC6Q@mail.gmail.com>
2017-02-25  6:34         ` Julia Lawall

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).