From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755074AbdERJaE (ORCPT ); Thu, 18 May 2017 05:30:04 -0400 Received: from mail-sn1nam01on0058.outbound.protection.outlook.com ([104.47.32.58]:31664 "EHLO NAM01-SN1-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1754202AbdERJaA (ORCPT ); Thu, 18 May 2017 05:30:00 -0400 Authentication-Results: spf=fail (sender IP is 192.88.168.50) smtp.mailfrom=nxp.com; nxp.com; dkim=none (message not signed) header.d=none;nxp.com; dmarc=fail action=none header.from=nxp.com; Message-ID: <1495099794.26925.18.camel@nxp.com> Subject: Re: [PATCH linux-next v2 1/1] spi: imx: dynamic burst length adjust for PIO mode From: Leonard Crestez To: Jiada Wang CC: , , Mark Brown , Octavian Purdila , "Fabio Estevam" , Robin Gong Date: Thu, 18 May 2017 12:29:54 +0300 In-Reply-To: <591CFDEA.1080204@mentor.com> References: <1493634704-3545-1-git-send-email-jiada_wang@mentor.com> <1493634704-3545-2-git-send-email-jiada_wang@mentor.com> <1495042325.26925.15.camel@nxp.com> <591CFDEA.1080204@mentor.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.5.2-0ubuntu3.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-EOPAttributedMessage: 0 X-Matching-Connectors: 131395733982178657;(91ab9b29-cfa4-454e-5278-08d120cd25b8);() X-Forefront-Antispam-Report: CIP:192.88.168.50;IPV:NLI;CTRY:US;EFV:NLI;SFV:NSPM;SFS:(10009020)(6009001)(336005)(39860400002)(39840400002)(39400400002)(39410400002)(39850400002)(39450400003)(39380400002)(2980300002)(1109001)(1110001)(339900001)(377424004)(24454002)(199003)(51914003)(189002)(9170700003)(50466002)(305945005)(85426001)(2950100002)(103116003)(229853002)(6916009)(50226002)(54906002)(53936002)(4326008)(8656002)(5660300001)(33646002)(93886004)(2906002)(2870700001)(104016004)(81166006)(8676002)(8936002)(5820100001)(50986999)(77096006)(356003)(106466001)(23676002)(47776003)(86362001)(498600001)(36756003)(38730400002)(110136004)(6246003)(76176999)(189998001)(105606002)(99106002);DIR:OUT;SFP:1101;SCL:1;SRVR:DM2PR0301MB0909;H:tx30smr01.am.freescale.net;FPR:;SPF:Fail;MLV:ovrnspm;A:1;MX:1;PTR:InfoDomainNonexistent;LANG:en; X-Microsoft-Exchange-Diagnostics: 1;BL2FFO11FD007;1:DYNDOAhFUtpiEhsJ54DyiRzCEG1q3CDmsRajXnMNN7D+OrYAWcDWn/TbBr1TGBYGw4xYh5Z8gJ1BHvxv8sXuaBtJGSNU73iw65w7Rhcs00xDnRHjEIkUhjdhp8taBfd1eVBuOsdu9Go8TSTPz1RqhouMeI8Qncpid4fC1uBmu8XJ3OMvT+iwLDC8p8FvsyzPWOpAJa2FquRT399Pv7O5/VZAG6rFLnQUPGvpKjpKX0ST98x3pelZHyy/KsU2vLv75Efqb7+d2+VeYLnLBYzYHvF27JnWPfMqe4L1bnNevfw6OwpN6aX0m7sKKxIWfIJ9CZeR+7/+X9wlEhAoIVcJyroPUHPCF52gX/vJqWWtFp2703MlY1ZrVXSzZj5eIeU23E4RJRucdzZcRW5IyJ+DD1vaEMlruMeozVJ4J/s2rmxCqhAgvDNSsIaEG2ZLbhBowA4QD7ovw6yjUVOZdtGXUVrXIskrGV4vwtYVsdQqlq3CJWbNiwHa7cPFrm/+1qa3D8NltZW5FoHVUfkjXzaIGZ1TzRdwSX5Y7YaCw6/V269R+qAXup9PszcXEtm0R0Uy+ptz6gCzQ3iFCqlBls6Njb4dpp/lFiDJtCHTEGdYthfPB0tmWiWRqHBt1sWuKZ8VcMs6PAQFcNyhWjNtTMtyydunIXnFE7savpagf71fQ5uP9SFttBeBvyBtBcNbB5yi1gAh9P8LJV03CWcUkVWh57eIoQ+KeFcy8RyAGALNE/g= X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: DM2PR0301MB0909: X-MS-Office365-Filtering-Correlation-Id: 6b80c848-e05f-4814-6d61-08d49dd07379 X-Microsoft-Antispam: UriScan:;BCL:0;PCL:0;RULEID:(22001)(201703131430075)(201703131517081);SRVR:DM2PR0301MB0909; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB0909;3:ctOJ0Bmj4Cxom0jhD7pLhaojGvWjjZSC/xz+atyUMGpBZby7gpRklbWV8GWN6AB8yn1UUxrUCm/IxgTMcVTkF9Uccect8YV2vcqcRTMCK17kVQ8D08ZULAyckm7TS0+rw3/oSVHs+5vvhxjv1sYoo3ifFLz5bwYb9X5V8h0lp3+6wzxk8eUvx2mEi0oFpfp58+j2vTMhXHR3xubkWs4IAd6mniK1z3oLkUAU3zqk4f3zU3sqcghVdgtUkXO/9oXlOagsaJp6P1LegBUaesZKmYItOQbgu/DFwtAu+tttZZJYOXnFJ6kD0B9Ek+HfFdyBULPSISITTf6K3Wyq8iI8m/yYbPy44bVGdTH35rNAS/8UyWpIa3ww5BI8zwToPfpNt3JJorl9JrS+Lqh59wefZPUn5B4mBFRpYsa1r++qLpze8ILUCNlln1XJ8LgD/nzz;25:q5cmccvK24JLUA52u8qDjArjJvqonFI8mdwgWr3q60XJDw73T2mcZlGaQR5OGezOg6qblXq/ZXMFgywqIRgmKQIvtmYCGTHPu4YLEWkZl/7aeScSISD6AfxmeICa7ao/2CAJbj7rsKq+f7wHEQDOTu79PA4rN5g6tiREpVc40u++x0FC3Kt2VNd9aVUOybTCH2k8YnzmmUfRi85HeK1ktUwdHLDMggLbyRN0kKhu4gieTw3z1jTbgIevR8mDJ6mSho0u5oIpOUOAp+2YxVI2FjMRcvV3IdJVtHbLJPgh5a4Ju0JZo4yWyr+ZvmGC0lvKoPVSIdCfTBniMHrggoIACLltt+q6PzfH8Ep0TvgOUysGSCqor0hKvOxXbfxvlv9xbHf99tydfwacqwsMzDkIFmnwYtdbXGMcD6J33MCZ1mDiem02nvshP/KL5b4CMYAJdl8WJiImOo+BXtU4nGRpI0vbIobVpgajDdaFYwRgEdk= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB0909;31:b7pd6yDWl7b2GYDN/O7JRvtonHcD+x+W7QAr9OMYhv8l7zJdySmvuFwghlbwJvR/v9FGMJvJgLYfRoYT/cVvPiWXk7CeRuw5o5yjQ4ZhFgIUS45ToPs+neRkrqkEhTaUBsETt+4e1X2v1Ub/G3Ahoceu3vdUHkZwYp92rg3VPTyTJHpkFaBfJ5nQ9RZ8qkhWwCTMSo1mhwISQJkEi5dVEy3eL7a9iiRKWnGulphNWSyQhuHsut1mWmdqi+0QMXDlMYTutEJWPWbSB5Fn8AzI8g== X-Microsoft-Antispam-PRVS: X-Exchange-Antispam-Report-Test: UriScan:; X-Exchange-Antispam-Report-CFA-Test: BCL:0;PCL:0;RULEID:(6095135)(601004)(2401047)(13018025)(13023025)(8121501046)(13015025)(5005006)(13024025)(13017025)(93006095)(93001095)(10201501046)(3002001)(6055026)(6096035)(20161123565025)(20161123556025)(20161123561025)(20161123563025)(201703131430075)(201703131441075)(201703131448075)(201703131433075)(201703161259150)(20161123559100);SRVR:DM2PR0301MB0909;BCL:0;PCL:0;RULEID:(400006);SRVR:DM2PR0301MB0909; X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB0909;4:OcTlGsrOddFjAOAkpQX+25XvOvqrhQn6M5bEi0D+vvCvurLXGsMfV1HS/h7UE6RDFFtzxOu6EhcWVqXooDWcU0XAdXYl4UUGWv0Ff9PCUgcHpIlNpbHQ85Kl6i2w73TFy1+QaSgG7MygKdxLQ3BVF4NQKM4N9/SBk9c8rqpbR8GHiUEWuoFjlQA85R/y0rI/lhGibynaKqqMKSZzwqPoizmZ14yvq6Ycu7ZqvvDzCqBkzgQT+DdVDdbD6HsnipR1bWdiDgwVfBpHkyAGu38kE4lwWCwDL5f3yAst3wiGarQOZSKczXGw36fQuFmd4ag/N44jeEWFGg9xT8U7P58nD1WyRP4rKXlD9PVNY446ayw/j7K8yCvKdKcEoZH8yrt6VNO/www4IUsNnnJlT4PHapInHAS1vuNOjZsKxKrPp6HMVyK55G4ljXNNLG8TcWNMeAAvxwxgikotIC5kkvuABQLpzM1re3R5v8A9q+h7TFHsKVJ+W64inEE4VVqH78iCSeQ0/L6ihZ/mg9lS/nR/jjNIWt/k8Y6F0s0gWYucNkBxpdLLyr4onHXAHiKJDQLhWQeRQPbe31u26uySg24eltM62mu6fff+puLkCrZEtkEyF5GNbehdYNGc/+cBetv7FnbQHZ3zmqNaIGrzmkWWVn8h9d5cB9t0+zTte1IChROUdIxtQjkHQ44m89177co4e/c0J2ejNLgki1Dq3zwZleFJWaf1fUpsShU6NJ2Oo8kshqQz35b/G8sgXZSgqT7l7EK8zmAeTGSmizZImlm8pk3uo2QkKgaWN7Ou+bjyGY4iRFXP9MVeOYb/3mWEtSdAGpQuGFjsC4NyRBC1P3fTil9BqHRUNlCx19/B/dxdXwQjFwwh0oNmwytTlUl0ahfyGbkNSTtq02YeCmDFOtSkofL5g6ywC5w5zUFC1xpiOvE= X-Forefront-PRVS: 0311124FA9 X-Microsoft-Exchange-Diagnostics: =?utf-8?B?MTtETTJQUjAzMDFNQjA5MDk7MjM6ZVRJV0RROWtZb3pOakwvYVpSUFp6M0xC?= =?utf-8?B?QzZ4eGoxY0pkOWg1ZVJ1WE1JNG9XZFZUZ3A0MmJYYjFMV2F3aGJ6ZSszOUxu?= =?utf-8?B?WFdIanRqWDhSQlRhbXZ6ZkVDbk1CMnJyeFAyUlFpY1k4UHdVaVV4STJ1ci9G?= =?utf-8?B?R09EcjFBUURNN2hKbi9zam1iL3hYN0ZDaGRHQkJoRDV0Y0dzYXp3aFhkS2cy?= =?utf-8?B?Z3RNRjJqcS80ZytZWjdvUUZZMXExYno1dDMyQVQ0Y0szSE5DNExNSTQxeDMv?= =?utf-8?B?bFdVNEVtK1ZZUUNBWlNoSGFMSnRHMEVIY096Nnc4eDJ6QnZiUlJiMFZRUUxJ?= =?utf-8?B?MHQ0c3hxSks5cCtuQlc1S0RCNDhjWHNIWUNWU1Bsemt4Q0F5eW5BRHN4Mmhn?= =?utf-8?B?dW9Xcko2VnlTdnNFTUk3ZlZzVHhkVEVhd1dtUHZHK0hGSURhaUFZRHJZclR4?= =?utf-8?B?aFF6NkdnckY2dS9oL1EyaGUvK09UTEN6QldIanI5dERoWHlUWGlxOGtLZkIv?= =?utf-8?B?ZEdNQlFDdDhLcGpsR0o4aVB5RzUxY2ZUTC96cTFvWUx3eXFjNjJVS0VIMEJY?= =?utf-8?B?cksraEJXTDF5QUJqUVczQ3lWamxKbUlweW1PT1BqaEJyNThERGZEd3NzNG9D?= =?utf-8?B?bGtzWDdESUt0aThFcThidGN4QnlUcStGTzFQcnBpWjNrYndYSUU5QlNzOS9E?= =?utf-8?B?a3RxcW96bFF6OGdFUHJoWGJRUUFiMFlkUFNOU1lmaFpLWFZhOEc0UHhyclY5?= =?utf-8?B?bFlEa3ZVMk40dFFIelJBOEI1b1BhNnB4a0o5Wm1pL1lqdG1DYks3ZkRSaXJJ?= =?utf-8?B?eFE2cFFVc1dkR0FYVWhBRjRwOVhHNUh3eVU5aTFJRjg0MEtGc01jaTlVR3hU?= =?utf-8?B?Z2tFeVM3V3MrSFE3VStpSDBWcnVXS29xQlRoQVBlK3o0dENEMXhCbFd2cWl5?= =?utf-8?B?ejVSN3FrYWlTZDh1MVovOXp4REtOK1V3OGFGODV0R3ROZFBGbjJRSXl0Mjdm?= =?utf-8?B?aDI2eTFBdU1FVGNqTzU0WURVWFR5TXU1MGJJVm1xMlZPVEJmVmNWdWJyenV6?= =?utf-8?B?bDV4ZGpLRVFhUU0vSFJjWUltODN4UWRSQWJ3OGRINDBqdHhSd2hObmFlQzd5?= =?utf-8?B?SXdYY2FVMW0zS2NwejFkVjkzbW8yWTU5OUs4THg4TlBDN3JVUnU5QXhCU3hm?= =?utf-8?B?blVtR0RpbktoY21XU2x3dExxemdlV0xCODBCb1RFWGpoQ282OHAyTGNVLzMz?= =?utf-8?B?V2o2YUIvdS9XUGl6M2J6aU84ZUhmVUhjRW1acUVFQUxHR0NMUUUyWlFlcEZt?= =?utf-8?B?T1UrVGNOUHNLTXVTOVlRSXh0UzFha1V6WHpXVFo2RVpUMzQzMDF5TG9EZkNu?= =?utf-8?B?cmpibnpSd3ZJNlhRTzNHVDB2Z05ETXRMOUZiUi8rVTUyY0dLZUtyUU54eEhI?= =?utf-8?B?dUNoNk5BUGhlVFBmaGhLY3BkKzRSdnB2S2RSd1dHZ3hzSVAyeVluUDRhZFpK?= =?utf-8?B?YzN6YnR3U3pYU0dVVzkzQVByN1F6NzdwOE5WYU1RbkhsM3ZEdHZNNEtjcWRs?= =?utf-8?B?bXlnSVRFczRzU2ZTMUtmdnZoS29WcGlTSURISk9aTHNDQndnaUFJa1FjaWVN?= =?utf-8?B?UGxoeFVOR0lkMVNqZG9sQ3dkdG1hQ09ZSkdoTmNrSGM3bzRvU2dHcWJsajVH?= =?utf-8?B?Ly9yZ2dOZ0p0VGZVbVNYZnk2TUtkbkFJNzRXd0JNazRmd0dhU0pOUWlTOHRv?= =?utf-8?B?UWVPK1pZYm1FYlVoV3dMczNSL3Zad3Q0dE1IUW16U282R3ZWMlVTZUliSFhJ?= =?utf-8?B?RTI4SUs0T1hnNHJVT0RwOVNLNnZoSW1ZN3N5cnZVTXNkNHJHWVVLREdhekRM?= =?utf-8?Q?OqBmp62VzMRXx4Y9DsLWGw2abiBXr/zbfE?= X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB0909;6:yiKof3pUaoy5opVeotJCT7t7u/t7tZe9wDW7wzsjIcA2J+1X5bATlpB2YF54toR3BSqS0hdbtTt8QBMfjmrHMztP11diK4iUFs0Kaj0LlQfZAihbW0KMcjhIcJBR8WFOFiE6/R/nabWD8WnJZ/le7qwbsMO9pDpeP+xAWEc9LzVlfHKIEeAXEpzmVh39lT8bimFFfm4FNHTrNEIjDcu8HgVEWdeEXPW+Bv/WxEyKulT4753Sl4B2dfq9ArIII+pNGtRMZd63L+a6eB9zNifdCFnZW07pM1YJS6lubcg/EmdzUebBrFydTK2Csj8Dw0KcKn3FAYgFaRnCQKvis2CNxswaycyqjkmbfVg1cYiu/f5aM1D/ZMb4LihNG4uloauNcZTfG5PNGIFYxCDstlJ+1j/NpUKYFuvp/nJro8bCIr/KNG/q2OEx+xjFeg8EZ3JEwoKFpBRSknNDUtL6sFoGZq5qUSicE48cpKtZdN8+MWwY0TOI69GmQJZ+gtRmIZz1MjR/Rm6X8aA1RshC/c85bQ==;5:LYZOJbHLKjmpkf/InDm76VDvO52MqN3G2QjlcYLC8opeRxsXOWljkX7f8OH/2wXcCpJ4FC59FqTVcdI4BF8DbScQ4HEMWatNe1ywEsEyL9aga5HNhJRfRBnEKAb2YfNx6dqW/I4ngHVIE9QTUKhpgNzxnmdUbsjdXf28xWGIAuKb2E4ORzofzld5N2QMeIOP;24:+rYUY0Z6jbGNwzeawwpeo2xnaYEnJId2ZyPnjwfR8WmyNUwVIjyb2NIP2MwP/bZpLBCXOhbYCNFNNgvHKZo4/KaYZThkWE4++82oeHQdEq4= SpamDiagnosticOutput: 1:99 SpamDiagnosticMetadata: NSPM X-Microsoft-Exchange-Diagnostics: 1;DM2PR0301MB0909;7:r36HBJtcFigU8cSIRy99QblcqtmWGvlizKeoI9TyQ7HKMkMkySL7TdVlRRn+PueMaDT9BCI72a+01HoccypKnsYOun+IqwsvTNc7gwaTmjgB8YyIm35eaoJX1lqDi4Y4uT6fdwr5LAFFjH3srjuWXg3qPNTm0eILSPoeAQ7Ho5TOU017w+EDg8Q4aiSxuB6L4yUPgB3r4wiCimtiTKhsZt3bsmfQhHQJO3GLoyBrlGL64o81d6DfGFWaZjPaW/aGn5q5mQNNX5CanicVbaO6wGZVzqz2IBXOfWGQPf/CWrpuyEXV8F5wWuL/NJ0xxdxlzpcwLtyj1MikpUI08Y0NRA== X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 May 2017 09:29:57.8122 (UTC) X-MS-Exchange-CrossTenant-Id: 5afe0b00-7697-4969-b663-5eab37d5f47e X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=5afe0b00-7697-4969-b663-5eab37d5f47e;Ip=[192.88.168.50];Helo=[tx30smr01.am.freescale.net] X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR0301MB0909 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2017-05-17 at 18:50 -0700, Jiada Wang wrote: > Hello Leonard > > Thanks for the report, can you help to check if the following change  > address the issue? > > diff --git a/drivers/spi/spi-imx.c b/drivers/spi/spi-imx.c > index 782045f..19b30cf 100644 > --- a/drivers/spi/spi-imx.c > +++ b/drivers/spi/spi-imx.c > @@ -288,6 +288,9 @@ static void spi_imx_u32_swap_u8(struct spi_transfer  > *transfer, u32 *buf) >   { >          int i; > > +       if (!buf) > +               return; > + >          for (i = 0; i < transfer->len / 4; i++) >                  *(buf + i) = cpu_to_be32(*(buf + i)); >   } > @@ -296,6 +299,9 @@ static void spi_imx_u32_swap_u16(struct spi_transfer  > *transfer, u32 *buf) >   { >          int i; > > +       if (!buf) > +               return; > + >          for (i = 0; i < transfer->len / 4; i++) { >                  u16 *temp = (u16 *)buf; > Yes, this does seem to fix it.