Received: by 2002:a05:6358:16cc:b0:ea:6187:17c9 with SMTP id r12csp11609954rwl; Tue, 3 Jan 2023 01:51:01 -0800 (PST) X-Google-Smtp-Source: AMrXdXuNpycAhd5ULYq5RFSKNKvRVJO00niuELAtqChqmOj9xOVC4DPO8ennWasI6mvCtHTVCCnC X-Received: by 2002:a62:2744:0:b0:581:140:f3b6 with SMTP id n65-20020a622744000000b005810140f3b6mr28880652pfn.34.1672739461426; Tue, 03 Jan 2023 01:51:01 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1672739461; cv=none; d=google.com; s=arc-20160816; b=J8LNxMfraL0QZufqqOH6701vqIqdNc0W8HsjXX5ukFT/wwDk9JOi3uJHxCz+5Pa0F0 l19SprM6DbMeqKjAc2t+6nrQ1L266T0uMcQfJQMuQkOemOXzfZQ/6rNbdXtRsiHXGGOy mJ52PnEtS4WweTaaagR8caXUosSKrscg6iflOAQTg0vV1bi7fFVYl4b6M6rSemuouKzT g1lvBmQpvwENUf2LjlIMRVztgbiwCc5m8mxsgypb6VDnvsgv9+R/UpA8ReEOJm3mXX2u +AwAY3+uiV/0rtx6wQsCgjc0DAcqe55NLzp5C8T3BecM+ywRDAK1mVl2yHczv1ERL2TV mMlA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=s+N6KD+86oG0++UXYbUpyhAUG0jvAMbglBu26PpGXg0=; b=PJG9hd8kY0dbE6r3ti9uxJoneZddJ5P5TNDVmVcfp7hIAtdTqI4sFN/WVNUrkLWNdV UcdRNEDX/9CLgTl8f0VC/edg70QHKnM9VxUHt3kSPMF4JyDwbF17z6gfrqZ6ROcXg/8q 10fsfNrnSjYGtIrblfoEDX7sFA96sZuPmaBODgKTnNLQn59F2JA91Cm4P2fdkFxVgm// 0a3en2Bw9MFnbmXvmEz8bD4ZXlG3XvDraYgJNLH1MnfK7Fm/nuhLXPXnfBCFx239GUwZ F8n3ILdQQ29okqvIKZsdkujs71lq4jr8JISebZLQW8bsVtFNoO/Kb4Q3f+4dIqMsV2x3 IUWg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=lbc0Jq1S; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id x23-20020aa793b7000000b00574a8619855si28519673pff.364.2023.01.03.01.50.51; Tue, 03 Jan 2023 01:51:01 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=k20201202 header.b=lbc0Jq1S; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233147AbjACJS2 (ORCPT + 61 others); Tue, 3 Jan 2023 04:18:28 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46284 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233084AbjACJSZ (ORCPT ); Tue, 3 Jan 2023 04:18:25 -0500 Received: from dfw.source.kernel.org (dfw.source.kernel.org [IPv6:2604:1380:4641:c500::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6D8BFE0B8; Tue, 3 Jan 2023 01:18:20 -0800 (PST) Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id E2DD261228; Tue, 3 Jan 2023 09:18:19 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4DE2FC433D2; Tue, 3 Jan 2023 09:18:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1672737499; bh=s+N6KD+86oG0++UXYbUpyhAUG0jvAMbglBu26PpGXg0=; h=References:In-Reply-To:From:Date:Subject:To:Cc:From; b=lbc0Jq1S28K22mZGoYVRQ693zlNISLskqNBrRIIcAEn06JIl4Z9AJnMc9yLfxL2/u 9HUZ+fRnKk/2PDE1C6f546lcUWcxxMB5w41nlXv4uuZNif3pFO4ESB+JVJ7WgLUGqW AET90onoKKpK58e6XCqlraEZgxqbyhoFAknP8vh1ZZSmPFciyi01FhPZZ21sloplfx sY52PFBgjX7xScPqARXwK7OSSCYbIxC6d8nQSkBGQYgiwFCgSwhqaWld9yQpdVqqrC CyqCQALt+la0WXF0v6x5pBGlj1sKiYb+dm0AffLXWvuIrBTF2TIxnwcSRibz3e6rnG y/DWoZVycmYXw== Received: by mail-lf1-f54.google.com with SMTP id cf42so44829100lfb.1; Tue, 03 Jan 2023 01:18:19 -0800 (PST) X-Gm-Message-State: AFqh2kp9Mx1O3EdSDnMRAVZLr+KuU2bcVf5TkVOOuJI9LEcKYuLCCElB w4DdScxwx8Ps1CrZgGMmWu4wMpUente63J26UUU= X-Received: by 2002:a19:ad4b:0:b0:4cb:315d:e9b8 with SMTP id s11-20020a19ad4b000000b004cb315de9b8mr924274lfd.110.1672737497327; Tue, 03 Jan 2023 01:18:17 -0800 (PST) MIME-Version: 1.0 References: <20221227040925.1619833-1-sdonthineni@nvidia.com> In-Reply-To: From: Ard Biesheuvel Date: Tue, 3 Jan 2023 10:18:06 +0100 X-Gmail-Original-Message-ID: Message-ID: Subject: Re: [PATCH 1/1] efi: rtc: Enable SET/GET WAKEUP services as optional To: Alexandre Belloni Cc: Shanker Donthineni , Alessandro Zummo , linux-rtc@vger.kernel.org, linux-kernel@vger.kernel.org, linux-efi@vger.kernel.org Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-7.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_HI, SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 2 Jan 2023 at 23:21, Alexandre Belloni wrote: > > On 02/01/2023 11:47:11+0100, Ard Biesheuvel wrote: > > On Tue, 27 Dec 2022 at 05:09, Shanker Donthineni wrote: > > > > > > The current implementation of rtc-efi is expecting all the 4 > > > time services GET{SET}_TIME{WAKEUP} must be supported by UEFI > > > firmware. As per the EFI_RT_PROPERTIES_TABLE, the platform > > > specific implementations can choose to enable selective time > > > services based on the RTC device capabilities. > > > > > > This patch does the following changes to provide GET/SET RTC > > > services on platforms that do not support the WAKEUP feature. > > > > > > 1) Relax time services cap check when creating a platform device. > > > 2) Clear RTC_FEATURE_ALARM bit in the absence of WAKEUP services. > > > 3) Conditional alarm entries in '/proc/driver/rtc'. > > > > > > Signed-off-by: Shanker Donthineni > > > > Queued as a fix in efi/urgent, thanks. > > This rather seems like an rtc heavy patch and the subject line is > misleading. This should be rtc: efi: > Also, I'm pretty sure this doesn't qualify as an urgent fix. > I'm happy to drop it from my tree, but please add a cc:stable so it gets backported to v6.1 at least. Otherwise, EFI compliant systems that implement get/set_time but not get/set_wakeup_time have no RTC at all on any LTS kernel until a year from now, and this was never the intent when we introduced the EFI_RT_PROPERTIES_TABLE.