Received: by 2002:a25:1506:0:0:0:0:0 with SMTP id 6csp1801053ybv; Fri, 14 Feb 2020 06:17:21 -0800 (PST) X-Google-Smtp-Source: APXvYqw2NSVW7FOqBQbsJPe1O3elPu0AIY1DAGACrIT+Uhy2avSUVGaTrujFf/xYv9PJcBmqNPUc X-Received: by 2002:a9d:7b4e:: with SMTP id f14mr2427524oto.355.1581689841565; Fri, 14 Feb 2020 06:17:21 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1581689841; cv=none; d=google.com; s=arc-20160816; b=vLq5dqKqxxdAJ1DEZy9c83rueeFAtsD3PGKzZzsYXwIrv5oy+QjwdEXZyU6p+bSt9B OH+0X4lxzkEnuMAkQnP4nwiGd6/AUYS6bEIGRfIEUGSTrEgehe3KrudMCPloe+fLbhC6 S1k/6sMK6vits8nzBxxTe33vXp95S/UyvhIqmw3+ADEQD3J0MwbNoCxE4bv1K3IdhDUo muAGtaaMtwGVWLjSgCQdpghoc4BkaiO9D/5WFBxWkdA2QZ4ycTKAG7QGQE+HknqukSnV eSgoGKu4bR623OgP7Mv3Mx2Q3Vlb/1ptnmuTA+NGmD2pJ6MVK7i+NMN8JA1kNsfSAKo+ E6UQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:in-reply-to :content-disposition:mime-version:references:message-id:subject:cc :to:from:date; bh=lKbi8IJysViCsr0VvZU5CEZDmD/MamAzKfbW+RNk43g=; b=TyEkjOitsshrTojoLPcJGu+BtmNv2VXUdGq9OzTkIMJMsnl/lc9WhzRFiFGf/aRKvP eKhCj3gRxIQ+1sHlUbXGlt3FLSUdADTN/kPBGE5EEEeZmku9WHB8C2FFxC5T4/ate9zK AxXz0yea4uFM8csSAH7uJQK3gSJ6re2JVXqdCUHzDps9E7HZ/ii55Mckc5T3XXHYhz58 CGWk4adoXhv9NEignS58mBWfK69CFAn1WJAXNpOvzlSKAZc+oZ1HqocG436Yg3II4aXw MmHa0i0QPsX8tDTZ27oxsSZdPNPCL+UU6rhCykQOtmxNTl0eeMQZ4tPYLAj+xe8Wk0xa 1BhA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id a194si2858585oib.227.2020.02.14.06.17.09; Fri, 14 Feb 2020 06:17:21 -0800 (PST) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=fail (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729412AbgBNOQt (ORCPT + 99 others); Fri, 14 Feb 2020 09:16:49 -0500 Received: from foss.arm.com ([217.140.110.172]:33560 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726191AbgBNOQs (ORCPT ); Fri, 14 Feb 2020 09:16:48 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id A4E141FB; Fri, 14 Feb 2020 06:16:47 -0800 (PST) Received: from localhost (unknown [10.37.6.21]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 283DB3F68F; Fri, 14 Feb 2020 06:16:47 -0800 (PST) Date: Fri, 14 Feb 2020 14:16:45 +0000 From: Mark Brown To: Simon Goldschmidt Cc: "Ramuthevar,Vadivel MuruganX" , linux-kernel , linux-spi@vger.kernel.org, Vignesh R , Mark Rutland , Rob Herring , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , dan.carpenter@oracle.com, cheol.yong.kim@intel.com, qi-ming.wu@intel.com Subject: Re: [PATCH v9 0/2] spi: cadence-quadpsi: Add support for the Cadence QSPI controller Message-ID: <20200214141645.GM4827@sirena.org.uk> References: <20200214114618.29704-1-vadivel.muruganx.ramuthevar@linux.intel.com> <20200214121145.GF4827@sirena.org.uk> <20200214131518.GJ4827@sirena.org.uk> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="rf72Gf+bfLC8kxKs" Content-Disposition: inline In-Reply-To: X-Cookie: Shipping not included. User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --rf72Gf+bfLC8kxKs Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Fri, Feb 14, 2020 at 02:49:48PM +0100, Simon Goldschmidt wrote: > On Fri, Feb 14, 2020 at 2:15 PM Mark Brown wrote: > > I'm not seeing anything in the driver that removes whatever the current > > support is? Unless it's just adding a duplicate driver for the same > > compatible strings which is obviously a bad idea but at least means that > > unless people enable the driver there's no risk of it colliding with the > > existing one. > It does add a duplicate driver for the same compatible strings. The current > working driver is in 'drivers/mtd/spi-nor/cadence-quadspi.c'. > In fact, the compatible string "cdns,qspi-nor" copied from the old driver to > this new driver is *only* used for altera. TI has its own compatible string, > the new Intel platform adds its own as well. Oh, that's not good - it's adding a completely new binding for the same compatibles which isn't OK. We can transition to a new driver using the same binding but we should be keeping the old binding. If we're moving the binding document around and/or transitioning to YAML that needs to be done explicitly rather than adding a new document for the same compatible. > As long as that one doesn't get removed, I have nothing against this driver > here. I'm only concerned that this will get forgotten. And given that I added > altera guys to the loop in one of the previous versions, I just was surprised > they aren't on CC in this version. Yes, like I say it'd be much better to get their review. --rf72Gf+bfLC8kxKs Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl5Gq8wACgkQJNaLcl1U h9DHoQf+LsSB3ZHCDD01+GU4dangDKO0Y7qLxBcScMbJ4+Xuws5182R7ns4mhTmt wTsTbbf+Y3qfIXqReAzBxs5lRRxl46CdLexDJe/CRPKr+ADYByxAOhYcQ8jtPZzd 9QNw3ftS8JICO81XdPsIH9FdPHbMfj1/H+Js5GhfppYUbsvvrxUI1njQ06mF6sOw oCymlCc+ruUgLSR0fA4FsBrLFmWMJTor8kudFBLupMeJs9F3dB2jgmQbkHmlfm1z X/fJnZRpD7voBp4x2TFfm+lx/m6OBPTLuOCuu6CON9p3ahH4KoSB1YXLl8S/d1Ew YV2Vi7U/k2epOpGy2yjWnN2je8whYA== =ysJH -----END PGP SIGNATURE----- --rf72Gf+bfLC8kxKs--