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;
From:   Sean Anderson <sean.anderson@seco.com>
Subject: Re: [BUG] ls1046a: eDMA does not transfer data from I2C
To:     Leo Li <leoyang.li@nxp.com>,
        Oleksij Rempel <linux@rempel-privat.de>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        "linux-i2c@vger.kernel.org" <linux-i2c@vger.kernel.org>,
        linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
        Vinod Koul <vkoul@kernel.org>,
        "dmaengine@vger.kernel.org" <dmaengine@vger.kernel.org>
Cc:     Sumit Semwal <sumit.semwal@linaro.org>,
        =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
        "dri-devel@lists.freedesktop.org" <dri-devel@lists.freedesktop.org>,
        "linaro-mm-sig@lists.linaro.org" <linaro-mm-sig@lists.linaro.org>,
        Joy Zou <joy.zou@nxp.com>, Shawn Guo <shawnguo@kernel.org>
References: <38974aab-06d0-f5ff-d359-5eedd2f3bb3e@seco.com>
 <AM0PR04MB62895815596165791DD59DD38F4D9@AM0PR04MB6289.eurprd04.prod.outlook.com>
Message-ID: <9fc5afd3-7595-fc56-9b7e-209b0b08036a@seco.com>
Date:   Mon, 19 Sep 2022 18:45:13 -0400
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.10.0
In-Reply-To: <AM0PR04MB62895815596165791DD59DD38F4D9@AM0PR04MB6289.eurprd04.prod.outlook.com>
Content-Type: text/plain; charset=utf-8
Content-Language: en-US
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?M2E0c2pyR3hYb25JbXJIbVlVSXZJT2pjcnJJbWl2bVQ2UkRsMDlGeXdFdTlI?=
 =?utf-8?B?dktDQ1FEWmR2L0xRYjJBc2grc2d3R2hCNkZDTTZUNndUT0hyaitpUWFaL25n?=
 =?utf-8?B?TmRuczZCb1l3TDZncWRaMHJQWTdXOU9KUk5sVnFIaGd2WUhIeEdvdnM0T09U?=
 =?utf-8?B?djBmdUFJWFFtMkQ4T0hmRloxQVVMQnpUajlIa1Q5Ykx1MVBPalZ4VWp0dDNY?=
 =?utf-8?B?ODMxYysyVUQ4dkpXbHNOcGNsVFhiMWN6b1hqVnNUeGc1K0N3QVdVRStTSnVM?=
 =?utf-8?B?bGJLVVRYWXJSa25SajZPTndDYk5PSTFFMzhmRTNKeFk5cCtDYlYyWXQwQWla?=
 =?utf-8?B?RUNpeW9PbHJRbEZua0R2ZERuWWJwVGxwV1BlOFVleGRmeDJQa3dFVit6K3JG?=
 =?utf-8?B?czd5UkJhb1I0QVVXQWxlT01tVzNZcjhIZGNpb1kyY3VmcEhYdWhSeVBObWJL?=
 =?utf-8?B?MW1jVEJjaWdZVlNPUnhDUStHR2NvM3BnRmhUdjdDWEpyR2paeHA1ZTI4MkFz?=
 =?utf-8?B?c0pndDcrYnhKRm4xS3VPZ3dON3BFQ3RDcnJWSXZ6R1RISXhzem03K2RWUlky?=
 =?utf-8?B?Y2xVM1ZJR0hFQXg4RGVQQXFYTWJuSnE5NHgxYUFJVlhRZXZnOWZmQ1lnUmZx?=
 =?utf-8?B?eFhCNS9Zd3RMVENnM3k5dVM4dERvVXg5Qkcrek9GT1AxQVNhdXdobnpmK1pk?=
 =?utf-8?B?Q0xnWi9zbEpYRk8yb1BISEkwK250WExyZmx4ZmFVbEVyS1QvMGFVTnVvc1ZD?=
 =?utf-8?B?Tm81c0c3VFV0bkVUMTVoVjA5YjY5WEZXSjBTZVZ4N3dHd2FpM28zUUJNbHNF?=
 =?utf-8?B?enR3ZmdjWERDNTVINnJDUm1qd2t4di9CWitaSHNsbDhDdmt2TjBCVzJ4MzNB?=
 =?utf-8?B?WlRnL3dCWkpYc0pXdTVRL0doWFlteEdYalhreHpEVVhsL0g1NXg3QkFRZ21y?=
 =?utf-8?B?a1gxSU40UzYvcUR0UG5NTUQ4OXcvU0JFOWUwa3p1QzQ2M0p6YUxwdHdqcTJ4?=
 =?utf-8?B?ejM0eE1mM010RG5Hc0RSZGtUeFdVc2toL01WUldHTVNtSm5neGIwNmFTSUVk?=
 =?utf-8?B?RUdpTHBxeGFCYkNyVDkwUVZSL1BDRytrRzRHdU02dUQ5c2ZGUXF2NFFsR0g4?=
 =?utf-8?B?bDk0bmhrQjdvVC9sUHJPUE8vRDhlUUs2YnJwU2VFdnR5UVRka3BqL0VlWVZr?=
 =?utf-8?B?dTdiUUlYQkZvSHhCc0xyQ0RIR1g0eGRCdmRtd3VWNEtHOTRzRGNXdzZwVnlo?=
 =?utf-8?B?S1IzZ0ExOVpjVDBuZWMxS0lmbkZHZmhhdTFhbkxKM0dQa1ZXRlV5VFhVa3Mz?=
 =?utf-8?B?a2NENFhyQmhNOHR0UFlwa2R4MTZ0cnNhN3VBQmg3TDVqN3ZENzkwaDgxTmJK?=
 =?utf-8?B?Wi9iUml4cE1qcEkxVnZNamJFMlJPUTN0ckwrRG8vcG5DWVVDMFN2dGlPVmhl?=
 =?utf-8?B?SlE0Tnl6NTYzYVpmelp6RHN3S1hZeHkyVHRQVmJNc3hLc1NGMjVQRGg0Zm9y?=
 =?utf-8?B?em4rVlF1ZWRiU01EaDBrTTduMDhWblFHWkpRQXRydWYreWtlckI2RGdHcXpp?=
 =?utf-8?B?ZXBqVWowRHhwV05mT05DZ2krU1U5cG02cUhGM0V0RWYxdDBvSllkQzg5TDBG?=
 =?utf-8?B?MThMTHcyL3VYOXp6TEU4TURRK1F6MVVNZ0RZbUFWMzlkL1FldWowUjZ0SnVv?=
 =?utf-8?B?T3RnaHNDL2g0UlY4SjMvS2FML3hVSkRhMjNTZ0dUdG1rS3l1VXdMK2U5OXNI?=
 =?utf-8?B?QmdLWG1EY0xDWEMzWndrcWlSWmlHcEVXQXlGOU45c0Z3VHFlWlYrV1RrelJj?=
 =?utf-8?B?ZE9KMkFKMjFabnU4YWhaRnZibU5tZVBvR0ZieDZlY1BFUnkxWDRBa0hvdzdP?=
 =?utf-8?B?ZDU2N2NIL1R2M1Zyb1p6RTBHOUpsTHpxc010RlVPeTVlUW05Q2NCNjQ0Vlh2?=
 =?utf-8?B?MjNjU3B4cTRRbWtDdG9BTVROWEsvNzVqWnNxTGpSdW05N2hxYWFKa1EzemVV?=
 =?utf-8?B?OS9lS1pwcVM1ZTJmUzlQczkxQ1VDSTBQcENyc3hVV0VsbURJS013K3dXTGFF?=
 =?utf-8?B?ZEVuZTVjZG9FQ1g1bTVHbDNLcmQ1SXl6VHlmWnJZdFliajBFQXRBYnYxaDBX?=
 =?utf-8?B?QVNGdjdFZVljNExiM3RaMGZIeDRXWVZ3QUV6NzIrWC92alVQa1BhVWRKQVNZ?=
 =?utf-8?B?cmc9PQ==?=
