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

On Tue, Apr 12, 2016 at 2:15 PM, Mark Brown 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 >>

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

On Tue, Apr 12, 2016 at 2:15 PM, Mark Brown 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

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

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

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

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

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

On Wednesday, April 06, 2016 11:39:27 AM Mark Rutland wrote: > On Thu, Apr 07, 2016 at 03:11:53PM +0300, Octavian Purdila wrote: > > On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > > > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > > >> On Tue, Apr

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

On Wednesday, April 06, 2016 11:39:27 AM Mark Rutland wrote: > On Thu, Apr 07, 2016 at 03:11:53PM +0300, Octavian Purdila wrote: > > On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > > > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > > >> On Tue, Apr 5, 2016 at 9:16 PM,

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

On Thu, Apr 7, 2016 at 4:17 PM, Octavian Purdila wrote: > On Wed, Apr 6, 2016 at 1:49 PM, Graeme Gregory wrote: >> Why has there been no attempt in ASWG to make these sort of features a >> 1st class citizen of ACPI so they can interact correctly

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

On Thu, Apr 7, 2016 at 4:17 PM, Octavian Purdila wrote: > On Wed, Apr 6, 2016 at 1:49 PM, Graeme Gregory wrote: >> Why has there been no attempt in ASWG to make these sort of features a >> 1st class citizen of ACPI so they can interact correctly with the other >> features? > > IMO having an

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

On Thu, Apr 07, 2016 at 03:11:53PM +0300, Octavian Purdila wrote: > On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > >> On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: > >>

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

On Thu, Apr 07, 2016 at 03:11:53PM +0300, Octavian Purdila wrote: > On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > >> On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: > >> > * The firmware is to some extent expected

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

On Wed, Apr 6, 2016 at 1:49 PM, Graeme Gregory wrote: >> >> I have to echo Mark's concern: wholesale importation of portions of current >> DT bindings simply because it's expedient is one of the things I've been >> hoping >> to avoid. These patches seem to be just that.

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

On Wed, Apr 6, 2016 at 1:49 PM, Graeme Gregory wrote: >> >> I have to echo Mark's concern: wholesale importation of portions of current >> DT bindings simply because it's expedient is one of the things I've been >> hoping >> to avoid. These patches seem to be just that. >> >> And while the

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

