From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.5 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM,HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D6807CA90AF for ; Wed, 13 May 2020 12:55:32 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 80D5A206D6 for ; Wed, 13 May 2020 12:55:32 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="GwqWEje5" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 80D5A206D6 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=free.fr Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:References :In-Reply-To:Subject:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=5gciucPPyjCfsd7QeG5asfzefbqUKZFOAnAXdpM2wy0=; b=GwqWEje5PG5jBZ LfnPRP+dVESXUlFrH3EQ6302M1LsO/tDAEAkgTBslZIFurTuVWyaMylKfAbKYNeRlUYQ9eMXjSwoW T24W9YjkBM5oeWeYZsxguLNPy38CF2KiYatGK5URmrx6SrfFpQfUvOnSGTh3b0QnzaqoSmoZ0jJiP AnuAsVBpjHvcLNs/NHCmMf6yoizO8BANZxYr2EBk8mfTzv1QWbbi8T3p88YRNLBi2Rwmnt+SDdEaU 0MYvgdiywed3zHWvJ5AIszqYP6qX/3vK7GGezAkZzhV6cHrnKDh1nGGkZ/uehrW9+fme2klRN/68S xeCLPJnTvAUcytCXhypg==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1jYquw-0000rI-9O; Wed, 13 May 2020 12:55:22 +0000 Received: from smtp11.smtpout.orange.fr ([80.12.242.133] helo=smtp.smtpout.orange.fr) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1jYqup-0000lq-UF for linux-arm-kernel@lists.infradead.org; Wed, 13 May 2020 12:55:18 +0000 Received: from belgarion ([80.214.67.31]) by mwinf5d46 with ME id eCv12200V0gURQY03Cv26X; Wed, 13 May 2020 14:55:11 +0200 X-ME-Helo: belgarion X-ME-Auth: amFyem1pay5yb2JlcnRAb3JhbmdlLmZy X-ME-Date: Wed, 13 May 2020 14:55:11 +0200 X-ME-IP: 80.214.67.31 From: Robert Jarzmik To: Miquel Raynal Subject: Re: [PATCH 0/3] mtd: rawnand: Get rid of the cmx270 driver In-Reply-To: <20200508121041.64f91276@xps13> (Miquel Raynal's message of "Fri, 8 May 2020 12:10:41 +0200") References: <20200429223134.789322-1-boris.brezillon@collabora.com> <20200508121041.64f91276@xps13> User-Agent: Gnus/5.130008 (Ma Gnus v0.8) Emacs/26 (gnu/linux) X-URL: http://belgarath.falguerolles.org/ Date: Wed, 13 May 2020 14:55:01 +0200 Message-ID: <87lflwngey.fsf@belgarion.home> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200513_055516_274886_8D3A4DFF X-CRM114-Status: GOOD ( 17.72 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Vignesh Raghavendra , Arnd Bergmann , Tudor Ambarus , Richard Weinberger , Haojian Zhuang , Boris Brezillon , linux-mtd@lists.infradead.org, linux-arm-kernel@lists.infradead.org, Mike Rapoport , Daniel Mack Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org Miquel Raynal writes: > Hi Robert, Mi Miquel, >> I hope someone still has a board to test that. No, unfortunately I don't have this board, nor do I know of anyone having one. It's the second time I see patches on cmx270, and the question to whether we shoud keep this board in kernel is still in my mind ... given that cm-x300 is fully supported and testable, and no one I know has a cm-x2700 ... Now for your series, I have 2 comments : - dsb() : can you explain the rationale of each of the 3 instances I saw please. - the +2 IOMEM offset I don't like it at all. I don't mind the offset, I disklike the use of readb() or readw() where before there was a readl().. Same thing for writeb() against writel(). The bus semantics are not the same, the alignment is not the same as well (and PXA is very old and doesn't cope well with alignment), and without a proper board to test, I would be very wary to have that change. Cheers. -- Robert _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel