Re: [PATCH 5/6] ARM: LG Optimus Black (P970) codename sniper support, with basic features
From: Paul Kocialkowski
Date: Thu Dec 24 2015 - 14:39:11 EST
Hi,
Le mercredi 23 dÃcembre 2015 Ã 13:03 -0300, Javier Martinez Canillas a
Ãcrit :
> Hello Paul,
>
> [snip]
>
> > +
> > +&omap3_pmx_core {
> > + pinctrl-names = "default";
> > +
> > + uart3_pins: pinmux_uart3_pins {
> > + pinctrl-single,pins = <
> > + 0x16e (PIN_INPUT | MUX_MODE0) /* uart3_rx_irrx */
> > + 0x170 (PIN_OUTPUT | MUX_MODE0) /* uart3_tx_irtx */
> > + >;
>
> Could you please use the IOPAD mux macros from
> include/dt-bindings/pinctrl/omap.h instead?
>
> We just did a massive cleanup on the OMAP DTS to use them instead of
> an offset from the padconf registers.
Sure thing, will do in v2.
Thanks for the review!
--
Paul Kocialkowski, Replicant developer
Replicant is a fully free Android distribution running on several
devices, a free software mobile operating system putting the emphasis on
freedom and privacy/security.
Website: https://www.replicant.us/
Blog: https://blog.replicant.us/
Wiki/tracker/forums: https://redmine.replicant.us/
Attachment:
signature.asc
Description: This is a digitally signed message part