Received: by 2002:ac0:a582:0:0:0:0:0 with SMTP id m2-v6csp4280968imm; Mon, 15 Oct 2018 12:02:23 -0700 (PDT) X-Google-Smtp-Source: ACcGV61AykQjg6F44qLp0c9a25uMoQUVH+xgE8tpTj2gLPU+GEynC6qGAkdxXLF6H8UTGd8XSk4P X-Received: by 2002:a63:5c1f:: with SMTP id q31-v6mr17267047pgb.452.1539630143006; Mon, 15 Oct 2018 12:02:23 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1539630142; cv=none; d=google.com; s=arc-20160816; b=lWjfBZkrjkxgZ+qJ6JmlxMJogBODgQTyHPiCu4/81+6xM/HYA95Pe9drs2+HDwhNkG xBNRSOV9zRnQ+Zl3X0JA+SpsFPQ2Q8AgiDyAjt+jC37yZ7+E+vWZUwodel6+T2h3joKQ aSFdxAJ6IJyGoWKRfefJJYXFX89R2SLL+cEz+zqK1GWvtf4LQWzaSTxCGNBzE2tIsmfM VOAm5bVc0w5tPle+KGXNklXFgP9E5qLmgL07eJqveTJiFSREG1XJyHklUQUTjlCkrdOC XCOrASY5cGiyHyMS9PubWfkVwK0ODSo4mfLAjIKxGuCnBFZTez1+suKaokuDVZHqySB3 Dywg== 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:dkim-signature; bh=CxEPtub+b+YM8U/o5jsxGq2aJXizKcwFu9tN+Axo+QE=; b=b63M42LWyAKl4m9y8i+B+PaNmrJWzo8R4H+P19u/BMXNZphphQbMUvcUuKBGIwQfeB bhHI6M1GQdb7KFvD+zv0t/lDEQAmNKZ5Eyhkgi2CzmmmQRZsPAZ9p87x7DwnmFYmP4wp GUrclNNmIn4nqPpotRmteQYoHtXi6lihyK8V+1L3tcN8OncACVWDV1/8OqGBejSjWm4M JKcQcteJ2rVx622qF0cWLxUICMrEcelfpWtHcRhor3GpAhNDVgTZw2dyiYhoIMKwIpfy cdEd2pBBw9t1Uph8lGMb54z25ZDtTGufXAt4SG2YJWiwyGZk/XrmDykIoXXHVEmcLE48 LFMA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=tJ2skZte; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id s12-v6si11157065pfk.213.2018.10.15.12.02.06; Mon, 15 Oct 2018 12:02:22 -0700 (PDT) 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; dkim=pass header.i=@kernel.org header.s=default header.b=tJ2skZte; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726904AbeJPCsJ (ORCPT + 99 others); Mon, 15 Oct 2018 22:48:09 -0400 Received: from mail.kernel.org ([198.145.29.99]:49926 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726654AbeJPCsJ (ORCPT ); Mon, 15 Oct 2018 22:48:09 -0400 Received: from localhost (ip-213-127-77-176.ip.prioritytelecom.net [213.127.77.176]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id A5785208B3; Mon, 15 Oct 2018 19:01:36 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1539630097; bh=eiazFKZ9+NMXhr8XNPOQYMwqdtLgfzda3bxx8qyvjI8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tJ2skZteAFjx8e0MeZsX1mxk2Sx0YDVN0zEFR5YeNt1Iaw73DfBlOsfcj2qyL6Pcd MgB6X8i8q7eOm0dE2cOto3FA+8tUAzYJf/6HaXkhA78i4J5a33v4zlKB3eB96h18zR n7JEU03PzR7+fRvi90TukQEsen+yOMyUxpvYF/eY= Date: Mon, 15 Oct 2018 21:01:34 +0200 From: Greg KH To: Roman Kiryanov Cc: linux-kernel@vger.kernel.org, Todd Kjos Subject: Re: [PATCH v3 09/15] platform: goldfish: pipe: Move goldfish_pipe to goldfish_pipe_v2 Message-ID: <20181015190134.GA6700@kroah.com> References: <20181003171720.169953-1-rkir@google.com> <20181003171720.169953-9-rkir@google.com> <20181015183807.GA6905@kroah.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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 On Mon, Oct 15, 2018 at 11:55:26AM -0700, Roman Kiryanov wrote: > > > This is the v2 driver. v1 will be added later. > > > > If you want to add a new driver later, great, but don't change the name > > of an existing driver for no good reason. > > I want our "v2" driver to be in a "v2" file and our "v1" driver in a > "v1" file. I think this is reasonable. The in kernel driver is the "v1" one. Why do you need a totally new driver file at all anyway? And again, can you GUARANTEE that userspace will not break if you rename the kernel module? Also, "vX" is an odd naming schemes for drivers, don't take naming lessons from qualcomm :) thanks, greg k-h