From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e23smtp01.au.ibm.com (e23smtp01.au.ibm.com [202.81.31.143]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 8A98B1A05DA for ; Tue, 29 Sep 2015 11:52:46 +1000 (AEST) Received: from /spool/local by e23smtp01.au.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 29 Sep 2015 11:52:44 +1000 Received: from d23relay08.au.ibm.com (d23relay08.au.ibm.com [9.185.71.33]) by d23dlp03.au.ibm.com (Postfix) with ESMTP id 5963D3578055 for ; Tue, 29 Sep 2015 11:52:40 +1000 (EST) Received: from d23av04.au.ibm.com (d23av04.au.ibm.com [9.190.235.139]) by d23relay08.au.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t8T1qLLv65798270 for ; Tue, 29 Sep 2015 11:52:29 +1000 Received: from d23av04.au.ibm.com (localhost [127.0.0.1]) by d23av04.au.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t8T1q7xV022461 for ; Tue, 29 Sep 2015 11:52:08 +1000 Message-ID: <5609EEC6.1010505@au1.ibm.com> Date: Tue, 29 Sep 2015 11:52:06 +1000 From: Andrew Donnellan MIME-Version: 1.0 To: "Matthew R. Ochs" , linux-scsi@vger.kernel.org, James Bottomley , "Nicholas A. Bellinger" , Brian King , Ian Munsie , Daniel Axtens , Tomas Henzl , David Laight CC: Michael Neuling , linuxppc-dev@lists.ozlabs.org, "Manoj N. Kumar" Subject: Re: [PATCH v4 24/32] cxlflash: Fix MMIO and endianness errors References: <1443222593-8828-1-git-send-email-mrochs@linux.vnet.ibm.com> <1443223127-9848-1-git-send-email-mrochs@linux.vnet.ibm.com> In-Reply-To: <1443223127-9848-1-git-send-email-mrochs@linux.vnet.ibm.com> Content-Type: text/plain; charset=windows-1252; format=flowed List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On 26/09/15 09:18, Matthew R. Ochs wrote: > Sparse uncovered several errors with MMIO operations (accessing > directly) and handling endianness. These can cause issues when > running in different environments. > > Introduce __iomem and proper endianness tags/swaps where > appropriate to make driver sparse clean. > > Signed-off-by: Matthew R. Ochs > Signed-off-by: Manoj N. Kumar > Reviewed-by: Brian King Reviewed-by: Andrew Donnellan -- Andrew Donnellan Software Engineer, OzLabs andrew.donnellan@au1.ibm.com Australia Development Lab, Canberra +61 2 6201 8874 (work) IBM Australia Limited