From: Dmitry Torokhov <dmitry.torokhov@gmail.com> To: Darren Hart <dvhart@linux.intel.com> Cc: Mika Westerberg <mika.westerberg@linux.intel.com>, "Rafael J. Wysocki" <rafael@kernel.org>, Al Stone <al.stone@linaro.org>, Olof Johansson <olof@lixom.net>, Matthew Garrett <matthew.garrett@nebula.com>, Matt Fleming <matt.fleming@intel.com>, David Woodhouse <dwmw2@infradead.org>, "H. Peter Anvin" <hpa@zytor.com>, Jacob Pan <jacob.jun.pan@linux.intel.com>, Josh Triplett <josh@joshtriplett.org>, Aaron Lu <aaron.lu@intel.com>, Max Eliaser <max.eliaser@intel.com>, Robert Moore <robert.moore@intel.com>, Len Brown <lenb@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Linus Walleij <linus.walleij@linaro.org>, Alexandre Courbot <gnurou@gmail.com>, Mark Brown <broonie@linaro.org>, Bryan Wu <cooloney@gmail.com>, Richard Purdie <rpurdie@rpsys.net>, Samuel Ortiz <sameo@linux.intel.com>, Lee Jones <lee.jones@linaro.org>, Grant Likely <grant.likely@linaro.org> Subject: Re: [RFC PATCH 0/9] Add ACPI _DSD and unified device properties support Date: Sun, 17 Aug 2014 07:11:06 -0700 [thread overview] Message-ID: <20140817141106.GA20345@core.coreip.homeip.net> (raw) In-Reply-To: <D014CE66.A19F5%dvhart@linux.intel.com> On Sat, Aug 16, 2014 at 09:06:05AM -0700, Darren Hart wrote: > On 8/15/14, 23:53, "Mika Westerberg" <mika.westerberg@linux.intel.com> > wrote: > > >The recent publication of the ACPI 5.1 specification [1] adds a reserved > >name > >for Device Specific Data (_DSD, Section 6.2.5). This mechanism allows for > >passing arbitrary hardware description data to the OS. The exact format > >of the > >_DSD data is specific to the UUID paired with it [2]. > > > >An ACPI Device Properties UUID has been defined [3] to provide a format > >compatible with existing device tree schemas. The purpose for this was to > >allow > >for the reuse of the existing schemas and encourage the development of > >firmware > >agnostic device drivers. > > > >This series accomplishes the following (as well as some other > >dependencies): > > > > * Add _DSD support to the ACPI core > > This simply reads the UUID and the accompanying Package > > > > * Add ACPI Device Properties _DSD format support > > This understands the hierarchical key:value pair structure > > defined by the Device Properties UUID > > > > * Add a unified device properties API with ACPI and OF backends > > This provides for the firmware agnostic device properties > > Interface to be used by drivers > > > > * Provides 2 example drivers that were previously Device Tree aware that > > can now be used with either Device Tree or ACPI Device Properties. The > > both drivers use an arbitrary _HID. > > > >This has been tested on Minnowboard with relevant parts of the modified > >DSDT at the end of this email. > > > This eliminates the need for the board files that were the subject of my > "How not to write x86 platform drivers" talk at ELC-E last year. With > These ACPI core changes and the small changes to the two example drivers, > the Minnowboard can now use the GPIO buttons and LEDs through these > drivers by adding the ASL fragment below to the DSDT. >From the drivers perspective I am less than impressed with the need to reshuffle all the drivers to support ACPI with the new API. I thought the plan was to try and keep OF API and try to translate as much as possible to it? The same goes for bringing arbitrary HIDs into the drivers. Can we have HID->OF naming hidden in ACPI (define a new property like "dt-name", "of_compat" or whatever) and have ACPI core map one to another. Thanks. -- Dmitry
WARNING: multiple messages have this Message-ID (diff)
From: Dmitry Torokhov <dmitry.torokhov@gmail.com> To: Darren Hart <dvhart@linux.intel.com> Cc: Mika Westerberg <mika.westerberg@linux.intel.com>, "Rafael J. Wysocki" <rafael@kernel.org>, Al Stone <al.stone@linaro.org>, Olof Johansson <olof@lixom.net>, Matthew Garrett <matthew.garrett@nebula.com>, Matt Fleming <matt.fleming@intel.com>, David Woodhouse <dwmw2@infradead.org>, "H. Peter Anvin" <hpa@zytor.com>, Jacob Pan <jacob.jun.pan@linux.intel.com>, Josh Triplett <josh@joshtriplett.org>, Aaron Lu <aaron.lu@intel.com>, Max Eliaser <max.eliaser@intel.com>, Robert Moore <robert.moore@intel.com>, Len Brown <lenb@kernel.org>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, Linus Walleij <linus.walleij@linaro.org>, Alexandre Courbot <gnurou@gmail.com>, Mark Brown <broonie@linaro.org>, Bryan Wu <cooloney@gmail.com>, Richard Purdie <rpurdie@rpsys.net>, Samuel Ortiz <sameo@linux.intel.com>, Lee Jones <lee.jones@linaro.org>, Grant Likely <grant.likely@linaro.org>, Rob Herring <robh+dt@kernel.org>, linux-acpi@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [RFC PATCH 0/9] Add ACPI _DSD and unified device properties support Date: Sun, 17 Aug 2014 07:11:06 -0700 [thread overview] Message-ID: <20140817141106.GA20345@core.coreip.homeip.net> (raw) In-Reply-To: <D014CE66.A19F5%dvhart@linux.intel.com> On Sat, Aug 16, 2014 at 09:06:05AM -0700, Darren Hart wrote: > On 8/15/14, 23:53, "Mika Westerberg" <mika.westerberg@linux.intel.com> > wrote: > > >The recent publication of the ACPI 5.1 specification [1] adds a reserved > >name > >for Device Specific Data (_DSD, Section 6.2.5). This mechanism allows for > >passing arbitrary hardware description data to the OS. The exact format > >of the > >_DSD data is specific to the UUID paired with it [2]. > > > >An ACPI Device Properties UUID has been defined [3] to provide a format > >compatible with existing device tree schemas. The purpose for this was to > >allow > >for the reuse of the existing schemas and encourage the development of > >firmware > >agnostic device drivers. > > > >This series accomplishes the following (as well as some other > >dependencies): > > > > * Add _DSD support to the ACPI core > > This simply reads the UUID and the accompanying Package > > > > * Add ACPI Device Properties _DSD format support > > This understands the hierarchical key:value pair structure > > defined by the Device Properties UUID > > > > * Add a unified device properties API with ACPI and OF backends > > This provides for the firmware agnostic device properties > > Interface to be used by drivers > > > > * Provides 2 example drivers that were previously Device Tree aware that > > can now be used with either Device Tree or ACPI Device Properties. The > > both drivers use an arbitrary _HID. > > > >This has been tested on Minnowboard with relevant parts of the modified > >DSDT at the end of this email. > > > This eliminates the need for the board files that were the subject of my > "How not to write x86 platform drivers" talk at ELC-E last year. With > These ACPI core changes and the small changes to the two example drivers, > the Minnowboard can now use the GPIO buttons and LEDs through these > drivers by adding the ASL fragment below to the DSDT. >From the drivers perspective I am less than impressed with the need to reshuffle all the drivers to support ACPI with the new API. I thought the plan was to try and keep OF API and try to translate as much as possible to it? The same goes for bringing arbitrary HIDs into the drivers. Can we have HID->OF naming hidden in ACPI (define a new property like "dt-name", "of_compat" or whatever) and have ACPI core map one to another. Thanks. -- Dmitry
next prev parent reply other threads:[~2014-08-17 14:11 UTC|newest] Thread overview: 39+ messages / expand[flat|nested] mbox.gz Atom feed top 2014-08-16 6:53 [RFC PATCH 0/9] Add ACPI _DSD and unified device properties support Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 1/9] ACPI: Add support for device specific properties Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 2/9] ACPI: Document ACPI " Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 3/9] Driver core: Unified device properties interface for platform firmware Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 4/9] of: Add property_ops callback for devices with of_node Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 5/9] mfd: Add ACPI support Mika Westerberg 2014-08-20 15:54 ` Lee Jones 2014-08-20 15:54 ` Lee Jones 2014-08-21 9:05 ` Mika Westerberg 2014-08-21 9:05 ` Mika Westerberg 2014-08-16 6:53 ` [RFC PATCH 6/9] gpiolib: add API to get gpio desc and flags Mika Westerberg 2014-08-18 16:24 ` Alexandre Courbot 2014-08-19 8:56 ` Mika Westerberg 2014-08-19 8:56 ` Mika Westerberg 2014-08-19 9:02 ` Aaron Lu 2014-08-19 9:02 ` Aaron Lu 2014-08-19 17:16 ` Alexandre Courbot 2014-08-19 17:16 ` Alexandre Courbot 2014-08-16 6:53 ` [RFC PATCH 7/9] gpio: sch: Consolidate core and resume banks Mika Westerberg 2014-08-29 6:36 ` Linus Walleij 2014-08-29 6:36 ` Linus Walleij 2014-08-16 6:53 ` [RFC PATCH 8/9] Input: gpio_keys_polled - Make use of device property API Mika Westerberg 2014-08-18 17:55 ` Jacob Pan 2014-08-18 17:55 ` Jacob Pan 2014-08-19 9:27 ` Mika Westerberg 2014-08-19 9:27 ` Mika Westerberg 2014-08-19 15:21 ` Darren Hart 2014-08-19 15:21 ` Darren Hart 2014-08-16 6:53 ` [RFC PATCH 9/9] leds: leds-gpio: " Mika Westerberg 2014-08-16 16:06 ` [RFC PATCH 0/9] Add ACPI _DSD and unified device properties support Darren Hart 2014-08-16 16:06 ` Darren Hart 2014-08-17 14:11 ` Dmitry Torokhov [this message] 2014-08-17 14:11 ` Dmitry Torokhov 2014-08-17 16:52 ` Darren Hart 2014-08-16 18:48 ` Josh Triplett 2014-08-16 18:48 ` Josh Triplett 2014-08-17 6:55 ` Mika Westerberg 2014-08-17 6:55 ` Mika Westerberg 2014-08-17 6:04 Mika Westerberg
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=20140817141106.GA20345@core.coreip.homeip.net \ --to=dmitry.torokhov@gmail.com \ --cc=aaron.lu@intel.com \ --cc=al.stone@linaro.org \ --cc=broonie@linaro.org \ --cc=cooloney@gmail.com \ --cc=dvhart@linux.intel.com \ --cc=dwmw2@infradead.org \ --cc=gnurou@gmail.com \ --cc=grant.likely@linaro.org \ --cc=gregkh@linuxfoundation.org \ --cc=hpa@zytor.com \ --cc=jacob.jun.pan@linux.intel.com \ --cc=josh@joshtriplett.org \ --cc=lee.jones@linaro.org \ --cc=lenb@kernel.org \ --cc=linus.walleij@linaro.org \ --cc=matt.fleming@intel.com \ --cc=matthew.garrett@nebula.com \ --cc=max.eliaser@intel.com \ --cc=mika.westerberg@linux.intel.com \ --cc=olof@lixom.net \ --cc=rafael@kernel.org \ --cc=robert.moore@intel.com \ --cc=rpurdie@rpsys.net \ --cc=sameo@linux.intel.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.