Re: [PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Hanjun Guo]

Hi Ethan,

On 2017/7/6 13:46, Ethan Zhao wrote:
> Hanjun，
>
> What branch is this patch for ? Check v4.12, failed to apply.

I prepared patches based on master branch of linux-next git tree,
but they can be applied cleanly on top of latest Linus tree.

This patch set is for latest merged 4.13 material, not for 4.12 :)

Thanks
Hanjun

Re: [PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Hanjun Guo]

Hi Ethan,

On 2017/7/6 13:46, Ethan Zhao wrote:
> Hanjun，
>
> What branch is this patch for ? Check v4.12, failed to apply.

I prepared patches based on master branch of linux-next git tree,
but they can be applied cleanly on top of latest Linus tree.

This patch set is for latest merged 4.13 material, not for 4.12 :)

Thanks
Hanjun

Re: [PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Ethan Zhao]

Hanjun，

What branch is this patch for ? Check v4.12, failed to apply.

Thanks,
Ethan

On Thu, Jul 6, 2017 at 12:35 PM, Hanjun Guo  wrote:
> From: Hanjun Guo 
>
> is_fwnode_irqchip() already check fwnode is NULL or not,
> just use is_fwnode_irqchip() directly.
>
> Signed-off-by: Hanjun Guo 
> ---
>  kernel/irq/irqdomain.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
> index 1bc38fa..14a21a3 100644
> --- a/kernel/irq/irqdomain.c
> +++ b/kernel/irq/irqdomain.c
> @@ -137,7 +137,7 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle 
> *fwnode, int size,
> if (WARN_ON(!domain))
> return NULL;
>
> -   if (fwnode && is_fwnode_irqchip(fwnode)) {
> +   if (is_fwnode_irqchip(fwnode)) {
> fwid = container_of(fwnode, struct irqchip_fwid, fwnode);
>
> switch (fwid->type) {
> --
> 1.7.12.4
>

Re: [PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Ethan Zhao]

Hanjun，

What branch is this patch for ? Check v4.12, failed to apply.

Thanks,
Ethan

On Thu, Jul 6, 2017 at 12:35 PM, Hanjun Guo  wrote:
> From: Hanjun Guo 
>
> is_fwnode_irqchip() already check fwnode is NULL or not,
> just use is_fwnode_irqchip() directly.
>
> Signed-off-by: Hanjun Guo 
> ---
>  kernel/irq/irqdomain.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
> index 1bc38fa..14a21a3 100644
> --- a/kernel/irq/irqdomain.c
> +++ b/kernel/irq/irqdomain.c
> @@ -137,7 +137,7 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle 
> *fwnode, int size,
> if (WARN_ON(!domain))
> return NULL;
>
> -   if (fwnode && is_fwnode_irqchip(fwnode)) {
> +   if (is_fwnode_irqchip(fwnode)) {
> fwid = container_of(fwnode, struct irqchip_fwid, fwnode);
>
> switch (fwid->type) {
> --
> 1.7.12.4
>

[PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Hanjun Guo]

From: Hanjun Guo 

is_fwnode_irqchip() already check fwnode is NULL or not,
just use is_fwnode_irqchip() directly.

Signed-off-by: Hanjun Guo 
---
 kernel/irq/irqdomain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
index 1bc38fa..14a21a3 100644
--- a/kernel/irq/irqdomain.c
+++ b/kernel/irq/irqdomain.c
@@ -137,7 +137,7 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle 
*fwnode, int size,
if (WARN_ON(!domain))
return NULL;
 
-   if (fwnode && is_fwnode_irqchip(fwnode)) {
+   if (is_fwnode_irqchip(fwnode)) {
fwid = container_of(fwnode, struct irqchip_fwid, fwnode);
 
switch (fwid->type) {
-- 
1.7.12.4

[PATCH 2/2] genirq: Use is_fwnode_irqchip() directly

[Hanjun Guo]

From: Hanjun Guo 

is_fwnode_irqchip() already check fwnode is NULL or not,
just use is_fwnode_irqchip() directly.

Signed-off-by: Hanjun Guo 
---
 kernel/irq/irqdomain.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
index 1bc38fa..14a21a3 100644
--- a/kernel/irq/irqdomain.c
+++ b/kernel/irq/irqdomain.c
@@ -137,7 +137,7 @@ struct irq_domain *__irq_domain_add(struct fwnode_handle 
*fwnode, int size,
if (WARN_ON(!domain))
return NULL;
 
-   if (fwnode && is_fwnode_irqchip(fwnode)) {
+   if (is_fwnode_irqchip(fwnode)) {
fwid = container_of(fwnode, struct irqchip_fwid, fwnode);
 
switch (fwid->type) {
-- 
1.7.12.4