On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: >> On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: >> > * The firmware is to some extent expected to manage pinctrl today (for

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

On Wed, Apr 6, 2016 at 3:01 AM, Mark Rutland wrote: > On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: >> On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: >> > * The firmware is to some extent expected to manage pinctrl today (for >> > power >> > management of devices it

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

On Tue, Apr 05, 2016 at 06:00:31PM -0600, Al Stone wrote: > On 04/05/2016 02:56 AM, Charles Garcia-Tobin wrote: > > > > > > On 04/04/2016 23:52, "Mark Rutland" wrote: > > > >> Hi, > >> > >> On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >>> This is a

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

On Tue, Apr 05, 2016 at 06:00:31PM -0600, Al Stone wrote: > On 04/05/2016 02:56 AM, Charles Garcia-Tobin wrote: > > > > > > On 04/04/2016 23:52, "Mark Rutland" wrote: > > > >> Hi, > >> > >> On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >>> This is a proposal for adding ACPI

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

On Tue, Apr 05, 2016 at 10:37:14PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: [...] > Lets look at this from a different perspective. The proposal is not > about importing the DT model into ACPI but importing the Linux pinctrl > model

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

On Tue, Apr 05, 2016 at 10:37:14PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: [...] > Lets look at this from a different perspective. The proposal is not > about importing the DT model into ACPI but importing the Linux pinctrl > model into ACPI. That will

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

On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: > > * The firmware is to some extent expected to manage pinctrl today (for power > > management of devices it does know about), and hence a pinctrl

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

On Tue, Apr 05, 2016 at 11:09:34PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: > > * The firmware is to some extent expected to manage pinctrl today (for power > > management of devices it does know about), and hence a pinctrl device is > > potentially

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

On 04/05/2016 02:56 AM, Charles Garcia-Tobin wrote: > > > On 04/04/2016 23:52, "Mark Rutland" wrote: > >> Hi, >> >> On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >>> This is a proposal for adding ACPI support for pin controller >>> configuration. >>> >>>

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

On 04/05/2016 02:56 AM, Charles Garcia-Tobin wrote: > > > On 04/04/2016 23:52, "Mark Rutland" wrote: > >> Hi, >> >> On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >>> This is a proposal for adding ACPI support for pin controller >>> configuration. >>> >>> It has been developed

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

On 04/05/2016 01:37 PM, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: >> On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: >>[snip...] > > I know that there are some discussions for pinmux configuration in the > ASWG, but it does not

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

On 04/05/2016 01:37 PM, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: >> On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: >>[snip...] > > I know that there are some discussions for pinmux configuration in the > ASWG, but it does not match the Linux

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

On Tue, Apr 05, 2016 at 10:37:14PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: > > On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > >> And these products are only update-able > >> with hairy BIOS patches that need to be

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

On Tue, Apr 05, 2016 at 10:37:14PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: > > On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > >> And these products are only update-able > >> with hairy BIOS patches that need to be applied > >> using

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

On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: >> >> Right, there is an overlap of the pinctrl "sleep" state with the ACPI power >> management model. >> >> However, the main reason for implementing this is setting initial pin >> multiplexing >> and configuration. This

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

On Tue, Apr 5, 2016 at 9:16 PM, Mark Rutland wrote: >> >> Right, there is an overlap of the pinctrl "sleep" state with the ACPI power >> management model. >> >> However, the main reason for implementing this is setting initial pin >> multiplexing >> and configuration. This is normally done by

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

On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: > On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > >> And these products are only update-able >> with hairy BIOS patches that need to be applied >> using $SPECIAL_TOOL that "normal users" do not want to >>

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

On Tue, Apr 5, 2016 at 7:59 PM, Mark Brown wrote: > On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > >> And these products are only update-able >> with hairy BIOS patches that need to be applied >> using $SPECIAL_TOOL that "normal users" do not want to >> concern themselves with,

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

On Tue, Apr 05, 2016 at 03:33:43PM +, Tirdea, Irina wrote: > > -Original Message- > > From: Mark Rutland [mailto:mark.rutl...@arm.com] > > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > > > This is a proposal for adding ACPI support for pin controller > > >

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

On Tue, Apr 05, 2016 at 03:33:43PM +, Tirdea, Irina wrote: > > -Original Message- > > From: Mark Rutland [mailto:mark.rutl...@arm.com] > > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > > > This is a proposal for adding ACPI support for pin controller > > >

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

On Tue, Apr 05, 2016 at 03:51:18PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 12:40 AM, Mark Brown wrote: > > So this is mainly targeted at modules being added to base boards? > That is the main use case, yes. Velocity of platform > debugging/enabling is another

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

On Tue, Apr 05, 2016 at 03:51:18PM +0300, Octavian Purdila wrote: > On Tue, Apr 5, 2016 at 12:40 AM, Mark Brown wrote: > > So this is mainly targeted at modules being added to base boards? > That is the main use case, yes. Velocity of platform > debugging/enabling is another one. The speed

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

On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > And these products are only update-able > with hairy BIOS patches that need to be applied > using $SPECIAL_TOOL that "normal users" do not want to > concern themselves with, as this is not an "apt-get upgrade" > kind of thing.

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

On Tue, Apr 05, 2016 at 10:43:11AM +0200, Linus Walleij wrote: > And these products are only update-able > with hairy BIOS patches that need to be applied > using $SPECIAL_TOOL that "normal users" do not want to > concern themselves with, as this is not an "apt-get upgrade" > kind of thing.

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

On Tue, Apr 05, 2016 at 11:00:50AM +0200, Linus Walleij wrote: > On Mon, Apr 4, 2016 at 11:40 PM, Mark Brown wrote: > > So this is mainly targeted at modules being added to base boards? > > Without getting into the binding at all here it seems like this is not > > solving the

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

On Tue, Apr 05, 2016 at 11:00:50AM +0200, Linus Walleij wrote: > On Mon, Apr 4, 2016 at 11:40 PM, Mark Brown wrote: > > So this is mainly targeted at modules being added to base boards? > > Without getting into the binding at all here it seems like this is not > > solving the problem at the

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

Heikki Krogerus; Andy Shevchenko; Purdila, Octavian; Ciocan, > Cristina; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; charles.garcia-to...@arm.com > Subject: Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration > > Hi, Hi Mark, > > On Thu, Mar 31, 20

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

Heikki Krogerus; Andy Shevchenko; Purdila, Octavian; Ciocan, > Cristina; devicet...@vger.kernel.org; linux- > ker...@vger.kernel.org; charles.garcia-to...@arm.com > Subject: Re: [RFC PATCH 0/4] Add ACPI support for pinctrl configuration > > Hi, Hi Mark, > > On Thu, Mar 31, 20

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

On Tue, Apr 5, 2016 at 12:40 AM, Mark Brown wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >> This is a proposal for adding ACPI support for pin controller >> configuration. > >> It has been developed to enable the MinnowBoard and IoT community >> by

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

On Tue, Apr 5, 2016 at 12:40 AM, Mark Brown wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >> This is a proposal for adding ACPI support for pin controller >> configuration. > >> It has been developed to enable the MinnowBoard and IoT community >> by providing an easy

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

On 04/04/2016 23:52, "Mark Rutland" wrote: >Hi, > >On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >> This is a proposal for adding ACPI support for pin controller >> configuration. >> >> It has been developed to enable the MinnowBoard and IoT community >>

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

On 04/04/2016 23:52, "Mark Rutland" wrote: >Hi, > >On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >> This is a proposal for adding ACPI support for pin controller >> configuration. >> >> It has been developed to enable the MinnowBoard and IoT community >> by providing an easy

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

On Mon, Apr 4, 2016 at 11:40 PM, Mark Brown wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >> This is a proposal for adding ACPI support for pin controller >> configuration. > >> It has been developed to enable the MinnowBoard and IoT community >> by

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

On Mon, Apr 4, 2016 at 11:40 PM, Mark Brown wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > >> This is a proposal for adding ACPI support for pin controller >> configuration. > >> It has been developed to enable the MinnowBoard and IoT community >> by providing an easy

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

On Tue, Apr 5, 2016 at 12:52 AM, Mark Rutland wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >> This proposal is based on using _DSD properties to specify device >> states and configuration nodes and it follows closely the device >> tree model.

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

On Tue, Apr 5, 2016 at 12:52 AM, Mark Rutland wrote: > On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: >> This proposal is based on using _DSD properties to specify device >> states and configuration nodes and it follows closely the device >> tree model. Device states are defined

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

On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > This is a proposal for adding ACPI support for pin controller > configuration. > It has been developed to enable the MinnowBoard and IoT community > by providing an easy way to specify pin multiplexing and > pin configuration. So

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

On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > This is a proposal for adding ACPI support for pin controller > configuration. > It has been developed to enable the MinnowBoard and IoT community > by providing an easy way to specify pin multiplexing and > pin configuration. So

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

Hi, On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > This is a proposal for adding ACPI support for pin controller > configuration. > > It has been developed to enable the MinnowBoard and IoT community > by providing an easy way to specify pin multiplexing and > pin configuration.

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

Hi, On Thu, Mar 31, 2016 at 02:44:41PM +0300, Irina Tirdea wrote: > This is a proposal for adding ACPI support for pin controller > configuration. > > It has been developed to enable the MinnowBoard and IoT community > by providing an easy way to specify pin multiplexing and > pin configuration.