* [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
@ 2013-09-23 12:03 Chander Kashyap
2013-09-23 12:10 ` Tomasz Figa
2013-09-26 5:05 ` Kukjin Kim
0 siblings, 2 replies; 7+ messages in thread
From: Chander Kashyap @ 2013-09-23 12:03 UTC (permalink / raw)
To: linux-kernel, linux-arm-kernel, linux-samsung-soc
Cc: kgene.kim, thomas.abraham, tglx, t.figa, Chander Kashyap
Replace irq_domain_add_simple with "irq_domain_add_linear" in order to use
linear irq domain, and to remove hardcoded irq_base_value.
Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
---
Changes since v1:
- Replaced irq_domain_add_simple with irq_domain_add_linear,
as suggested by Tomasz
drivers/irqchip/exynos-combiner.c | 15 +++------------
1 file changed, 3 insertions(+), 12 deletions(-)
diff --git a/drivers/irqchip/exynos-combiner.c b/drivers/irqchip/exynos-combiner.c
index 4c68265..2551046 100644
--- a/drivers/irqchip/exynos-combiner.c
+++ b/drivers/irqchip/exynos-combiner.c
@@ -206,8 +206,7 @@ static unsigned int combiner_lookup_irq(int group)
static void __init combiner_init(void __iomem *combiner_base,
struct device_node *np,
- unsigned int max_nr,
- int irq_base)
+ unsigned int max_nr)
{
int i, irq;
unsigned int nr_irq;
@@ -221,7 +220,7 @@ static void __init combiner_init(void __iomem *combiner_base,
return;
}
- combiner_irq_domain = irq_domain_add_simple(np, nr_irq, irq_base,
+ combiner_irq_domain = irq_domain_add_linear(np, nr_irq,
&combiner_irq_domain_ops, combiner_data);
if (WARN_ON(!combiner_irq_domain)) {
pr_warning("%s: irq domain init failed\n", __func__);
@@ -248,7 +247,6 @@ static int __init combiner_of_init(struct device_node *np,
{
void __iomem *combiner_base;
unsigned int max_nr = 20;
- int irq_base = -1;
combiner_base = of_iomap(np, 0);
if (!combiner_base) {
@@ -262,14 +260,7 @@ static int __init combiner_of_init(struct device_node *np,
__func__, max_nr);
}
- /*
- * FIXME: This is a hardwired COMBINER_IRQ(0,0). Once all devices
- * get their IRQ from DT, remove this in order to get dynamic
- * allocation.
- */
- irq_base = 160;
-
- combiner_init(combiner_base, np, max_nr, irq_base);
+ combiner_init(combiner_base, np, max_nr);
return 0;
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-09-23 12:03 [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value Chander Kashyap
@ 2013-09-23 12:10 ` Tomasz Figa
2013-09-26 5:05 ` Kukjin Kim
1 sibling, 0 replies; 7+ messages in thread
From: Tomasz Figa @ 2013-09-23 12:10 UTC (permalink / raw)
To: Chander Kashyap
Cc: linux-kernel, linux-arm-kernel, linux-samsung-soc, kgene.kim,
thomas.abraham, tglx
Hi Chander,
On Monday 23 of September 2013 17:33:35 Chander Kashyap wrote:
> Replace irq_domain_add_simple with "irq_domain_add_linear" in order to
> use linear irq domain, and to remove hardcoded irq_base_value.
>
> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> ---
> Changes since v1:
> - Replaced irq_domain_add_simple with irq_domain_add_linear,
> as suggested by Tomasz
>
> drivers/irqchip/exynos-combiner.c | 15 +++------------
> 1 file changed, 3 insertions(+), 12 deletions(-)
Reviewed-by: Tomasz Figa <t.figa@samsung.com>
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 7+ messages in thread
* RE: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-09-23 12:03 [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value Chander Kashyap
2013-09-23 12:10 ` Tomasz Figa
@ 2013-09-26 5:05 ` Kukjin Kim
2013-10-17 17:53 ` Tomasz Figa
1 sibling, 1 reply; 7+ messages in thread
From: Kukjin Kim @ 2013-09-26 5:05 UTC (permalink / raw)
To: 'Chander Kashyap',
linux-kernel, linux-arm-kernel, linux-samsung-soc
Cc: thomas.abraham, tglx, t.figa
Chander Kashyap wrote:
>
> Replace irq_domain_add_simple with "irq_domain_add_linear" in order to use
> linear irq domain, and to remove hardcoded irq_base_value.
>
> Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> ---
> Changes since v1:
> - Replaced irq_domain_add_simple with irq_domain_add_linear,
> as suggested by Tomasz
>
> drivers/irqchip/exynos-combiner.c | 15 +++------------
> 1 file changed, 3 insertions(+), 12 deletions(-)
>
> diff --git a/drivers/irqchip/exynos-combiner.c b/drivers/irqchip/exynos-
> combiner.c
> index 4c68265..2551046 100644
> --- a/drivers/irqchip/exynos-combiner.c
> +++ b/drivers/irqchip/exynos-combiner.c
> @@ -206,8 +206,7 @@ static unsigned int combiner_lookup_irq(int group)
>
> static void __init combiner_init(void __iomem *combiner_base,
> struct device_node *np,
> - unsigned int max_nr,
> - int irq_base)
> + unsigned int max_nr)
> {
> int i, irq;
> unsigned int nr_irq;
> @@ -221,7 +220,7 @@ static void __init combiner_init(void __iomem
> *combiner_base,
> return;
> }
>
> - combiner_irq_domain = irq_domain_add_simple(np, nr_irq, irq_base,
> + combiner_irq_domain = irq_domain_add_linear(np, nr_irq,
> &combiner_irq_domain_ops, combiner_data);
> if (WARN_ON(!combiner_irq_domain)) {
> pr_warning("%s: irq domain init failed\n", __func__);
> @@ -248,7 +247,6 @@ static int __init combiner_of_init(struct device_node
> *np,
> {
> void __iomem *combiner_base;
> unsigned int max_nr = 20;
> - int irq_base = -1;
>
> combiner_base = of_iomap(np, 0);
> if (!combiner_base) {
> @@ -262,14 +260,7 @@ static int __init combiner_of_init(struct device_node
> *np,
> __func__, max_nr);
> }
>
> - /*
> - * FIXME: This is a hardwired COMBINER_IRQ(0,0). Once all devices
> - * get their IRQ from DT, remove this in order to get dynamic
> - * allocation.
> - */
> - irq_base = 160;
> -
> - combiner_init(combiner_base, np, max_nr, irq_base);
> + combiner_init(combiner_base, np, max_nr);
>
> return 0;
> }
> --
> 1.7.9.5
Looks nice to me, applied with Tomasz's review.
Thanks,
Kukjin
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-09-26 5:05 ` Kukjin Kim
@ 2013-10-17 17:53 ` Tomasz Figa
2013-10-20 21:02 ` Kukjin Kim
0 siblings, 1 reply; 7+ messages in thread
From: Tomasz Figa @ 2013-10-17 17:53 UTC (permalink / raw)
To: linux-arm-kernel
Cc: Kukjin Kim, 'Chander Kashyap',
linux-kernel, linux-samsung-soc, tglx, thomas.abraham, t.figa
Hi Kukjin,
On Thursday 26 of September 2013 14:05:09 Kukjin Kim wrote:
> Chander Kashyap wrote:
> > Replace irq_domain_add_simple with "irq_domain_add_linear" in order to
> > use linear irq domain, and to remove hardcoded irq_base_value.
> >
> > Signed-off-by: Chander Kashyap <chander.kashyap@linaro.org>
> > ---
> >
> > Changes since v1:
> > - Replaced irq_domain_add_simple with irq_domain_add_linear,
> >
> > as suggested by Tomasz
> >
> > drivers/irqchip/exynos-combiner.c | 15 +++------------
> > 1 file changed, 3 insertions(+), 12 deletions(-)
[snip]
>
> Looks nice to me, applied with Tomasz's review.
I don't see this patch in your tree. Did you apply it in the end?
Best regards,
Tomasz
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-10-17 17:53 ` Tomasz Figa
@ 2013-10-20 21:02 ` Kukjin Kim
2013-11-25 6:37 ` Chander Kashyap
0 siblings, 1 reply; 7+ messages in thread
From: Kukjin Kim @ 2013-10-20 21:02 UTC (permalink / raw)
To: Tomasz Figa
Cc: linux-arm-kernel, linux-samsung-soc, t.figa, linux-kernel,
'Chander Kashyap',
Kukjin Kim, thomas.abraham, tglx
On 10/18/13 02:53, Tomasz Figa wrote:
> Hi Kukjin,
>
> On Thursday 26 of September 2013 14:05:09 Kukjin Kim wrote:
>> Chander Kashyap wrote:
>>> Replace irq_domain_add_simple with "irq_domain_add_linear" in order to
>>> use linear irq domain, and to remove hardcoded irq_base_value.
>>>
>>> Signed-off-by: Chander Kashyap<chander.kashyap@linaro.org>
>>> ---
>>>
>>> Changes since v1:
>>> - Replaced irq_domain_add_simple with irq_domain_add_linear,
>>>
>>> as suggested by Tomasz
>>>
>>> drivers/irqchip/exynos-combiner.c | 15 +++------------
>>> 1 file changed, 3 insertions(+), 12 deletions(-)
> [snip]
>>
>> Looks nice to me, applied with Tomasz's review.
>
> I don't see this patch in your tree. Did you apply it in the end?
>
Thanks for your gentle reminder.
Applied.
- Kukjin
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-10-20 21:02 ` Kukjin Kim
@ 2013-11-25 6:37 ` Chander Kashyap
2013-12-01 22:37 ` Kukjin Kim
0 siblings, 1 reply; 7+ messages in thread
From: Chander Kashyap @ 2013-11-25 6:37 UTC (permalink / raw)
To: Kukjin Kim
Cc: Tomasz Figa, linux-arm-kernel, linux-samsung-soc, t.figa,
linux-kernel, Kukjin Kim, Thomas Abraham, Thomas Gleixner
Hi Kikjin,
On 21 October 2013 02:32, Kukjin Kim <kgene.kim@samsung.com> wrote:
> On 10/18/13 02:53, Tomasz Figa wrote:
>>
>> Hi Kukjin,
>>
>> On Thursday 26 of September 2013 14:05:09 Kukjin Kim wrote:
>>>
>>> Chander Kashyap wrote:
>>>>
>>>> Replace irq_domain_add_simple with "irq_domain_add_linear" in order to
>>>> use linear irq domain, and to remove hardcoded irq_base_value.
>>>>
>>>> Signed-off-by: Chander Kashyap<chander.kashyap@linaro.org>
>>>> ---
>>>>
>>>> Changes since v1:
>>>> - Replaced irq_domain_add_simple with irq_domain_add_linear,
>>>>
>>>> as suggested by Tomasz
>>>>
>>>> drivers/irqchip/exynos-combiner.c | 15 +++------------
>>>> 1 file changed, 3 insertions(+), 12 deletions(-)
>>
>> [snip]
>>>
>>>
>>> Looks nice to me, applied with Tomasz's review.
>>
>>
>> I don't see this patch in your tree. Did you apply it in the end?
>>
> Thanks for your gentle reminder.
I still can not see these patches in yours branches ?
>
> Applied.
> - Kukjin
--
with warm regards,
Chander Kashyap
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value
2013-11-25 6:37 ` Chander Kashyap
@ 2013-12-01 22:37 ` Kukjin Kim
0 siblings, 0 replies; 7+ messages in thread
From: Kukjin Kim @ 2013-12-01 22:37 UTC (permalink / raw)
To: Chander Kashyap
Cc: Kukjin Kim, Tomasz Figa, linux-arm-kernel, linux-samsung-soc,
t.figa, linux-kernel, Kukjin Kim, Thomas Abraham,
Thomas Gleixner
On 11/25/13 15:37, Chander Kashyap wrote:
> Hi Kikjin,
>
Hi Chander,
> On 21 October 2013 02:32, Kukjin Kim<kgene.kim@samsung.com> wrote:
>> On 10/18/13 02:53, Tomasz Figa wrote:
>>>
>>> Hi Kukjin,
>>>
>>> On Thursday 26 of September 2013 14:05:09 Kukjin Kim wrote:
>>>>
>>>> Chander Kashyap wrote:
>>>>>
>>>>> Replace irq_domain_add_simple with "irq_domain_add_linear" in order to
>>>>> use linear irq domain, and to remove hardcoded irq_base_value.
>>>>>
>>>>> Signed-off-by: Chander Kashyap<chander.kashyap@linaro.org>
>>>>> ---
>>>>>
>>>>> Changes since v1:
>>>>> - Replaced irq_domain_add_simple with irq_domain_add_linear,
>>>>>
>>>>> as suggested by Tomasz
>>>>>
>>>>> drivers/irqchip/exynos-combiner.c | 15 +++------------
>>>>> 1 file changed, 3 insertions(+), 12 deletions(-)
>>>
>>> [snip]
>>>>
>>>>
>>>> Looks nice to me, applied with Tomasz's review.
>>>
>>>
>>> I don't see this patch in your tree. Did you apply it in the end?
>>>
>> Thanks for your gentle reminder.
>
> I still can not see these patches in yours branches ?
>
Oops, sorry for missing. This should be queued in next.
Thanks.
Kukjin
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2013-12-01 22:37 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-09-23 12:03 [PATCH v2] irqchip: exynos-combiner: remove hard-coded irq_base value Chander Kashyap
2013-09-23 12:10 ` Tomasz Figa
2013-09-26 5:05 ` Kukjin Kim
2013-10-17 17:53 ` Tomasz Figa
2013-10-20 21:02 ` Kukjin Kim
2013-11-25 6:37 ` Chander Kashyap
2013-12-01 22:37 ` Kukjin Kim
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).