Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <5aed9905-78ed-4c3c-9212-3bb826cd5846@amd.com>
Date:   Mon, 3 Apr 2023 13:02:02 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.9.1
Cc:     yazen.ghannam@amd.com, bp@alien8.de, tony.luck@intel.com,
        james.morse@arm.com, mchehab@kernel.org, rric@kernel.org,
        linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org,
        Abaci Robot <abaci@linux.alibaba.com>
Subject: Re: [PATCH -next] EDAC/amd64: clean up some inconsistent indentings
Content-Language: en-US
To:     Yang Li <yang.lee@linux.alibaba.com>
References: <20230330012057.114811-1-yang.lee@linux.alibaba.com>
From:   Yazen Ghannam <yazen.ghannam@amd.com>
In-Reply-To: <20230330012057.114811-1-yang.lee@linux.alibaba.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?eUdZanlOaXRtekRhOWNLY0tFcE1IRkFTU3QzNHpkdFZHR0pCOGlsODZCY04r?=
 =?utf-8?B?OXZNN3ZXZlloUFVXdStnVjNONTlQQ0JONElWOWNMRVFBSHJ6TTlGNUs0Q2l1?=
 =?utf-8?B?YnZnaHJWY3JEREtsMHZ1Nm5nMEtESUU3UndFUkx1NXZJeWdlR3ZCMEt3SjNR?=
 =?utf-8?B?T2hlNDJwNGViOWs1bVZnK1RoN0RRN2ZWQ2s4NVp6dThYNGxtbkN4eXc4S1lI?=
 =?utf-8?B?clhaVFVJWERlaFRuT01ZWTdua1BpVzdkVkVKVktpbXRYS3ZROUR2UFE3YnZ5?=
 =?utf-8?B?SGd5NlhWOWI0N3RXTGpPczlCUk5tNFNkRFdvNHg5bWdZN0wxcWkvLzhhYUVM?=
 =?utf-8?B?dFFSUVo4Y1MyMjQ4SkpWa1pyS3J1NHJCODVrSHN6ajBTTkg1STZxdWFGRjJN?=
 =?utf-8?B?aVpJaWw1V2pQQTdkZEJTVjBZeXJWeDVVelVpa0htS1AwR1ZIbDFYakxnL1RL?=
 =?utf-8?B?YVMxOUM3S2t2aU5tQlc2cEpiVUtTeUV0RGVpc1htb2FqSWtTUUhjNGdkNkhs?=
 =?utf-8?B?U1FXVmxDd2s0RXFycEt6cXhaQnB0WHU5WGpvT3JocFc3Q2dSeEs2d2Z2YmVx?=
 =?utf-8?B?SFlVM1cvMUYyVEZNSFJVT2hjbk02ckthWGg1WmxCSU1LTSt4NDA2QkFOSjcr?=
 =?utf-8?B?Wks5Y1NDb3dyRjBCeXA2Umh6Z2dBZjFtT2NiTGorYVVzMmVieHJqZUdCU2hO?=
 =?utf-8?B?MS9yUDdROGdvNGlyVm9MUXJqNVhwMENLdWV3dDBkOFc4UDM2amxzZ05QTVp1?=
 =?utf-8?B?aldsMWxoNWQ5ZzRESWZlamE2RFlyUG5LWi9YVGc3SzlnL0NJVURjY1lPM29j?=
 =?utf-8?B?aEVkODJrWmRPWXM4RDNZN0hDd2xDaXl5UWx2aUhmUVAxNGdFNDJ1U1Y5eEpr?=
 =?utf-8?B?eGtweThTWmFveTFCVk9oWnR3SUxNT0pqMUtubDdiZ0dOWFZPN08rQ0dhMWFB?=
 =?utf-8?B?SnlMNmVMVW4yOUZyVGRHRmlVM0dZcHY0NlliRmNQUXJJSUlDWXA2L0xncXRk?=
 =?utf-8?B?OWg0V3ZGbHdCN25mN3kzT1F6Mko0WUJqVUFrYWxKaTJLNlA2b2I4Sm1PT0Rz?=
 =?utf-8?B?UlJ5S3crQkxURUZxWXFFUjRTODdKb2k2N2YzTy92R1NxQ2EzTVpuNW9MYnVO?=
 =?utf-8?B?N3V5cnM3OWpPV2V0VGg1OHFhYmxtV3B5VXFvQ1hyWHR5dWl4cUFsYlZySDRE?=
 =?utf-8?B?RkVZYUdMcTR2Y0NGMUFFZk02U0lramZVbUxtSlpHcm5MRnhROVlFd1ZhK001?=
 =?utf-8?B?Rm1rS0lqejB4ZDlabjRDWXZWV0FTODFyOWNhMHRmdXhRbUFNaW9RQk9DcC9B?=
 =?utf-8?B?OUV1U2FsZmRSVFBzQ0V6Z3VOS0ZDRStwb1UvanV4enBpSDYydFpCZHk1dGhB?=
 =?utf-8?B?QzVnWmtwZUdyU1pwRnZ3RXBiNFZqaXAveEFJWVJXZ1pPQmVwZnZaV3FubmNt?=
 =?utf-8?B?R2I5TkF4eVl0eWVSdlhQS3cvaFhRZG1qZ3AvWHhkT25WT1d6TDAvOU9seGFz?=
 =?utf-8?B?eW5LRmo2dTZjWGN6cURobjZTeEJYMkl0ckE4bEFQUDR5SDhCTTBBaGhoUUlS?=
 =?utf-8?B?dzlzNjlPVGUzQ3Mvb3BBdlo1VVJ2bUltSGNEWVdEb1M0V3BHNkRGZmtRcjg2?=
 =?utf-8?B?cnhvaHBjZFNPbTRZLytFTnYyNWZvUWcxQkhIQXIybkxxY1E1VHpUV3o1Z3Vr?=
 =?utf-8?B?VDdJWmZEd0IxSXBZMjUrd2xwc3g3dnFCbEJ1aWt3NzB0ZVFDUHVGWUh6cEZJ?=
 =?utf-8?B?Q0tyYURzMmIrbTBPNTJQcWdwUWhoa2tHM2JLQlJrTW9LTldlOWRFNXZyVzlH?=
 =?utf-8?B?TStJTGxFN0hYRHcwRXQwb2lFKzJLa3MwMW95NWVpcWVCMGU2SGZ1Uk5ScDlo?=
 =?utf-8?B?eU1UQTM0UEMwTktBRnNiLzRwSitWcmRET1ZsbjRyZWFMaWJTSnJHRlpsTksx?=
 =?utf-8?B?VENFZjhjOW1vZFlHbTNBNmhyRWZUTEFQbWpqQnI5MDNkdmxJcDF5K0lIdFlx?=
 =?utf-8?B?OVdEZ0tmNEhBQmpOeTJidGNpbGtObS9KTVZ3QTJQanFLeCtpZW8wTUlRbUs4?=
 =?utf-8?B?N0l3SmJPb3A1K2N3V25KcGowOE9wSnloZU9EVlREeGp6eWdLS0ZvTWNZUGRh?=
 =?utf-8?Q?Nx/m3T17pcx+XczKu0AE2DkkL?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f2628fe1-21b0-4759-df90-08db346526f4
X-MS-Exchange-CrossTenant-AuthSource: BN8PR12MB3108.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 03 Apr 2023 17:02:05.4928
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: lCLKrnG7DJV5qohyS5dW3+0mj3vLJrHiqr1RZe/z6/Hifah3t0vF0OP1KwgooF022cRb+OMF02vqkPEd19MScQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR12MB5310
X-Spam-Status: No, score=-0.6 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
        DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,NICE_REPLY_A,
        RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_NONE
        autolearn=unavailable autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 3/29/23 21:20, Yang Li wrote:
> drivers/edac/amd64_edac.c:1279 umc_determine_edac_cap() warn: inconsistent indenting
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Link: https://bugzilla.openanolis.cn/show_bug.cgi?id=4639
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
> ---

Hi Yang,

Thanks for the patch.

It looks like this issue was introduced with the following commit.

f6a4b4a1aa16 ("EDAC/amd64: Split determine_edac_cap() into dct/umc functions")

Can you please include a Fixes tag and describe your changes in the commit message?

Thanks,
Yazen