[lkml]   [2012]   [Aug]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 5/8] mfd: Provide the PRCMU with its own IRQ domain
    On Mon, Aug 20, 2012 at 05:29:23PM +0100, Mark Brown wrote:
    > On Mon, Aug 20, 2012 at 01:55:32PM +0100, Lee Jones wrote:
    > > Right, that was the initial intention. It would be a trivial semantic
    > > change if drivers without DT support wished to use the functionality
    > > though. However, the only examples I found of a non-DT enabled driver
    > > that could make good use of it in order to strip out some cruft would
    > > be the Arizona and one of the Samsung drivers, and they each have
    > All of the regmap devices could use this.

    Only if they have linear domains and don't support DT.

    If they don't have linear domains there's no point, if they support DT
    then they can use it as it is.

    > > their own hand-rolled methods of hwirq -> virq conversion now, so any
    > > change to support them would result in multiple invocations of
    > > irq_create_mapping which would likely cause breakage.
    > Multiple calls to irq_create_mapping() are totally fine.

    Sorry, perhaps I wasn't clear.

    +-> mfd_add_device()
    +-> res->start = res->end = irq_create_mapping(res->start); // Now a VIRQ

    +-> <mfd_parent_device>_irq_get_virq(res->start) // Ahhh, double VIRQ conversion

    To stop being DT dependent we'd need to remove all hand-rolled stuff
    that these drivers are currently using, or else they will be attempting
    to convert and already converted VIRQ value.

    Lee Jones
    Linaro ST-Ericsson Landing Team Lead │ Open source software for ARM SoCs
    Follow Linaro: Facebook | Twitter | Blog
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to
    More majordomo info at
    Please read the FAQ at

     \ /
      Last update: 2012-08-20 19:41    [W:0.029 / U:9.300 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site