Received: by 2002:a05:6a10:c7d3:0:0:0:0 with SMTP id h19csp573446pxy; Sat, 14 Aug 2021 16:25:21 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzYuULLS5II5u87Dr0dTlbbYW8GertBoQTCkWzhYxaxRI4xfYOV+Tx34hu5QvD72nJHuhdw X-Received: by 2002:aa7:d3cf:: with SMTP id o15mr11155804edr.98.1628983520977; Sat, 14 Aug 2021 16:25:20 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1628983520; cv=none; d=google.com; s=arc-20160816; b=bgys38yc7OrHS7f0dIIc7KHTe2xU2qmmdKZwvpQrn4Z2m4nDg95FPrLTFtG+mNthzS ZWIAvd3dHEvFsCHoeERckBOhbeCqz4nlCV9IpRNHWgDCs2Bgen8dqXWe2QFs+K3L9S3V qsP01qmpxIwkM19DXOZFPRKK1n/MznUsFUpEHrb4ps7LoEsKZeDij+zOHDQMBXVT6Me0 qNLp8vAll5saqgoAU+wHgQOSudCsvAfuov1EPGSb0eqkqizE2fRB8js+ThE2/Ikfouj4 NRfZwOYByE/J8cXWCu0UtrksAeRuTdSTCcdGtP01XhE1zq89PuuDHJ1YbPmOtESfSWa2 7eJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date; bh=uyV0DMI/UfRMUPqLHCZbLrLxsniTUkhn9r0ZgnlWQjk=; b=Nh2Vrcz4Sv9bKmXXNA25NR5EuxxOmlomt4i7XFVu+Um8aH6plq9daS8OodrtLUJskO 6MIqjvgRGWH+Kxm13ynXcTLwAs0QuxFf1I5CihARNhBXEBG/rOMclz6eH2z4OB8a4xpc 4b9AW44b0bVF4mM05PTtOP3n0T77I0gH588Sq1yzEgxXQvPN7XLoLJm9FzysfkGj16Pj OxJmxNJte0Ly17rFaBqlizau1nsrfM7dwwlRj9v1MOQHq0IG5NK7PVS8M8ScJBsPHbci kT2AzRrCbcHGBmtGLX0UCIoK/vAfPsEULM9ju8n5l07Rao8nlIxkPweo/NeOu80PZILO 7sNQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id w19si5194955ejk.259.2021.08.14.16.24.57; Sat, 14 Aug 2021 16:25:20 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234160AbhHNXWs (ORCPT + 99 others); Sat, 14 Aug 2021 19:22:48 -0400 Received: from relay4-d.mail.gandi.net ([217.70.183.196]:51973 "EHLO relay4-d.mail.gandi.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233223AbhHNXWq (ORCPT ); Sat, 14 Aug 2021 19:22:46 -0400 Received: (Authenticated sender: alexandre.belloni@bootlin.com) by relay4-d.mail.gandi.net (Postfix) with ESMTPSA id E15C8E0005; Sat, 14 Aug 2021 23:22:15 +0000 (UTC) Date: Sun, 15 Aug 2021 01:22:15 +0200 From: Alexandre Belloni To: Paul Fertser Cc: Ivan Mikhaylov , Alessandro Zummo , openbmc@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 0/2] rtc: pch-rtc: add Intel Series PCH built-in read-only RTC Message-ID: References: <20210810154436.125678-1-i.mikhaylov@yadro.com> <20210814224215.GX15173@home.paul.comp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20210814224215.GX15173@home.paul.comp> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 15/08/2021 01:42:15+0300, Paul Fertser wrote: > On Tue, Aug 10, 2021 at 06:44:34PM +0300, Ivan Mikhaylov wrote: > > Add RTC driver with dt binding tree document. Also this driver adds one sysfs > > attribute for host power control which I think is odd for RTC driver. > > Need I cut it off and use I2C_SLAVE_FORCE? I2C_SLAVE_FORCE is not good > > way too from my point of view. Is there any better approach? > > Reading the C620 datasheet I see this interface also allows other > commands (wake up, watchdog feeding, reboot etc.) and reading statuses > (e.g Intruder Detect, POWER_OK_BAD). > > I think if there's any plan to use anything other but RTC via this > interface then the driver should be registered as an MFD. > This is not the current thinking, if everything is integrated, then there is no issue registering a watchdog from the RTC driver. I'll let you check with Lee... However, I'm not sure what is the correct interface for poweroff/reboot control. -- Alexandre Belloni, co-owner and COO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com