lkml.org 
[lkml]   [2012]   [Aug]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    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_parent_device>_probe()
    |
    +-> mfd_add_device()
    |
    +-> res->start = res->end = irq_create_mapping(res->start); // Now a VIRQ


    <mfd_child_device>_probe()
    |
    +-> <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
    Linaro.org │ 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 majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2012-08-20 19:41    [W:2.250 / U:0.120 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site