Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask

From	Tony Lindgren <tony@atomide.com>
Newsgroups	linux.kernel
Subject	Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask
Date	2015-12-02 02:30 +0100
Message-ID	<qB4p4-ui-11@gated-at.bofh.it> (permalink)
References	<qB2GC-7Ot-3@gated-at.bofh.it> <qB2GC-7Ot-5@gated-at.bofh.it> <qB3t0-8kJ-15@gated-at.bofh.it> <qB3Ml-8rf-1@gated-at.bofh.it> <qB4fn-qG-3@gated-at.bofh.it>
Organization	linux.* mail to news gateway

Show all headers | View raw

* Matthijs van Duin <matthijsvanduin@gmail.com> [151201 17:15]:
> On 2 December 2015 at 01:46, Tony Lindgren <tony@atomide.com> wrote:
> > Ouch. We should probably have separate PIN_INPUT_3V3 and PIN_OUTPUT_3V3
> > dts macros that ensure that?
> 
> Can't we just keep bit 18 out of the function mask? The bootloader
> should already have made sure all pins have bit 18 set (and bit 19 set
> to correct values after ROM mucked them up, see advisory 2.1.88), so
> all that needs to be done is avoid touching them.

Sounds good to me. And people who really want to override the mask can
do it in the board specifc dts file.

> Are the power savings from disabling unnecessary inputs significant
> enough to spend any headache on it?

Only for some battery powered devices, not in this case for sure.

Regards,.

Tony
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Thread

[PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Tony Lindgren <tony@atomide.com> - 2015-12-02 00:40 +0100
  Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Matthijs van Duin <matthijsvanduin@gmail.com> - 2015-12-02 01:30 +0100
    Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Tony Lindgren <tony@atomide.com> - 2015-12-02 01:50 +0100
      Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Matthijs van Duin <matthijsvanduin@gmail.com> - 2015-12-02 02:20 +0100
        Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Tony Lindgren <tony@atomide.com> - 2015-12-02 02:30 +0100
          Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Tony Lindgren <tony@atomide.com> - 2015-12-02 17:20 +0100
      Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Matthijs van Duin <matthijsvanduin@gmail.com> - 2015-12-02 02:30 +0100
        Re: [PATCH 10/10] ARM: dts: Fix dm814x pinctrl address and mask Tony Lindgren <tony@atomide.com> - 2015-12-02 17:20 +0100

csiph-web