Commit 1899ccc0 authored by Icenowy Zheng's avatar Icenowy Zheng Committed by Linus Walleij

pinctrl: sunxi: fix wrong irq_banks number for H5 pinctrl

The pin controller of Allwinner H5 has three IRQ banks, however in old
versions of drivers and device trees, only two are set, which makes
PG bank IRQ not available.

If it's directly set to 3, the old device trees will fail to boot.

Add a workaround (and a warning) for older device trees, and allow new
device trees to use correct 3 IRQ banks.

Fixes: 838adb57 ("drivers: pinctrl: add driver for Allwinner H5 SoC")
Signed-off-by: default avatarIcenowy Zheng <icenowy@aosc.io>
Acked-by: default avatarChen-Yu Tsai <wens@csie.org>
Signed-off-by: default avatarLinus Walleij <linus.walleij@linaro.org>
parent 5ff56b01
...@@ -19,6 +19,7 @@ ...@@ -19,6 +19,7 @@
#include <linux/platform_device.h> #include <linux/platform_device.h>
#include <linux/of.h> #include <linux/of.h>
#include <linux/of_device.h> #include <linux/of_device.h>
#include <linux/of_irq.h>
#include <linux/pinctrl/pinctrl.h> #include <linux/pinctrl/pinctrl.h>
#include "pinctrl-sunxi.h" #include "pinctrl-sunxi.h"
...@@ -530,17 +531,36 @@ static const struct sunxi_desc_pin sun50i_h5_pins[] = { ...@@ -530,17 +531,36 @@ static const struct sunxi_desc_pin sun50i_h5_pins[] = {
SUNXI_FUNCTION_IRQ_BANK(0x6, 2, 13)), /* PG_EINT13 */ SUNXI_FUNCTION_IRQ_BANK(0x6, 2, 13)), /* PG_EINT13 */
}; };
static const struct sunxi_pinctrl_desc sun50i_h5_pinctrl_data = { static const struct sunxi_pinctrl_desc sun50i_h5_pinctrl_data_broken = {
.pins = sun50i_h5_pins, .pins = sun50i_h5_pins,
.npins = ARRAY_SIZE(sun50i_h5_pins), .npins = ARRAY_SIZE(sun50i_h5_pins),
.irq_banks = 2, .irq_banks = 2,
.irq_read_needs_mux = true .irq_read_needs_mux = true
}; };
static const struct sunxi_pinctrl_desc sun50i_h5_pinctrl_data = {
.pins = sun50i_h5_pins,
.npins = ARRAY_SIZE(sun50i_h5_pins),
.irq_banks = 3,
.irq_read_needs_mux = true
};
static int sun50i_h5_pinctrl_probe(struct platform_device *pdev) static int sun50i_h5_pinctrl_probe(struct platform_device *pdev)
{ {
switch (of_irq_count(pdev->dev.of_node)) {
case 2:
dev_warn(&pdev->dev,
"Your device tree's pinctrl node is broken, which has no IRQ of PG bank routed.\n");
dev_warn(&pdev->dev,
"Please update the device tree, otherwise PG bank IRQ won't work.\n");
return sunxi_pinctrl_init(pdev,
&sun50i_h5_pinctrl_data_broken);
case 3:
return sunxi_pinctrl_init(pdev, return sunxi_pinctrl_init(pdev,
&sun50i_h5_pinctrl_data); &sun50i_h5_pinctrl_data);
default:
return -EINVAL;
}
} }
static const struct of_device_id sun50i_h5_pinctrl_match[] = { static const struct of_device_id sun50i_h5_pinctrl_match[] = {
......
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