linux-kernel - Re: [PATCH] blackfin: balance parenthesis in macros

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:	Fri, 16 Mar 2007 12:02:27 +0800
From:	"Wu, Bryan" <bryan.wu@...log.com>
To:	Mariusz Kozlowski <m.kozlowski@...land.pl>
Cc:	Andrew Morton <akpm@...ux-foundation.org>,
	uclinux-dist-devel@...ckfin.uclinux.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] blackfin: balance parenthesis in macros

On Thu, 2007-03-15 at 18:12 -0400, Mariusz Kozlowski wrote:
> Hello,
> 
>         This patch (against 2.6.21-rc3-mm1) balances parenthesis in blackfin
> header files.
> 
> Signed-off-by: Mariusz Kozlowski <m.kozlowski@...land.pl>
> 
>  include/asm-blackfin/mach-bf535/bf535.h |    4 ++--
>  include/asm-blackfin/scatterlist.h      |    2 +-
>  2 files changed, 3 insertions(+), 3 deletions(-)
> 
> diff -u linux-2.6.21-rc3-mm1-a/include/asm-blackfin/mach-bf535/bf535.h linux-2.6.21-rc3-mm1-b/include/asm-blackfin/mach-bf535/bf535.h
> --- linux-2.6.21-rc3-mm1-a/include/asm-blackfin/mach-bf535/bf535.h      2007-03-15 22:25:34.000000000 +0100
> +++ linux-2.6.21-rc3-mm1-b/include/asm-blackfin/mach-bf535/bf535.h      2007-03-15 22:33:09.000000000 +0100
> @@ -224,7 +224,7 @@
>  #define UART0_LSR_TEMT                 0x40    /* TSR and UARTx_thr both empty */
> 
>  #define UART0_MSR_ADDR         0xffc0180c      /* UART 0 Modem status register  16 bit */
> -#define UART0_MSR                      HALFWORD_REF(UART0_MSR_ADDR
> +#define UART0_MSR                      HALFWORD_REF(UART0_MSR_ADDR)
>  #define UART0_SCR_ADDR         0xffc0180e      /* UART 0 Scratch register  16 bit */
>  #define UART0_SCR                      HALFWORD_REF(UART0_SCR_ADDR)
>  #define UART0_IRCR_ADDR                0xffc01810      /* UART 0 IrDA Control register  16 bit */
> @@ -331,7 +331,7 @@
>  #define UART1_LSR_TEMT          0x40   /* TSR and UARTx_thr both empty */
> 
>  #define UART1_MSR_ADDR         0xffc01c0c      /* UART 1 Modem status register  16 bit */
> -#define UART1_MSR                      HALFWORD_REF(UART1_MSR_ADDR
> +#define UART1_MSR                      HALFWORD_REF(UART1_MSR_ADDR)
>  #define UART1_SCR_ADDR         0xffc01c0e      /* UART 1 Scratch register  16 bit */
>  #define UART1_SCR                      HALFWORD_REF(UART1_SCR_ADDR)
> 
> diff -upr linux-2.6.21-rc3-mm1-a/include/asm-blackfin/scatterlist.h linux-2.6.21-rc3-mm1-b/include/asm-blackfin/scatterlist.h
> --- linux-2.6.21-rc3-mm1-a/include/asm-blackfin/scatterlist.h   2007-03-15 22:25:34.000000000 +0100
> +++ linux-2.6.21-rc3-mm1-b/include/asm-blackfin/scatterlist.h   2007-03-15 22:30:18.000000000 +0100
> @@ -17,7 +17,7 @@ struct scatterlist {
>   * returns, or alternatively stop on the first sg_dma_len(sg) which
>   * is 0.
>   */
> -#define sg_address(sg) (page_address((sg)->page) + (sg)->offset
> +#define sg_address(sg) (page_address((sg)->page) + (sg)->offset)
>  #define sg_dma_address(sg)      ((sg)->dma_address)
>  #define sg_dma_len(sg)          ((sg)->length)
> 
> 
> 
> Regards,
> 
>         Mariusz Kozlowski

Thank you Mariusz,

Mike applied your patch to our SVN repo. I will send out blackfin-arch
update patch including your contribution later.

Your review is very helpful for our development.

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