Date:   Wed, 5 Jan 2022 15:48:56 -0400
From:   Jason Gunthorpe <jgg@nvidia.com>
To:     Li Zhijian <lizhijian@fujitsu.com>
Cc:     zyjzyj2000@gmail.com, hch@infradead.org,
        linux-rdma@vger.kernel.org, linux-kernel@vger.kernel.org,
        leon@kernel.org
Subject: Re: [PATCH v3] RDMA/rxe: Fix indentations and operators sytle
Message-ID: <20220105194856.GB2872048@nvidia.com>
References: <20220105042605.14343-1-lizhijian@fujitsu.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20220105042605.14343-1-lizhijian@fujitsu.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?psIxBFiZi+I4+QSVZI3vWtFXHZ0bGQt2+g9QBvOsejXXuxo5I0kA45R7GyY+?=
 =?us-ascii?Q?5akvhlgD+xafLofD1kmSte46dNbW8iDBhLRRbAdoXm1hBMoImxOIoeKFRLSL?=
 =?us-ascii?Q?P0dBbFh70Zi/GFPVoMQo8gozzrmXCVmT2MK/TjLd2GT+Irv834/Wm2Zwt06O?=
 =?us-ascii?Q?eYhFpbYjerbOj3ECBa/orV7UY9bAemx1QGR+T9X8M6G0iDcGefGcP23Ryo4Z?=
 =?us-ascii?Q?QdWA46btLbTToLC7dspimpxgVGDSXf4J9RyHcVoPsYbDAP5pELPiiW6eMoEq?=
 =?us-ascii?Q?B9+P2+RPTOHW0CUwBE1FChUm/ELTePU8HZILpfkiXm7087Dv7+D7D2Luggwa?=
 =?us-ascii?Q?LBRgUB96hGr7FtXiEjaHtOcYiShO2QMdNfYbJVuCacG1pJpuv9R2zXG1bLAG?=
 =?us-ascii?Q?9Yih9psR1VWeSggtwib6vZVv7Xwc5D+ZSAQZASLJo5r7gtremNklodzzIZae?=
 =?us-ascii?Q?VM7cDH/V+PCIQmaq9CBNmMjBUCIlMfaYag/XGyvxw3Ua2SswlM5++NLiECLP?=
 =?us-ascii?Q?VH76q6gHe51zJnh7u+UmY0v/0o003yEWU0QeKqXNWGUpWuePVW1evxdgAy3U?=
 =?us-ascii?Q?Myaeq04UfK7EeWVboP+O9mgKUIvYgmMuXvI/M1zuI5GMaBwihuwyjgwe1WIO?=
 =?us-ascii?Q?OPV1WR3wrwwHvS7o5UbcWUUOOlx23XKJEENiClsBqdrH4LVIMo4wUvb5BHB7?=
 =?us-ascii?Q?hgGiIRe1onB7qhRbr+d++FgRK9b9qn1cLq+cCJOdarpDNc7Z+H97CZGihJAw?=
 =?us-ascii?Q?bSY/m5xlyvfsS3F/MxltCtytuHqb7hzUu/m0DeSHfsQiqwsaOAMaTTe7Mw+F?=
 =?us-ascii?Q?Ev9AXeXzIBLYMksfUsu1pGToRSxaRtQgSfTeyY9HxXJtuMJs/e3OE0736KIe?=
 =?us-ascii?Q?z5wi6cOlN67IBfMPngXBerm5b8fschAf06J0dJn0Wwsjv4JTsidSUNFDHw6N?=
 =?us-ascii?Q?6vrWz02bCB5m/eqT3mnlbKnxsgvzSbqSDUd2jpx6o20LXvdcUFXpsOZIiMg2?=
 =?us-ascii?Q?nbK7wm8hY13vQWMhWt683cPXCDPEkEPebAeE8aka4d1tNDOekewE0p2JzCdU?=
 =?us-ascii?Q?qygEYGNufIxrRvadYJ4VEhSSndmgrt9Lnlfmk5xt5eyNCXQLFn7jhIZi48fT?=
 =?us-ascii?Q?P5/46wMfQff45NlVSxtCDmzpp4PO6xeNfTmbr3+wEIYy09z256CgEhBWKhMp?=
 =?us-ascii?Q?gT3uHr55Q/sMORqANle3Pl/W8l4AmsSNpR7OLbcR/YyLW5rDchGJvrkrzAAl?=
 =?us-ascii?Q?8Rmx5/BHj2mlQEY9/9MrOJ/790bvpqOa5tpNoN9+YqiMBB16fjtJi+ekt2E9?=
 =?us-ascii?Q?7VY8JOOq/1p2qQ2zu5qKPjJdSNn5WP8W8smRECGb0zdAr0uJ4QodImpWpJPo?=
 =?us-ascii?Q?D15TuIY5M5q/pTYEIuY0ZruczaZmlTpqJNwIHRVZiU8PyZSIM2+cF+cat4EC?=
 =?us-ascii?Q?oTtKPYIiayDMPl2GgIW/gnQYNWCk3wR/ygkfxvvrejiApyv985Cq55hlujqs?=
 =?us-ascii?Q?ttcH5aMXZSySmuO0FTaFvmmAVWSAOWMPi5o08vSchRQOdJbkfRlU739F17YH?=
 =?us-ascii?Q?T5W2edwD+MsimHrigx0=3D?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: cdb10e72-185c-4207-c6e2-08d9d0846a8c
X-MS-Exchange-CrossTenant-AuthSource: BL0PR12MB5506.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Jan 2022 19:48:59.3225
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: /J58wJggMEphKOsdKv5OzCNZPOrCQu1DnFRI5+DiLVy0lHbvCSSEY9j/dwsEr2/A
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR12MB5031
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Wed, Jan 05, 2022 at 12:26:04PM +0800, Li Zhijian wrote:
> * Fix these up to always have the '+', and '|' on the continuing line which
>   is the normal kernel style.
> * Fix indentations correspondingly
> 
> NOTE: this patch also remove the 2 redundant plus in IB_OPCODE_RD_FETCH_ADD
> and IB_OPCODE_RD_COMPARE_SWAP
> 
> Signed-off-by: Li Zhijian <lizhijian@fujitsu.com>
> Reviewed-by: Christoph Hellwig <hch@lst.de>
> ---
> V3: it's former version is [PATCH v2] RDMA/rxe: Get rid of redundant plus
> ---
>  drivers/infiniband/sw/rxe/rxe_opcode.c | 737 +++++++++++++------------
>  1 file changed, 369 insertions(+), 368 deletions(-)

Applied to for-next, thanks

Jason