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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 5D12CC433FE for ; Tue, 10 May 2022 10:10:39 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239615AbiEJKOe (ORCPT ); Tue, 10 May 2022 06:14:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:36954 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239611AbiEJKOZ (ORCPT ); Tue, 10 May 2022 06:14:25 -0400 Received: from ams.source.kernel.org (ams.source.kernel.org [IPv6:2604:1380:4601:e00::1]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1D0DF2AACFD; Tue, 10 May 2022 03:10:28 -0700 (PDT) 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 ams.source.kernel.org (Postfix) with ESMTPS id BF4D9B81CA8; Tue, 10 May 2022 10:10:26 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id B7F4EC385C6; Tue, 10 May 2022 10:10:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1652177425; bh=DdPFsUVSv7R67frGX61raO5JCCbtyzeekzE/nESThhU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=IoJjIXCCfHHivO8X0PqX8WAbWdVeWOcgJe8Pw2cjWC1GJNQAqEp3rb2SBqvEZYo3A XzGOdszW5aH8i4v65YuDO5qKM7reUe3SXBCfxbkTWuJoHQnk9TbbTphisu9pAA2fEf 3o8rkTmLwm6xUODtE5n/3P+/dOrtV26GhDBGS5UQ= Date: Tue, 10 May 2022 12:10:22 +0200 From: Greg Kroah-Hartman To: Andy Shevchenko Cc: Muhammad Usama Anjum , "Rafael J. Wysocki" , Len Brown , Hans de Goede , Mark Gross , Benson Leung , Enric Balletbo i Serra , Collabora Kernel ML , Guenter Roeck , Dmitry Torokhov , Gwendal Grignou , vbendeb@chromium.org, Andy Shevchenko , Ayman Bagabas , Benjamin Tissoires , =?utf-8?B?Qmxhxb4=?= Hrastnik , Darren Hart , Dmitry Torokhov , Jeremy Soller , Mattias Jacobsson <2pi@mok.nu>, Mauro Carvalho Chehab , Rajat Jain , Srinivas Pandruvada , Platform Driver , Linux Kernel Mailing List , ACPI Devel Maling List , "Rafael J . Wysocki" , chrome-platform@lists.linux.dev Subject: Re: [PATCH RESEND v11] platform/chrome: Add ChromeOS ACPI device driver Message-ID: References: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-acpi@vger.kernel.org On Tue, May 10, 2022 at 11:33:19AM +0200, Andy Shevchenko wrote: > On Tue, May 10, 2022 at 8:44 AM Muhammad Usama Anjum > wrote: > > > > From: Enric Balletbo i Serra > > > > The x86 Chromebooks have the ChromeOS ACPI device. This driver attaches > > to the ChromeOS ACPI device and exports the values reported by ACPI in a > > sysfs directory. This data isn't present in ACPI tables when read > > through ACPI tools, hence a driver is needed to do it. The driver gets > > data from firmware using the ACPI component of the kernel. The ACPI values > > are presented in string form (numbers as decimal values) or binary > > blobs, and can be accessed as the contents of the appropriate read only > > files in the standard ACPI device's sysfs directory tree. This data is > > consumed by the ChromeOS user space. > > > Cc: Rafael J. Wysocki > > Cc: Dmitry Torokhov > > Cc: Hans de Goede > > You can use --cc parameter to `git send-email` instead of putting > these lines in the commit message. > > ... > > > +#define DEV_ATTR(_var, _name) \ > > + static struct device_attribute dev_attr_##_var = \ > > + __ATTR(_name, 0444, chromeos_first_level_attr_show, NULL); > > + > > Why not ATTR_RO()? Try it and see, that will not work here. thanks, greg k-h