X-OriginatorOrg: seco.com
X-MS-Exchange-CrossTenant-Network-Message-Id: c059e07c-2571-45e8-e5ff-08da9a90a047
X-MS-Exchange-CrossTenant-AuthSource: DB7PR03MB4972.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Sep 2022 22:45:18.2474
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: bebe97c3-6438-442e-ade3-ff17aa50e733
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 0VOcz0pm7KTvQOtaBf+b+pdAsAYZZV+WoCi2PZR5w38xvFSW3lDOqudRgWzv/WWn8PumE4fyWPlr82Vct6ADBg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DBBPR03MB7082
X-Spam-Status: No, score=-3.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        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 9/19/22 6:40 PM, Leo Li wrote:
> 
> 
>> -----Original Message-----
>> From: Sean Anderson <sean.anderson@seco.com>
>> Sent: Monday, September 19, 2022 5:24 PM
>> To: Oleksij Rempel <linux@rempel-privat.de>; Pengutronix Kernel Team
>> <kernel@pengutronix.de>; linux-i2c@vger.kernel.org; linux-arm-kernel
>> <linux-arm-kernel@lists.infradead.org>; Vinod Koul <vkoul@kernel.org>;
>> dmaengine@vger.kernel.org
>> Cc: Sumit Semwal <sumit.semwal@linaro.org>; Christian König
>> <christian.koenig@amd.com>; Linux Kernel Mailing List <linux-
>> kernel@vger.kernel.org>; linux-media@vger.kernel.org; dri-
>> devel@lists.freedesktop.org; linaro-mm-sig@lists.linaro.org; Joy Zou
>> <joy.zou@nxp.com>; Peng Ma <peng.ma@nxp.com>; Robin Gong
>> <yibin.gong@nxp.com>; Shawn Guo <shawnguo@kernel.org>; Leo Li
>> <leoyang.li@nxp.com>
>> Subject: [BUG] ls1046a: eDMA does not transfer data from I2C
>> 
>> Hi all,
>> 
>> I discovered a bug in either imx_i2c or fsl-edma on the LS1046A where no
>> data is read in i2c_imx_dma_read except for the last two bytes (which are
>> not read using DMA). This is perhaps best illustrated with the following
>> example:
> 
> What is the kernel tree/tag that you are testing with?

5.15.40

Sorry, I forgot to put that in the first email. I can test with
linux/master, but I reviewed the different commits and I did not think
there would be any difference. I also cherry-picked the eDMA memcpy
commit, but the dma test passed.

--Sean