Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.31 as permitted sender) client-ip=23.128.96.31;
Message-ID: <d4c305b9-00ea-e135-bce6-aa502012f922@intel.com>
Date:   Thu, 21 Sep 2023 15:56:33 -0700
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Betterbird/102.13.0
Subject: Re: [PATCH 5/7] cxl: Fix block comment style
To:     Jeongtae Park <jtp.park@samsung.com>,
        Dan Williams <dan.j.williams@intel.com>,
        Alison Schofield <alison.schofield@intel.com>,
        Vishal Verma <vishal.l.verma@intel.com>,
        Ben Widawsky <bwidawsk@kernel.org>,
        Jonathan Cameron <jonathan.cameron@huawei.com>,
        Davidlohr Bueso <dave@stgolabs.net>,
        Fan Ni <fan.ni@samsung.com>, <linux-cxl@vger.kernel.org>
CC:     <linux-kernel@vger.kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Kyungsan Kim <ks0204.kim@samsung.com>,
        "Wonjae Lee" <wj28.lee@samsung.com>,
        Hojin Nam <hj96.nam@samsung.com>,
        Junhyeok Im <junhyeok.im@samsung.com>,
        Jehoon Park <jehoon.park@samsung.com>,
        "Jeongtae Park" <jeongtae.park@gmail.com>
References: <20230921025110.3717583-1-jtp.park@samsung.com>
 <CGME20230921024832epcas2p4ecfc828890e076024e402fd5ef7d8c70@epcas2p4.samsung.com>
 <20230921025110.3717583-6-jtp.park@samsung.com>
