Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1377510

Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration

Show all headers | View raw

On Tue, Apr 12, 2016 at 2:15 PM, Mark Brown <broonie@kernel.org> wrote:
> On Thu, Apr 07, 2016 at 11:24:00PM +0200, Rafael J. Wysocki wrote:
>> On Wednesday, April 06, 2016 11:39:27 AM Mark Rutland wrote:
>
>> > The ACPI model implies FW-driven pinctrl management, so if we're going to put
>> > the OS in direct control of pinctrl, we have to make clear what expectation FW
>> > and OS can have.
>
>> Well, orthodox ACPI people from the Windows camp would definitely agree with you,
>> but I'm not one of them, so let me play a devil's advocate for a while. :-)
>
>> IMO, exposing anything in the ACPI tables without explicitly saying "but you
>> should not touch that", eg by defining an operation region covering it or similar,
>> is equivalent to giving the OS a license to play with that thing as it wishes.
>
>> Therefore I would regard exposing pin control information in cases when the
>> OS is not allowed to use it to its fit as a firmware bug.
>
> This is something where it seems like it would be very easy for people
> to end up relying on current OS behaviour even if that only happened
> through accident rather than design.  You might assume that such
> behaviour is a bug but that may not be obvious to the firmware author if
> the OS behaviour makes sense to them.

If the OS started to support that interface, it would need to
guarantee consistent behavior with respect to it going forward.

It would be an interface for interacting with firmware, so it is not
much different from an interface to user space as far as the stability
rules go.  Which may be a reason for refusing to support it in the
first place if the OS is not willing to make such a guarantee.

> There may also be some expectation on the part of firmware that the OS
> will do some management of the pins even for devices that it's not
> actively working with (put them in an idle mode for example).
>
>> Now, the question in this patch series is how to expose things and not when to
>> do that.  It adds support for a specific method of exposing information to the
>> OS, but should it be concerned about the possible consequences of inappropriate
>> use of that method by firmware?
>
> The other issue here is that if (as Octavian mentioned) there are
> ongoing discussions within ASWG on producing an actual spec for this it
> doesn't seem great that we'd just go and do something completely
> different rather than trying to make sure that the spec works well.  Or
> if there isn't any spec work then writing one there for other OSs to
> pick up if they like.  This seems like it'd make us much better
> community players.

Agreed.  I actually asked Octavian to look at the other proposal and
see if/how that could be used to address the use case in question
and/or possibly extended to cover it.

Thanks,
Rafael

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Rutland <mark.rutland@arm.com> - 2016-04-05 01:00 +0200
  Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Linus Walleij <linus.walleij@linaro.org> - 2016-04-05 10:50 +0200
    Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Brown <broonie@kernel.org> - 2016-04-05 19:00 +0200
      Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Octavian Purdila <octavian.purdila@intel.com> - 2016-04-05 21:40 +0200
        Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Brown <broonie@kernel.org> - 2016-04-06 00:50 +0200
        Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Al Stone <astone@redhat.com> - 2016-04-06 01:50 +0200
        Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Lorenzo Pieralisi <lorenzo.pieralisi@arm.com> - 2016-04-06 11:00 +0200
  RE: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration "Tirdea, Irina" <irina.tirdea@intel.com> - 2016-04-05 17:40 +0200
    Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Rutland <mark.rutland@arm.com> - 2016-04-05 20:30 +0200
      Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Octavian Purdila <octavian.purdila@intel.com> - 2016-04-05 22:10 +0200
        Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Rutland <mark.rutland@arm.com> - 2016-04-06 02:10 +0200
          Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Octavian Purdila <octavian.purdila@intel.com> - 2016-04-07 14:20 +0200
            Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Rutland <mark.rutland@arm.com> - 2016-04-07 18:00 +0200
              Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration "Rafael J. Wysocki" <rjw@rjwysocki.net> - 2016-04-07 23:30 +0200
                Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Mark Brown <broonie@kernel.org> - 2016-04-12 14:20 +0200
                Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration "Rafael J. Wysocki" <rafael@kernel.org> - 2016-04-13 07:10 +0200
  Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Al Stone <astone@redhat.com> - 2016-04-06 02:10 +0200
    Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Graeme Gregory <gg@slimlogic.co.uk> - 2016-04-06 12:50 +0200
      Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Octavian Purdila <octavian.purdila@intel.com> - 2016-04-07 16:20 +0200
        Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration Linus Walleij <linus.walleij@linaro.org> - 2016-04-07 20:10 +0200

csiph-web