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: <b4e780ed-0ab8-164d-e158-ce9450fee705@amd.com>
Date:   Tue, 4 Jul 2023 14:15:55 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.12.0
Subject: Re: [PATCH v1] drivers: xilinx: Fix Judgment of the return value of
 the devm_ioremap_resource function
Content-Language: en-US
To:     Minjie Du <duminjie@vivo.com>, Lizhi Hou <lizhi.hou@amd.com>,
        Brian Xu <brian.xu@amd.com>,
        Raj Kumar Rampelli <raj.kumar.rampelli@amd.com>,
        Vinod Koul <vkoul@kernel.org>,
        "open list:XILINX XDMA DRIVER" <dmaengine@vger.kernel.org>,
        "moderated list:ARM/ZYNQ ARCHITECTURE" 
        <linux-arm-kernel@lists.infradead.org>,
        open list <linux-kernel@vger.kernel.org>
Cc:     opensource.kernel@vivo.com
References: <20230704120458.11125-1-duminjie@vivo.com>
From:   Michal Simek <michal.simek@amd.com>
In-Reply-To: <20230704120458.11125-1-duminjie@vivo.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?UkRaMGVkaFBKaU1TOUYwaENCSWZCL1hIRTI2YXBSRVFMVUttTkhlYTBkeXBx?=
 =?utf-8?B?d3FkV215eXpFSlduK2tzM1JUdTAxUTROVGNneG9oSGszQ2pScnBCd0JGbmhh?=
 =?utf-8?B?alA5TXgzV0VaNTRQeUJrdUQ5c2l0dWFWK08vUzhpMS9kSHYyOVpSTUpPR3Jz?=
 =?utf-8?B?UE12UWs1QmNFRzh4bTFvVlRqUnZ2TlQ0UUlXMjVRME9VN21Wb0lUQ2tMVU8x?=
 =?utf-8?B?Ym1sQjE2M0JiYTRXaDJ2RzVzdDgwWk1kcVlqelhMNHF2c1VrbG5ZK0xEd1l5?=
 =?utf-8?B?TmdrQk11a1JEOEtXL29CK3dtdk9HN0lvSWZlZWpJamY1eWZpK1F5ZlJzajJ2?=
 =?utf-8?B?UXVsd2tDeGJZUlFKS092NlI2T21Idlkram5tR1NqVzhiaU4rL0hsT0c2ekJB?=
 =?utf-8?B?TEh4Mllwa1FHbzRpZWV3QWxqVGVrY0JCUm4xWFdoa2k2SDl4dFpiY0tZYTJE?=
 =?utf-8?B?Sk9kTW9kZmNaZGZTTk5yNzFNbTVqc2Q5NGo4QXBpK1R5ZUVhRlJVOUtQVE1C?=
 =?utf-8?B?R3VmNkNDMFFQbXpyTDI3M0xidmI0c25WL3dwd2NGYXJwWFhTK253eTJrWk1z?=
 =?utf-8?B?YUNqNVNsKy9McElKemxYcEEraWVSTnNtSVI5dXY1dW1uSHBNVkhUeFVaaWlw?=
 =?utf-8?B?Z0hzYUgzcS92T0FEMUkrUm5taTVoWUZOcE1wSzBqMU9XMFlQMDRjSDQrOTgw?=
 =?utf-8?B?a21jaXdDMmEyMnlrcWd6b1I0QlR6MDRoVlFwRXloVEhBYzFJNlpxQ0JjcUF6?=
 =?utf-8?B?KzZhK0tsZk9saXdkUjBSMzI4OG5DeTJWRk9YTjNGd3plRVZRK0xSNWpGS3dj?=
 =?utf-8?B?MkhLN3doaDQwTHNxREFLU043bnp1WUFUdGJ1YktKRGFIVVZJWmsra2ZHMG4r?=
 =?utf-8?B?eE1HRFUyQWNYcjJ2ZFlHUmZNbDZmK3Q1UjJUTmhjdEI5WTNEMHYyTHM4U0tR?=
 =?utf-8?B?YXZvSEhiNjBYcWlhTnQ4ZTdEL3pxWkRhMzRFNmZ0NEN3VE05RUF4R21OSHpl?=
 =?utf-8?B?Wlpqb0Ewdk8wbHpOa2VjNnVZclFsQmYxRm5EK092aEJKRkRNS1pMYStXQXRO?=
 =?utf-8?B?OXFQUW94cjhJNE1YdW5uWkRCTXJqQ2VYMzNSV3BYTFBPdmdSZjZWZHdHUW5Q?=
 =?utf-8?B?SnJ6Y2xud1cyS2FXM2ljcS9rcml3eEl6MTZzdHNHZW8zQTI1WEtFTEQ5bU5k?=
 =?utf-8?B?N0taMmlQanhWS3JnZG5PMHU5TzZvRmtYaVNONTVvbW53bjF0Tkp2azhnbm5m?=
 =?utf-8?B?TDJCbFNPbVpRQU9XVE1rL0FYcEdrZVVZczdJUVZnZWg4Yzh4c3M2UkNDS3kz?=
 =?utf-8?B?MklCbW13RFZTSG5zQjM1cEZHTDJMdElTLzNxZlBqZzVjdEgxT28vUVI4Z1ZM?=
 =?utf-8?B?eU1NY29vZEJOMHROSHVDUGZNSllCOVBPZ2JTYU1Vdzk0VDRTcFNMa2I0VDZ2?=
 =?utf-8?B?MFRBbVFUdFpkQzdCMnNMNEcveHlQTzN4ZU9XWld3eTVyZU5BV0lKNjBWa0Vq?=
 =?utf-8?B?VCtuaytnL2N1K1hPeVYrZ0RqaFdGS2pGS2lmNkg5eDB6ZHlwSEc5QjMyVzBZ?=
 =?utf-8?B?Q2ZxY2hMVmtpb3hxalpkMlJXeDB4YmxIakFsV2llaXVqUnV5dmRkc3RZNjRI?=
 =?utf-8?B?bjU0NnRVQ2s3dUJZNVBjVXlZZXAwcGt1cXR0aWNTVkRodmVjVTJUKzJtc2ZY?=
 =?utf-8?B?Ylg3VG1nQzgzVi9rNUU2aW5NWkgzMVFoOG1WYW4xODNDOEVoUWRmWVEyU2M1?=
 =?utf-8?B?bk9UTnZCSEQvOS9NOFR4Wmh3VEdacmFPWmFaaEdvUHNyNlpNeU9NeGt3ay9Z?=
 =?utf-8?B?WHAzSzhYVzJUQjc3alBSb1lJWk1IT3BzUVc2SWRFaVJLa1Jsa2s3MlVTdE1K?=
 =?utf-8?B?YXYrM09iSHpySXpzU2dBWXBKbTlFYUhSb1lYZW5HcUdPK2JQOTgwSnNueE41?=
 =?utf-8?B?ZS9OMTJzUmswYitGUkx2U01JQnNadDRRZWQ1eEYxWkZGblpYSEw0d1ZaYkc0?=
 =?utf-8?B?MFJkWlNDQVZrTEV3WGZuQWlEZ05Yd2JwL0FhTGdKQ2gzSDFLYTlMYU03V2wz?=
 =?utf-8?B?aVIwYUhIcWJIWjB0ZVc1R1d5ZGJOcXBZTDdDWkpBU1c2N05DOWZWZXA1UUtW?=
 =?utf-8?Q?p+xQH07BDqxLk2Ofmt0cIaGOI?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e67859d5-94a7-4f78-2abe-08db7c886e6d
X-MS-Exchange-CrossTenant-AuthSource: BYAPR12MB4758.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Jul 2023 12:16:01.6203
 (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: u0Snm92dI1NVG3z9zffvI6kTwVLF4TQzZZ8nogc5FbojtFfk2v5yaYtluWHJciK9
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN0PR12MB6224
X-Spam-Status: No, score=-1.2 required=5.0 tests=BAYES_00,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,
        T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=no 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


Start subject with:

dmaengine: xilinx: xdma:

And shorten description in subject.


On 7/4/23 14:04, Minjie Du wrote:
> IS_ERR() fix devm_ioremap_resource function return judge.
> Could you help check it out?
> Thank you!

These two lines shouldn't be the part of commit message.

Also add there.
Fixes: 17ce252266c7 ("dmaengine: xilinx: xdma: Add xilinx xdma driver")

Thanks,
Michal