Content-Language: en-US
From:   Dave Jiang <dave.jiang@intel.com>
In-Reply-To: <20230921025110.3717583-6-jtp.park@samsung.com>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?bFVQd2E0dXRGN3lMRE9UWFBSU1JNRVlUdk9ZV1p3ZFdSc1Z2YmhhcG92OFRZ?=
 =?utf-8?B?eWUrdmpzcURNVGZ6K1hoQjZHY3ZQVEtDc3krMkJEQ1NIOGZYUUJUWFkrMXN2?=
 =?utf-8?B?aXBNTnlmVEM4cmNXNEdTRTV6a0d6Y1FaczE3SkNrbWtkU2JUZncxWGJpdjRy?=
 =?utf-8?B?bWw5ZFdxLzcrdi9UUGhhS3FHN1hlNVB6QmpHMXRMTHRWcTI5RGxlME9xVUZx?=
 =?utf-8?B?ZWRUanRkMy9lWGduWUF4WGo4K0E1dlUxT1M4QXBWZ0JMQlhkVTFGdER0TElh?=
 =?utf-8?B?UkRZV3NyOWN4bTU1M01rU3pxMW5GaHZOSVMwaUYrdzNseG82Rnp2M2ZQOGlC?=
 =?utf-8?B?ZnV1bkNMa3hQY2cveE5MUUxmQmhraFJKVmRyWDM3Unl5VEtpcDg5UzRGNEc1?=
 =?utf-8?B?QVVFOW9ON1ZROTI0d2NYTFJGWFhZWEJhaE9WaFhubk9BTXVadjdNM3JsRHJF?=
 =?utf-8?B?Vm11YmM3M1o5c1RQcnpUaWtyREV4ZVpnM3VsYktvZEpOS3V5RzQwOHVKYlRC?=
 =?utf-8?B?WXdlZURQSDl3MC9CcTVkcStZaTh0OHVZWm9pdUxESm14K0Y5YUhYMU1JS0ox?=
 =?utf-8?B?T244eWxteWh0K1J0dGhXUlFmYUdHd2JubDFoM2ZBTDlxenI4cWg1NHlubkps?=
 =?utf-8?B?ajBkNnU2cjl2dGFRTURYZVdsNGd0dk1MbVd3a2pHMmp1OGZGR1VxUzd3Vk1W?=
 =?utf-8?B?a1dPM1k0OGxWN1hGck1aUVJ2VW55c0dmKzNzR1Y3SkdJTHhUYi9LekZyUDVn?=
 =?utf-8?B?M1A1U0J4djdnbERMM0kzZnJkRi9YVEdKQUFHbmtyUjFJMXM1NEJSNEJNVlN6?=
 =?utf-8?B?azVMYUNBUkw2VVBxbThuRzByRmZRajZLdGJMMWJkSGRDUTRHbTVQcTJBZEow?=
 =?utf-8?B?SEhGZm9GcjhhUEhsaEdoMHM1MWRNeDJVTHZrazZwSFN0c09hWTUrVEp5QkxS?=
 =?utf-8?B?TlpjZStZWVhqcGVxV0JvSFJvWnpHWVJMdWZLTXcvbE9BMndyNmVTZVM0NGFN?=
 =?utf-8?B?eXBBejQwRTQ0QmUrVHZNdm1nRWhkUTRqNHBaTFpzaGxmYW5obTF1eDZMUkRB?=
 =?utf-8?B?UnFINE5xSWw0U2VBTmRoM2lFam1UdzFMN1YzRnVGMWFDL3A1a0RXS3I2TDNE?=
 =?utf-8?B?WjQwVlhVcE9kOGtxNTdoMzJWKytVbWt6WTVuSFN2TmVLWC8wMFQ3WUpyc0xy?=
 =?utf-8?B?NkFFWHlMSkJtUndiYUE3VytNV2pOb2xsQW4yRlN6SVQzWVgydmk2NjB5eFM0?=
 =?utf-8?B?d0pmb3NFRWpnOEN1RXlmZm9FZktMQ1h5cmtVMXhwUHNKZU95dnEwdGVLUHgx?=
 =?utf-8?B?ZSt0SDdFTkhXV21QTEgrYTZTZ3VsbXFyZW8vTDRzcU8xYUxJN1J6QXdWYmtG?=
 =?utf-8?B?VExwcDFPejNTUnRrUnVyYzdhL0hTSTlwNkRSNm9qTjU0VmhEaytDN1RRMDgr?=
 =?utf-8?B?aW1JWElKTkJkM3VsWHdEMnhZVWh5K1Q0SFNETzlnbGhuSG1wU2RsU3djYXRL?=
 =?utf-8?B?dVJCMUY1dW5NYzJhYnlvTjlYdUZhN3A4TTVUakJ0OXpQN0lRQ0RLcG1iVEo3?=
 =?utf-8?B?L1ZVV24ycXdMK09FUFhpUDZBL0NsM2dwS2tXUkJNRDRQVW5qWjBPUUJ2N0do?=
 =?utf-8?B?T0Z4Ylc0U3ZPVXBEaXJiRkRxMnBwOWo3cGVyblpGdSs3Z0VMY0w5UlRVdTVp?=
 =?utf-8?B?U0NlK3lpSEk5Sjlqb3FYVDhVVHpYTHFIMk9TVjJ4R092aE9DMFV0WGhXNm1C?=
 =?utf-8?B?eVk4Z3Y1QW5lZ0lmME1JWUU1K1ZFczFTMDFraGxYd2pLdVNDaHJQdDh0dVRG?=
 =?utf-8?B?TDhoSGM1MHJwWDBjSjFZaVljNENkNEFyZXVCRkM3clFMZkY2L1lRb3MyeENm?=
 =?utf-8?B?YUJSVVF3Yk9kYVJSWmxqYjBrREpCSVB0Vkx2dGN4WXVld2ZJMDdrN1krUkNo?=
 =?utf-8?B?OG5uckZDYWxsVkNWcUtTb1FUOUI0WndZLytEUDB5QzgybkRXVW1kT0RLSGlL?=
 =?utf-8?B?K2YrdHBPUFFQVE9Zc0x4bXQvY1pTSjRXY0h6NzYzNUtvSU0wekhlRmE1ZHZr?=
 =?utf-8?B?ck1nR3hCSlU2ZWU5NkNxSEUzQ1dpMXdyNk9hSHNaOExnY3I3MHhMMDJ3cksw?=
 =?utf-8?Q?HmQ7jX2PmO0mMRJWeLBUldFve?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 4c77c326-5c2f-4b83-5693-08dbbaf60294
X-MS-Exchange-CrossTenant-AuthSource: PH7PR11MB5984.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Sep 2023 22:56:37.2624
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: a+WE/uvy/Gg7flU3y+gcCj9jIp0WvxqBVYEU7MWwVyiUX6Q0VeQ5McbKrKHwtlJwWgBdX58rrLBtcqNAzYJQPQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB6988
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.3 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,NICE_REPLY_A,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED
	autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on morse.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (morse.vger.email [0.0.0.0]); Thu, 21 Sep 2023 15:57:22 -0700 (PDT)


On 9/20/23 19:51, Jeongtae Park wrote:
> WARNING: Block comments should align the * on each line
> 
> Signed-off-by: Jeongtae Park <jtp.park@samsung.com>

Reviewed-by: Dave Jiang <dave.jiang@intel.com>
> ---
>  drivers/cxl/cxl.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/cxl/cxl.h b/drivers/cxl/cxl.h
> index 545381355efb..0bbe0c15bfa9 100644
> --- a/drivers/cxl/cxl.h
> +++ b/drivers/cxl/cxl.h
> @@ -334,7 +334,7 @@ enum cxl_decoder_type {
>   * @flags: memory type capabilities and locking
>   * @commit: device/decoder-type specific callback to commit settings to hw
>   * @reset: device/decoder-type specific callback to reset hw settings
> -*/
> + */
>  struct cxl_decoder {
>  	struct device dev;
>  	int id;