From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D050DC433F5 for ; Mon, 11 Oct 2021 12:52:30 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id AF50860E9C for ; Mon, 11 Oct 2021 12:52:30 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234885AbhJKMy3 (ORCPT ); Mon, 11 Oct 2021 08:54:29 -0400 Received: from us-smtp-delivery-124.mimecast.com ([216.205.24.124]:59528 "EHLO us-smtp-delivery-124.mimecast.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231628AbhJKMy3 (ORCPT ); Mon, 11 Oct 2021 08:54:29 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1633956748; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=QaBIGi0oAdZ+bifrwuDRXIfKMEk3gl8Olbtkp3fD4yo=; b=RK4rQP3oqPrHIAKTSVOLe0ezgq5tPiqWKptrxyEmunnS6qKS4M79xyMfCDMSs9UwO5NIP/ W9oP5nv4GGjooc23NivbtrSU77JZBJfpJDGc+luKAQcHZrIBEH1Iqvt0BRp0Lj6DmNJENY Q084Buh9F5/ksZ8zrovs8AXajUH84zU= Received: from mail-ed1-f69.google.com (mail-ed1-f69.google.com [209.85.208.69]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-501-Jw4iijHWOCKYYM6-GKp7AQ-1; Mon, 11 Oct 2021 08:52:27 -0400 X-MC-Unique: Jw4iijHWOCKYYM6-GKp7AQ-1 Received: by mail-ed1-f69.google.com with SMTP id i7-20020a50d747000000b003db0225d219so15863974edj.0 for ; Mon, 11 Oct 2021 05:52:26 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:subject:to:cc:references:from:message-id:date :user-agent:mime-version:in-reply-to:content-language :content-transfer-encoding; bh=QaBIGi0oAdZ+bifrwuDRXIfKMEk3gl8Olbtkp3fD4yo=; b=MWxx9VYGOoIPH03mnn06PR9BN3rDt4GticqPd9Ot47pFxTe5qos3H5g85JxOFw7KXJ Y8QQTBRIg/XwodTWienuxM1TXI1XzsN2KkYlXNmQ+4DM2wevcgN7MNuambMDWd/Bzl8C du8PuDaFgocswW/BVgwIPMX7x+KNvaXeqFBsg6yk3iQ1fm2mtXD9QeB+SxtpMLaHYgxY S9NOkvykQnYN2sIz8szVNfDtHSlXpGHc7CaSb8NxhVMXohChPWq3uxVI8HPbSFhBQBsX yi2hXIbWmnKJ5dLkbgkWpaMnLBhZO+Zv6PV7vOoTcmTEOoR9DgOpk11aOB8FTnwHO+CI XWCw== X-Gm-Message-State: AOAM532YQkW1x4AtADuOHkYiQWMoH8CEL+1IvPlcHPxnNQ6YBA4VLEVp 5GU4/XTv+QxJUItfyOxQHWc9U2zD5xnxhAe39GN5kCdQAJeT1WN7QZewp1X0ki09x36zGtwDiHV uVjuKLPdRLClW98L9ttFW X-Received: by 2002:a17:906:b00c:: with SMTP id v12mr25574521ejy.222.1633956745596; Mon, 11 Oct 2021 05:52:25 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzuB4IMrDwLNLmZu6++TamTcz3ffKC3IdkeBOEaxJk43O3zJ6vyUZCqDugm3PErNzR6Spdtkg== X-Received: by 2002:a17:906:b00c:: with SMTP id v12mr25574491ejy.222.1633956745398; Mon, 11 Oct 2021 05:52:25 -0700 (PDT) Received: from x1.localdomain ([81.30.35.201]) by smtp.gmail.com with ESMTPSA id t20sm3469789ejc.105.2021.10.11.05.52.24 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 11 Oct 2021 05:52:25 -0700 (PDT) Subject: Re: [PATCH 2/7] ABI: sysfs-platform-dell-privacy-wmi: correct ABI entries To: Mauro Carvalho Chehab , Linux Doc Mailing List , Greg Kroah-Hartman Cc: Jonathan Corbet , Perry Yuan , linux-kernel@vger.kernel.org References: <42846621fdf2bf206feb114d06b14cbc47475fb5.1632740376.git.mchehab+huawei@kernel.org> From: Hans de Goede Message-ID: <5e4e394d-6005-e784-9e45-610ab66ce331@redhat.com> Date: Mon, 11 Oct 2021 14:52:24 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.11.0 MIME-Version: 1.0 In-Reply-To: <42846621fdf2bf206feb114d06b14cbc47475fb5.1632740376.git.mchehab+huawei@kernel.org> Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 8bit Precedence: bulk List-ID: X-Mailing-List: linux-doc@vger.kernel.org Hi, On 9/27/21 1:10 PM, Mauro Carvalho Chehab wrote: > As described at Documentation/ABI/README doesn't contain an > Attribute: field. > > The way sysfs ABI is supposed to work is that each different > attribute would have a separate file. So, the right way to > map this would be like: > > /sys/.../dell_privacy_supported_type/mic_mute > /sys/.../dell_privacy_supported_type/camera_shutter > > /sys/.../dell_privacy_current_state/mic_mute > /sys/.../dell_privacy_current_state/camera_shutter > > However, it seems to late to fix that, as this was merged already on > Kernel 5.13, and a change right now would be a regression. > > So, instead, let's at least fix the entry to match the expected > format. > > While here, fix the format of the contact, which is not a valid > e-mail URL. > > This should also fix the current warnings produced when building the > docs: > > Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi:35: WARNING: Unexpected indentation. > Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi:2: WARNING: Unexpected indentation. > > Fixes: 8af9fa37b8a3 ("platform/x86: dell-privacy: Add support for Dell hardware privacy") > Signed-off-by: Mauro Carvalho Chehab Thank you for your patch, I've applied this patch to my review-hans branch: https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans Note it will show up in my review-hans branch once I've pushed my local branch there, which might take a while. Once I've run some tests on this branch the patches there will be added to the platform-drivers-x86/for-next branch and eventually will be included in the pdx86 pull-request to Linus for the next merge-window. Regards, Hans > --- > > See [PATCH 0/7] at: https://lore.kernel.org/all/cover.1632740376.git.mchehab+huawei@kernel.org/T/#t > > .../testing/sysfs-platform-dell-privacy-wmi | 60 ++++++++++++------- > 1 file changed, 38 insertions(+), 22 deletions(-) > > diff --git a/Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi b/Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi > index 7f9e18705861..1f1f274a6979 100644 > --- a/Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi > +++ b/Documentation/ABI/testing/sysfs-platform-dell-privacy-wmi > @@ -1,55 +1,71 @@ > What: /sys/bus/wmi/devices/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_supported_type > Date: Apr 2021 > KernelVersion: 5.13 > -Contact: "perry.yuan@dell.com>" > +Contact: "" > Description: > Display which dell hardware level privacy devices are supported > “Dell Privacy” is a set of HW, FW, and SW features to enhance > Dell’s commitment to platform privacy for MIC, Camera, and > ePrivacy screens. > The supported hardware privacy devices are: > -Attributes: > - Microphone Mute: > + > + Attributes: > + Microphone Mute: > Identifies the local microphone can be muted by hardware, no applications > is available to capture system mic sound > > - Camera Shutter: > + Camera Shutter: > Identifies camera shutter controlled by hardware, which is a micromechanical > shutter assembly that is built onto the camera module to block capturing images > from outside the laptop > > - supported: > + Values: > + > + supported: > The privacy device is supported by this system > > - unsupported: > + unsupported: > The privacy device is not supported on this system > > - For example to check which privacy devices are supported: > + For example to check which privacy devices are supported:: > > - # cat /sys/bus/wmi/drivers/dell-privacy/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_supported_type > - [Microphone Mute] [supported] > - [Camera Shutter] [supported] > - [ePrivacy Screen] [unsupported] > + # cat /sys/bus/wmi/drivers/dell-privacy/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_supported_type > + [Microphone Mute] [supported] > + [Camera Shutter] [supported] > + [ePrivacy Screen] [unsupported] > > What: /sys/bus/wmi/devices/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_current_state > Date: Apr 2021 > KernelVersion: 5.13 > -Contact: "perry.yuan@dell.com>" > +Contact: "" > Description: > Allow user space to check current dell privacy device state. > Describes the Device State class exposed by BIOS which can be > consumed by various applications interested in knowing the Privacy > feature capabilities > -Attributes: > - muted: > - Identifies the privacy device is turned off and cannot send stream to OS applications > > - unmuted: > - Identifies the privacy device is turned on ,audio or camera driver can get > - stream from mic and camera module to OS applications > + Attributes: > + Microphone: > + Identifies the local microphone can be muted by hardware, no applications > + is available to capture system mic sound > > - For example to check all supported current privacy device states: > + Camera Shutter: > + Identifies camera shutter controlled by hardware, which is a micromechanical > + shutter assembly that is built onto the camera module to block capturing images > + from outside the laptop > > - # cat /sys/bus/wmi/drivers/dell-privacy/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_current_state > - [Microphone] [unmuted] > - [Camera Shutter] [unmuted] > + Values: > + muted: > + Identifies the privacy device is turned off > + and cannot send stream to OS applications > + > + unmuted: > + Identifies the privacy device is turned on, > + audio or camera driver can get stream from mic > + and camera module to OS applications > + > + For example to check all supported current privacy device states:: > + > + # cat /sys/bus/wmi/drivers/dell-privacy/6932965F-1671-4CEB-B988-D3AB0A901919/dell_privacy_current_state > + [Microphone] [unmuted] > + [Camera Shutter] [unmuted] >