All of lore.kernel.org
 help / color / mirror / Atom feed
From: Michal Simek <michal.simek@xilinx.com>
To: Grant Likely <grant.likely@linaro.org>
Cc: Michal Simek <monstr@monstr.eu>,
	Jean-Christophe PLAGNIOL-VILLARD <plagnioj@jcrosoft.com>,
	Michal Simek <michal.simek@xilinx.com>,
	devicetree-discuss <devicetree-discuss@lists.ozlabs.org>,
	Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	Rob Herring <rob.herring@calxeda.com>
Subject: Re: [PATCH] of: Export of_irq_count for using in modules
Date: Thu, 6 Jun 2013 15:26:31 +0200	[thread overview]
Message-ID: <bfbf8745-1dd4-4a67-be76-e563cade5293@AM1EHSMHS019.ehs.local> (raw)
In-Reply-To: <CACxGe6synO5mksUPF4bG3NGnkS7HB47m8u_O+WVb6GG-0153Yw@mail.gmail.com>

On 06/06/2013 01:55 PM, Grant Likely wrote:
> On Thu, Jun 6, 2013 at 9:39 AM, Michal Simek <monstr@monstr.eu> wrote:
>> On 06/06/2013 10:29 AM, Jean-Christophe PLAGNIOL-VILLARD wrote:
>>> On 18:45 Fri 31 May     , Michal Simek wrote:
>>>> ok. good to know. Btw: Let's return to my origin point why not to
>>>> export of_irq_count for modules?
>>>> Or opposite question if platform_get_resource is correct way
>>>> why to export of_irq_to_resource for modules?
>>>
>>> for old ppc drivers that are not converted yet to pdev
>>>
>>> if you can do so just use pdev resource I should have fix the pb or irq_domain
>>> hopefully for 3.11
>>
>> ok. It means it is currently deprecated.
>> I just wanted to be sure that I understand it correctly.
> 
> It's deprecated for platform_devices, but should still be used for
> other bus types that don't provide the device driver with a list of
> irqs. For platform devices it is best to use the platform_bus irq
> table since that is portable regardless of how the device is
> instantiated.
> 
> As for not exporting of_irq_count(), I don't have a problem with it,
> but only if there is a user. It sounds like with the rework you
> discussed with Jean-Christophe that it is no longer the case.

We have talked about out of tree driver anyway
because it targets arm-arm remoteproc AMP which is not in mainline yet.
I have changed my drivers and using resource tables.
Interesting thing was that long interrupt list
is translated to resource table entry per cell and res->end is not
used to cover bigger ranges.

interrupts = < 0 29 0 0 30 0 0 31 0 0 32 0 0 33 0 0 34 0 0 35 0 0 36 0 0 52 0 0 53 0 0 54 0 0 55 0 0 56 0 0 57 0 0 58 0 0 59 0 0 37 0 0 38 0 0 39 0>;


BTW: can you please look at my microblaze email around interrupt
handlers?

Thanks,
Michal



  reply	other threads:[~2013-06-06 13:26 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-05-30 13:49 [PATCH] of: Export of_irq_count for using in modules Michal Simek
2013-05-30 13:49 ` Michal Simek
2013-05-30 20:17 ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-30 20:17   ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31  8:14   ` Michal Simek
2013-05-31  8:14     ` Michal Simek
2013-05-31 11:00     ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 11:00       ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 13:57       ` Michal Simek
2013-05-31 13:57         ` Michal Simek
2013-05-31 15:16         ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 15:16           ` Jean-Christophe PLAGNIOL-VILLARD
2013-05-31 16:45           ` Michal Simek
2013-06-06  8:29             ` Jean-Christophe PLAGNIOL-VILLARD
2013-06-06  8:39               ` Michal Simek
2013-06-06 11:55                 ` Grant Likely
2013-06-06 13:26                   ` Michal Simek [this message]
2013-06-06 14:49                 ` Jean-Christophe PLAGNIOL-VILLARD
     [not found]                   ` <20130606144929.GI19834-RQcB7r2h9QmfDR2tN2SG5Ni2O/JbrIOy@public.gmane.org>
2013-06-06 17:16                     ` Michal Simek
2013-07-18  7:02                   ` Michal Simek
2013-07-18  7:02                     ` Michal Simek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=bfbf8745-1dd4-4a67-be76-e563cade5293@AM1EHSMHS019.ehs.local \
    --to=michal.simek@xilinx.com \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=grant.likely@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=monstr@monstr.eu \
    --cc=plagnioj@jcrosoft.com \
    --cc=rob.herring@calxeda.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.