s3c24xx pinctrl help
Tomasz Figa
tomasz.figa at gmail.com
Sat Mar 9 08:09:19 EST 2013
On Saturday 09 of March 2013 13:56:25 Heiko Stübner wrote:
> Am Freitag, 8. März 2013, 19:57:09 schrieb Tomasz Figa:
> > Hi Heiko,
> >
> > On Friday 08 of March 2013 15:38:04 Heiko Stübner wrote:
> > > Hi Thomas,
> > >
> > > taking you up on your offer of helping, I would be cool if you could
> > > simply give me a push in the right direction :-) .
> > >
> > >
> > > From what I've seen so far, the bank handling itself is very similar
> > > between exynos and s3c24xx as the underlying structures already
> > > handle
> > > multiple widths of the register contents. More interesting is the
> > > eint
> > > handling around which I couldn't wrap my head yet.
> > >
> > > The basic structure is again similar with special eint registers,
> > > but
> > > adds some quirks. EINT banks are gpf (8 eints) and gpg (8 or 16
> > > eints
> > > depending on the SoC).
> > >
> > > The current way on Exynos seems to be to mark the offset in the eint
> > > register and attach an irq_domain to the bank, which gets mapped to
> > > the
> > > eints starting at the offset. The eints seem to have a parent
> > > interrupt
> > > that is provided via the dt.
> > >
> > > On the S3C24xx the gpg bank is doing this similar but gpf is very
> > > strange.
> > >
> > > The first half of the bank (gpf0 to gpf3) is not handled in eintpend
> > > registers but in the main interrupt controller (bits 0 to 3), while
> > > the
> > > second half of gpf is handled in eintpend. The new interrupt
> > > declaration might show this better, which can be found at [0].
> > >
> > > An exception is the s3c2412 which adds still another quirk where
> > > each
> > > interrupt of gpf0 to gpf3 is represented in both the normal intc and
> > > eint registers, again for reference probably easier to see in [1].
> > >
> > >
> > > So I'm still quite stumped on how this could fit into the current
> > > framework and would be really glad for some small pointers :-)
> >
> > I wonder if some of my patches for pinctrl on S3C64xx might be
> > helpful:
> >
> > https://github.com/tom3q/linux/commits/v3.8-s3c64xx-dt-pinctrl
>
> Partially ... I got to know the (new to me) interrupt-map part of dt :-)
> .
>
> But on the S3C64XX you also have the benefit of all eints being in
> dedictated eint registers. I'm still thinking on the best way to
> integrate the eints that live directly in the main interrupt
> controller.
>
> One option I could think of, would be to add an optional to_irq callback
> to the bank definition and then do something like the following to
> handle the special case:
>
> static int samsung_gpio_to_irq(struct gpio_chip *gc, unsigned offset)
> {
> struct samsung_pin_bank *bank = gc_to_pin_bank(gc);
> unsigned int virq;
>
> if (bank->to_irq)
> return bank->to_irq(bank, offset);
>
> if (!bank->irq_domain)
> return -ENXIO;
>
> virq = irq_create_mapping(bank->irq_domain, offset);
>
> return (virq) ? : -ENXIO;
> }
>
> The GPG bank could simply use the existing mechanisms, as it follows the
> generic paradigm, and for GPF the callback would handle the distinction
> between eints handled in the eint regs or in the main interrupt
> controller.
Right, I checked how EINTs work on S3C2440A in documentation and indeed it
is a bit different than I initially thought.
However this is still similar to S3C64xx in some aspects. On S3C64xx there
is following hierarchy of interrupts in EINT0 group:
EINT0
EINT0
> And as always, talking about stuff like this helps in hashing out ideas
> ... my thinking is now a bit clearer than yesterday ;-)
>
>
> Thanks
> Heiko
More information about the linux-arm-kernel
mailing list