Incorrect definition of SLEWCTRL_FAST in include/dt-bindings/pinctrl/am33xx.h

[chdh]

The following definition in am33xx.h is wrong:

#define SLEWCTRL_FAST           (1 << 6)

A correct definition would be:

#define SLEWCTRL_FAST           (0 << 6)      /* dummy, fast is default */
#define SLEWCTRL_SLOW           (1 << 6)

References:

1. It was correct in the Rowboat kernel: mux.h:

#define AM33XX_SLEWCTRL_FAST            (0 << 6)
#define AM33XX_SLEWCTRL_SLOW            (1 << 6)

2. AM335x Technical Reference Manual

Section "9.2.2 Pad Control Registers": Bit 6 SLEWCTRL: 0 = Fast, 1 = Slow
Section "9.3.1.50 conf_<module>_<pin> Register": Bit 6, Field conf_<module>_pin_slewctrl: Select between faster or slower slew rate. 0 = Fast, 1 = Slow.

3. meta-ti message from Peter A. Bigot: "mis-definition of SLEWCTRL_FAST in dt-bindings amxxxx header"

4. TI forum discussion: AM335x Slew rate setup in the device tree

5. It's not fixed at kernel.org

[RobertCNelson]

Updated post by Peter:
https://lists.yoctoproject.org/pipermail/meta-ti/2014-December/005615.html

I'm letting TI fix this upstream and in our 3.14-ti base..

[RobertCNelson]

@chdh looks like the e2e post came back here, please a post a rfc to linux-omap@vger.kernel.org

[RobertCNelson]

http://git.ti.com/gitweb/?p=ti-linux-kernel/ti-linux-kernel.git;a=commit;h=945e20d2734db817bcacd18f71acfb2389805e06