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