Received: by 2002:a25:d7c1:0:0:0:0:0 with SMTP id o184csp746381ybg; Wed, 23 Oct 2019 05:24:36 -0700 (PDT) X-Google-Smtp-Source: APXvYqysaRi5/5TQiPtFac8MDTISQs0IP1n1oAstWGdWmt3gQthD1SrDY3KXsRnfeL0QEqCTNjoG X-Received: by 2002:a05:6402:170e:: with SMTP id y14mr37106161edu.82.1571833476073; Wed, 23 Oct 2019 05:24:36 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1571833476; cv=none; d=google.com; s=arc-20160816; b=B6tyCAgJSdapzIYkiwPrSKHKvj8IbcRQxN2M9kyIjKTGqLehydIMABqHPjH9Z8SNKZ xzndRbnkn5Q7D+ABo5jUyyAgwhw5UJS+p3LhhcP2nsprx7dON4mSQE5EYPJ8JGu/V+Vp lWyTkmu/nCBbCua9h5+n03c6e8128J1rRWkPQpbzjYD8MAfAJ6f7dvcPmhX03gNHg1sl LJUHQoC6YMxIwYt8o1DD0dbem4JyezoXSIlnEnRTJzIEWSEBw2Mmtql9YMZl3fApUOfI KDzz462Dcg4/TjivAjrPz2XUeY1qZfzD3JEd96Jk5KoYSuMiYkxpS7dK+ATqId/HVt8p 0kFA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:user-agent:references :message-id:in-reply-to:subject:cc:to:from:date:dkim-signature; bh=2TS9P6vv1/KjpwotibIUiV1WTOJaYFrPVFUeByFlL1Q=; b=Z9MqhELJSL5P02VOybq9kRzBydCZq2cUZ5QtYDQtNft+mE4WkG74lC/bAdzFDckOIB z4IqACJw4B6W+aLL2qiAER8JbBFBoU3OFD2ZPmLr6Hl/ZWLhWgGQnRBFb9xuWy6ygnzq hN5S6pbipvuFHhTymDn2QOuN5hbrdumRBjKwpyby1Az7ocItuRFwdFL7bSYfUXhEDSzo ajwL1+0TmVSauv/zdqE/C/zuRuiKIe4J+zvghDywzbjS8avhVBZMeUGXGXVKn1qfLYCz xFtKDd2SCmx5Xi+CGZ3QV1bUwLjIBK3rawKPKJ5jAqr71TTGgEamDw1sWzSDmUQU1iZ9 T7Bw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@sifive.com header.s=google header.b=mNuisqxP; 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 nl6si12738664ejb.207.2019.10.23.05.24.11; Wed, 23 Oct 2019 05:24:36 -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=@sifive.com header.s=google header.b=mNuisqxP; 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 S1732931AbfJWGAs (ORCPT + 99 others); Wed, 23 Oct 2019 02:00:48 -0400 Received: from mail-io1-f67.google.com ([209.85.166.67]:37718 "EHLO mail-io1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728697AbfJWGAs (ORCPT ); Wed, 23 Oct 2019 02:00:48 -0400 Received: by mail-io1-f67.google.com with SMTP id 1so12136502iou.4 for ; Tue, 22 Oct 2019 23:00:48 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sifive.com; s=google; h=date:from:to:cc:subject:in-reply-to:message-id:references :user-agent:mime-version; bh=2TS9P6vv1/KjpwotibIUiV1WTOJaYFrPVFUeByFlL1Q=; b=mNuisqxPnrBkatCUeMElaoaTrErkbBygAOl6zWPZOAVkX2Stz0GU1mUC12VlrMoIbW vuB6nAAC3k/JQeWJfBTnYtBOOUhZX2mUFquCESHdcxKKbbYvmThayGWeR4D5WTYku5SG V6pKuxz6q2+L15NEIGb3U3UbEUsfFcA27BJUW4ntmGeqAx2kKrTzc8al36BVMB9g4OmS hbibqpfFJ5I/0cR6dBzDFAfcNaX1aVGgfegFIwikFbid0wmroSYHtAVd17W4M+aPFiBz OvDUhUiD/1LX+MjSV3HijvBZRo6c5tD2wTptIJUNY1xBWEMkJL9Ww1tbYqmITjTbotUy S/kA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:in-reply-to:message-id :references:user-agent:mime-version; bh=2TS9P6vv1/KjpwotibIUiV1WTOJaYFrPVFUeByFlL1Q=; b=o61wNnxr8SxuBSyI8+9BoGSyq12ILRRkuu+3kPqVDwVSDte+5zdw4/Fy+DllDv/UpH liGTWVwhQpno48a2mx3fgkJluEPylj64KlXpgxVKdL/DcivbJbYHeUYZEynoRpBOMEPp LF8rTNxI5+BhsBNYwkd9wiH0PKLRpFw1umUHWI0K69K0hUF8d1p0Dc9e46lAYqnm5WRm 1opEWGYRKGdqGC1h6KKuOI79i4zgi7p6d4ONzACfmA6GAMorlZd2EyClj7s2ae6JvduP eEmdvM531EvbxrUeT9yPsGeaRHdBUqKnl3o/Hjnn+HmesmWdieAQfE+WOHOcd6n4AIXA 2Lpg== X-Gm-Message-State: APjAAAXrctNpkwdIEkq+rT6C0HcO9rJlONEaFMwknu0i1qUCwY1fnuoY LrPY4AmkqDciV3/zaEliGIMaWQ== X-Received: by 2002:a02:a157:: with SMTP id m23mr3994445jah.135.1571810447594; Tue, 22 Oct 2019 23:00:47 -0700 (PDT) Received: from localhost ([64.62.168.194]) by smtp.gmail.com with ESMTPSA id b11sm7664661ilr.87.2019.10.22.23.00.45 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Oct 2019 23:00:46 -0700 (PDT) Date: Tue, 22 Oct 2019 23:00:44 -0700 (PDT) From: Paul Walmsley X-X-Sender: paulw@viisi.sifive.com To: Anup Patel cc: Alistair Francis , Anup Patel , "linux-riscv@lists.infradead.org" , Atish Patra , "gregkh@linuxfoundation.org" , "rkir@google.com" , "hch@infradead.org" , "aou@eecs.berkeley.edu" , "palmer@sifive.com" , "linux-kernel@vger.kernel.org" Subject: Re: [PATCH v2 2/2] RISC-V: defconfig: Enable Goldfish RTC driver In-Reply-To: Message-ID: References: <20190925063706.56175-3-anup.patel@wdc.com> <17db4a6244d09abf867daf2a6c10de6a5cd58c89.camel@wdc.com> User-Agent: Alpine 2.21.9999 (DEB 301 2018-08-15) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 23 Oct 2019, Anup Patel wrote: > On Wed, Oct 23, 2019 at 6:37 AM Paul Walmsley wrote: > > > Incidentally, just looking at drivers/platform/goldfish, that driver seems > > to be some sort of Google-specific RPC driver. Are you all really sure > > Nopes, it's not RPC driver. In fact, all Goldfish virtual platform > devices are MMIO devices. Is drivers/platform/goldfish/goldfish_pipe.c required for the Goldfish RTC driver or not? If not, then the first patch that was sent isn't the right fix. It would be better to remove the Kbuild dependency between the code in drivers/platform/goldfish and the Goldfish RTC. If it is required, then surely there must be a simpler RTC implementation available. > The problem is VirtIO spec does not define any RTC device so instead of > inventing our own virtual RTC device we re-use RTC device defined in > Goldfish virtual platform for QEMU virt machine. This way we can re-use > the Linux Goldfish RTC driver. With 160+ RTC drivers in the kernel tree already, we certainly agree that it doesn't make sense to invent a new RTC. - Paul