Re: gpio: dwapb: Synopsys Designware GPIO

[Steffen Trumtrar <s.trumtrar-bIcnvbaLZ9MEGnE8C9+IrQ@public.gmane.org>]

Hi Alan,

On Thu, Oct 31, 2013 at 10:34:50AM -0500, delicious quinoa wrote:
> On Thu, Oct 31, 2013 at 10:18 AM, Dinh Nguyen <dinh.linux-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> > Yes, we'll post a patch up on rocketboards-next and maybe we can post them
> > to the mailing list too?
> >
> Hi Steffen,
> 
> I've posted a branch on rocketboards.orgs' linux-socfpga-next.git.
> The branch name is dwapb-gpio-3.11.
> 

good. I will have a look.

> The top five patches are:
> 1. Remove the Altera gpio-dw.c driver
> 2 & 3.  Cherrypick Jamie's stuff from his git repo
> 4. Enable gpio-dwapb in our defconfig and dts
> 5. This is the main patch here: use irq_domain_add_linear for
> gpio-dwapb.c plus a few bug fixes.
> 
> If you'd rather see that last patch on the mailing list, I can post it
> there for review.
> 

I wonder if we want to really keep the binding as it is proposed by
Jamie. Do we really win anything by having to specify the banks in the
DT? In my version I get the number of ports, width etc. from the config registers
of the device. I think everything that the device knows itself and can be read
out at runtime shouldn't be specified in the DT.
And it seems that the binding was never merged, so I guess we can change it.

What do you think? I'd then finish up my driver and rebase your irq patch onto
it.

Regards,
Steffen

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html