Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-06 Thread Raphaël Poggi
Ok, let's do it like that Best regards, Raphaël Poggi 2014-08-06 5:48 GMT+02:00 Jean-Christophe PLAGNIOL-VILLARD : > > On Aug 6, 2014, at 11:46 AM, Jean-Christophe PLAGNIOL-VILLARD > wrote: > >> >> On Aug 5, 2014, at 7:38 PM, Raphaël Poggi wrote: >> >>> >>> Ok, so I drop my patch ? Or your ver

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-05 Thread Raphaël Poggi
Ok, so I drop my patch ? Or your version is very similar of my patch and you can review mine to improve/clean it ? Best regards, Raphaël Poggi 2014-08-05 12:26 GMT+02:00 Jean-Christophe PLAGNIOL-VILLARD : > > On Aug 5, 2014, at 2:37 AM, Raphaël Poggi wrote: > >> >> I have planned to add device t

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-05 Thread Jean-Christophe PLAGNIOL-VILLARD
On Aug 5, 2014, at 2:37 AM, Raphaël Poggi wrote: > > I have planned to add device tree support for the AT91 port. > > So I port the pinctrl driver from linux to barebox to be able to use > the pinctrl in device tree, but maybe I made a mistake and there is > another solution. > I’ve a versio

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-04 Thread Sascha Hauer
On Mon, Aug 04, 2014 at 08:37:35PM +0200, Raphaël Poggi wrote: > I have planned to add device tree support for the AT91 port. > > So I port the pinctrl driver from linux to barebox to be able to use > the pinctrl in device tree, but maybe I made a mistake and there is > another solution. Nope. Wh

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-04 Thread Raphaël Poggi
I have planned to add device tree support for the AT91 port. So I port the pinctrl driver from linux to barebox to be able to use the pinctrl in device tree, but maybe I made a mistake and there is another solution. Best Regards, Raphaël 2014-08-04 20:26 GMT+02:00 Jean-Christophe PLAGNIOL-VILLAR

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-04 Thread Jean-Christophe PLAGNIOL-VILLARD
why do we need it on barebox? this driver was design for dt-only in linux Best Regards, J. On Aug 1, 2014, at 9:24 PM, Raphaël Poggi wrote: > This driver is based on mach-at91/gpio.c and linux pinctrl driver. > The driver contains the gpio and pinctrl parts (like in linux) because the > two pa

Re: [PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-04 Thread Sascha Hauer
Looks mostly fine. On Fri, Aug 01, 2014 at 03:24:23PM +0200, Raphaël Poggi wrote: > This driver is based on mach-at91/gpio.c and linux pinctrl driver. > The driver contains the gpio and pinctrl parts (like in linux) because the > two parts > share some structures and logics. > > Signed-off-by: R

[PATCH 1/2] pinctrl: at91: add pinctrl driver

2014-08-01 Thread Raphaël Poggi
This driver is based on mach-at91/gpio.c and linux pinctrl driver. The driver contains the gpio and pinctrl parts (like in linux) because the two parts share some structures and logics. Signed-off-by: Raphaël Poggi --- drivers/pinctrl/Kconfig|6 + drivers/pinctrl/Makefile |