[linux-sunxi] Re: [PATCH 2/6] pinctrl: sunxi: add allwinner A33 PIO controller support

2015-05-10 Thread Hans de Goede
Hi, On 10-05-15 08:46, Vishnu Patekar wrote: A33 PIO has 7 ports which starts from PB and has two interrupt ports. Signed-off-by: Vishnu Patekar Is this patch really necessary at all? The A33 is a pin compatible drop in for the A23, I would expect things to work just fine using the A23 pinmu

[linux-sunxi] Re: [PATCH 2/6] pinctrl: sunxi: add allwinner A33 PIO controller support

2015-05-10 Thread Chen-Yu Tsai
Hi, On Sun, May 10, 2015 at 4:50 PM, Hans de Goede wrote: > Hi, > > On 10-05-15 08:46, Vishnu Patekar wrote: >> >> A33 PIO has 7 ports which starts from PB and has two interrupt ports. >> >> Signed-off-by: Vishnu Patekar > > > Is this patch really necessary at all? The A33 is a pin compatible dr

[linux-sunxi] Re: [PATCH 2/6] pinctrl: sunxi: add allwinner A33 PIO controller support

2015-05-10 Thread Hans de Goede
Hi, On 10-05-15 11:00, Chen-Yu Tsai wrote: Hi, On Sun, May 10, 2015 at 4:50 PM, Hans de Goede wrote: Hi, On 10-05-15 08:46, Vishnu Patekar wrote: A33 PIO has 7 ports which starts from PB and has two interrupt ports. Signed-off-by: Vishnu Patekar Is this patch really necessary at all?

[linux-sunxi] Re: [PATCH 2/6] pinctrl: sunxi: add allwinner A33 PIO controller support

2015-05-10 Thread Maxime Ripard
On Sun, May 10, 2015 at 12:16:19PM +0530, Vishnu Patekar wrote: > A33 PIO has 7 ports which starts from PB and has two interrupt ports. > > Signed-off-by: Vishnu Patekar Acked-by: Maxime Ripard Thanks! Maxime -- Maxime Ripard, Free Electrons Embedded Linux, Kernel and Android engineering htt

[linux-sunxi] Re: [PATCH 2/6] pinctrl: sunxi: add allwinner A33 PIO controller support

2015-05-11 Thread Paul Bolle
On Sun, 2015-05-10 at 12:16 +0530, Vishnu Patekar wrote: > --- a/drivers/pinctrl/sunxi/Kconfig > +++ b/drivers/pinctrl/sunxi/Kconfig > + (Unneeded white line.) > +config PINCTRL_SUN8I_A33 > + def_bool MACH_SUN8I > + select PINCTRL_SUNXI_COMMON > --- a/drivers/pinctrl/sunxi/Makefile > ++