Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1670532 > unrolled thread
| Started by | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| First post | 2017-06-20 11:30 +0200 |
| Last post | 2017-06-21 04:40 +0200 |
| Articles | 2 — 2 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for Spreadtrum pin controller Linus Walleij <linus.walleij@linaro.org> - 2017-06-20 11:30 +0200
Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for Spreadtrum pin controller Baolin Wang <baolin.wang@linaro.org> - 2017-06-21 04:40 +0200
| From | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| Date | 2017-06-20 11:30 +0200 |
| Subject | Re: [PATCH v2 1/2] DT: pinctrl: Add binding documentation for Spreadtrum pin controller |
| Message-ID | <tUnDZ-6Ll-47@gated-at.bofh.it> |
On Mon, Jun 12, 2017 at 8:07 AM, Baolin Wang <baolin.wang@spreadtrum.com> wrote:
> I agree we should add these into core code, something like:
> static const struct pinconf_generic_params dt_params[] = {
(...)
> { "slew-rate", PIN_CONFIG_SLEW_RATE, 0 },
> + { "sleep-bias-pull-up", PIN_CONFIG_SLEEP_BIAS_PULL_UP, 1},
> + { "sleep-bias-pull-down", PIN_CONFIG_SLEEP_BIAS_PULL_DOWN 1},
> ......
> };
>
> If you agree that, I can help to add these configuration.
Actually, yes indeed, that is a nice thing. Very nice. I think my Nomadik
driver could use this too.
Just need to hear what the DT maintainers have to say about it.
Yours,
Linus Walleij
[toc] | [next] | [standalone]
| From | Baolin Wang <baolin.wang@linaro.org> |
|---|---|
| Date | 2017-06-21 04:40 +0200 |
| Message-ID | <tUDIK-9t-3@gated-at.bofh.it> |
| In reply to | #1670532 |
On 20 June 2017 at 17:10, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Mon, Jun 12, 2017 at 8:07 AM, Baolin Wang <baolin.wang@spreadtrum.com> wrote:
>
>> I agree we should add these into core code, something like:
>> static const struct pinconf_generic_params dt_params[] = {
> (...)
>> { "slew-rate", PIN_CONFIG_SLEW_RATE, 0 },
>> + { "sleep-bias-pull-up", PIN_CONFIG_SLEEP_BIAS_PULL_UP, 1},
>> + { "sleep-bias-pull-down", PIN_CONFIG_SLEEP_BIAS_PULL_DOWN 1},
>> ......
>> };
>>
>> If you agree that, I can help to add these configuration.
>
> Actually, yes indeed, that is a nice thing. Very nice. I think my Nomadik
> driver could use this too.
>
> Just need to hear what the DT maintainers have to say about it.
OK, I will submit one patch to review. Thanks.
--
Baolin.wang
Best Regards
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web