linux-parisc.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] parisc: math-emu: fcnvxf: Add space after that ','
       [not found] <tencent_86762B5284E20421D8A676F4CC1B205D2F0A@qq.com>
@ 2023-07-13  9:37 ` hanyu001
  2023-07-20  6:49   ` Helge Deller
  0 siblings, 1 reply; 2+ messages in thread
From: hanyu001 @ 2023-07-13  9:37 UTC (permalink / raw)
  To: deller; +Cc: linux-parisc, linux-kernel


Fix Error reported by checkpatch.pl

arch/parisc/math-emu/fcnvxf.c:99: ERROR: space required after that ',' 
(ctx:VxV
)

Signed-off-by: maqimei <2433033762@qq.com>
---
  arch/parisc/math-emu/fcnvxf.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/arch/parisc/math-emu/fcnvxf.c 
b/arch/parisc/math-emu/fcnvxf.c
index 6940179..f5f9376 100644
--- a/arch/parisc/math-emu/fcnvxf.c
+++ b/arch/parisc/math-emu/fcnvxf.c
@@ -96,7 +96,7 @@
                      Sgl_increment(result);
                  break;
              case ROUNDNEAREST:
-                Sgl_roundnearest_from_int(src,result);
+                Sgl_roundnearest_from_int(src, result);
          }
          if (Is_inexacttrap_enabled()) {
              *dstptr = result;

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

* Re: [PATCH] parisc: math-emu: fcnvxf: Add space after that ','
  2023-07-13  9:37 ` [PATCH] parisc: math-emu: fcnvxf: Add space after that ',' hanyu001
@ 2023-07-20  6:49   ` Helge Deller
  0 siblings, 0 replies; 2+ messages in thread
From: Helge Deller @ 2023-07-20  6:49 UTC (permalink / raw)
  To: hanyu001; +Cc: linux-parisc, linux-kernel

Hello hanyu001,

On 7/13/23 11:37, hanyu001@208suo.com wrote:
>
> Fix Error reported by checkpatch.pl
>
> arch/parisc/math-emu/fcnvxf.c:99: ERROR: space required after that ',' (ctx:VxV
> )
>
> Signed-off-by: maqimei <2433033762@qq.com>

Thanks for that patch!

Your patch is correct and as such it should be applied.
But please notice the arch/parisc/math-emu/README file (in the same
directory), which states:
All files except driver.c are snapshots from the HP-UX kernel.  They've
been modified as little as possible.  Even though they don't fit the
Linux coding style, please leave them in their funny format just in case
someone in the future, with access to HP-UX source code, is generous
enough to update our copies with later changes from HP-UX -- it'll
make their 'diff' job easier if our code is relatively unmodified.

So, I won't apply any patch to that directory, unless it fixes a real bug
or is required to compile the kernel.

Thanks,
Helge

> ---
>   arch/parisc/math-emu/fcnvxf.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/parisc/math-emu/fcnvxf.c b/arch/parisc/math-emu/fcnvxf.c
> index 6940179..f5f9376 100644
> --- a/arch/parisc/math-emu/fcnvxf.c
> +++ b/arch/parisc/math-emu/fcnvxf.c
> @@ -96,7 +96,7 @@
>                       Sgl_increment(result);
>                   break;
>               case ROUNDNEAREST:
> -                Sgl_roundnearest_from_int(src,result);
> +                Sgl_roundnearest_from_int(src, result);
>           }
>           if (Is_inexacttrap_enabled()) {
>               *dstptr = result;


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

end of thread, other threads:[~2023-07-20  6:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <tencent_86762B5284E20421D8A676F4CC1B205D2F0A@qq.com>
2023-07-13  9:37 ` [PATCH] parisc: math-emu: fcnvxf: Add space after that ',' hanyu001
2023-07-20  6:49   ` Helge Deller

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