Commit 2a61e7b7 authored by Rob Herring's avatar Rob Herring Committed by Jassi Brar

mailbox: Use of_property_read_bool() for boolean properties

It is preferred to use typed property access functions (i.e.
of_property_read_<type> functions) rather than low-level
of_get_property/of_find_property functions for reading properties.
Convert reading boolean properties to to of_property_read_bool().
Signed-off-by: default avatarRob Herring <robh@kernel.org>
Signed-off-by: default avatarJassi Brar <jaswinder.singh@linaro.org>
parent 76d4adac
...@@ -325,10 +325,7 @@ static int hi6220_mbox_probe(struct platform_device *pdev) ...@@ -325,10 +325,7 @@ static int hi6220_mbox_probe(struct platform_device *pdev)
writel(~0x0, ACK_INT_CLR_REG(mbox->ipc)); writel(~0x0, ACK_INT_CLR_REG(mbox->ipc));
/* use interrupt for tx's ack */ /* use interrupt for tx's ack */
if (of_find_property(node, "hi6220,mbox-tx-noirq", NULL)) mbox->tx_irq_mode = !of_property_read_bool(node, "hi6220,mbox-tx-noirq");
mbox->tx_irq_mode = false;
else
mbox->tx_irq_mode = true;
if (mbox->tx_irq_mode) if (mbox->tx_irq_mode)
mbox->controller.txdone_irq = true; mbox->controller.txdone_irq = true;
......
...@@ -749,8 +749,7 @@ static int omap_mbox_probe(struct platform_device *pdev) ...@@ -749,8 +749,7 @@ static int omap_mbox_probe(struct platform_device *pdev)
finfo->name = child->name; finfo->name = child->name;
if (of_find_property(child, "ti,mbox-send-noirq", NULL)) finfo->send_no_irq = of_property_read_bool(child, "ti,mbox-send-noirq");
finfo->send_no_irq = true;
if (finfo->tx_id >= num_fifos || finfo->rx_id >= num_fifos || if (finfo->tx_id >= num_fifos || finfo->rx_id >= num_fifos ||
finfo->tx_usr >= num_users || finfo->rx_usr >= num_users) finfo->tx_usr >= num_users || finfo->rx_usr >= num_users)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment