Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for Spreadtrum pin controller

From	Linus Walleij <linus.walleij@linaro.org>
Newsgroups	linux.kernel
Subject	Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for Spreadtrum pin controller
Date	2017-06-20 11:40 +0200
Message-ID	<tUnND-6PZ-9@gated-at.bofh.it> (permalink)
References	<tOZ9g-2Na-7@gated-at.bofh.it> <tQpXI-LP-21@gated-at.bofh.it> <tRqRI-6Wp-7@gated-at.bofh.it> <tRKx4-2BE-7@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

On Tue, Jun 13, 2017 at 5:15 AM, Baolin Wang <baolin.wang@spreadtrum.com> wrote:

> I forgot one most important reason why we can not use the "sleep" state. As I explained
> above, the sleep related configuration will bind with the pin's sleep mode. If we set the
> pin's sleep mode as AP_SLEEP, then we can select "sleep" state when AP system goes into
> deep sleep mode by issuing "pinctrl_force_sleep()" in pinctrl suspend function.
>
> But if we set the pin's sleep mode as PUBCP_SLEEP and pubcp system doesn't run linux kernel
> (it run another thread OS), then we can not select "sleep" state since the AP system does
> not go into deep sleep mode (AP system run linux kernel OS).

Allright yes it makes sense, and also there are systems that just go into
"hardware sleep" and just put the pin into some pre-programmed mode.

I'm a bit back-and-forth. I didn't mean that some code would actually
switch the state to "sleep" when we go to sleep, I meant that when
the system configures "default" mode it should also look up and
program the "sleep" mode, but this approach with a special property
is just another way of achieveing the same thing.

But then we should add a whole slew of sleep states.

I was thinking whether we could avoid having a special DT property
by parsing ahead to states we do not currently use and programming
that into the sleep mode registers.

Yours,
Linus Walleij

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

Thread

Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for  Spreadtrum pin controller Linus Walleij <linus.walleij@linaro.org> - 2017-06-20 11:40 +0200
  Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for  Spreadtrum pin controller Baolin Wang <baolin.wang@linaro.org> - 2017-06-21 10:20 +0200
    Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for  Spreadtrum pin controller Linus Walleij <linus.walleij@linaro.org> - 2017-06-26 00:20 +0200
      Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for  Spreadtrum pin controller Baolin Wang <baolin.wang@linaro.org> - 2017-06-27 10:00 +0200

csiph-web