From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v6] mfd: syscon: Add hardware spinlock support Date: Fri, 15 Dec 2017 10:42:04 +0000 Message-ID: <20171215104204.p7i27sen77ohbmzi@dell> References: <87ec50c846bbc7afc09ba0855aba1cdea6473308.1512048582.git.baolin.wang@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <87ec50c846bbc7afc09ba0855aba1cdea6473308.1512048582.git.baolin.wang-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Baolin Wang Cc: arnd-r2nGTMty4D4@public.gmane.org, robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: devicetree@vger.kernel.org Arnd, > Some system control registers need hardware spinlock to synchronize > between the multiple subsystems, so we should add hardware spinlock > support for syscon. > > Signed-off-by: Baolin Wang > Acked-by: Rob Herring > --- > Changes since v5: > - Fix the case that hwspinlock is not enabled. > > Changes since v4: > - Add one exapmle to show how to add hwlock. > - Fix the coding style issue. > > Changes since v3: > - Add error handling for of_hwspin_lock_get_id() > > Changes since v2: > - Add acked tag from Rob. > > Changes since v1: > - Remove timeout configuration. > - Modify the binding file to add hwlocks. > --- > Documentation/devicetree/bindings/mfd/syscon.txt | 8 +++++++ > drivers/mfd/syscon.c | 25 ++++++++++++++++++++++ > 2 files changed, 33 insertions(+) > > diff --git a/Documentation/devicetree/bindings/mfd/syscon.txt b/Documentation/devicetree/bindings/mfd/syscon.txt > index 8b92d45..25d9e9c 100644 > --- a/Documentation/devicetree/bindings/mfd/syscon.txt > +++ b/Documentation/devicetree/bindings/mfd/syscon.txt > @@ -16,9 +16,17 @@ Required properties: > Optional property: > - reg-io-width: the size (in bytes) of the IO accesses that should be > performed on the device. > +- hwlocks: reference to a phandle of a hardware spinlock provider node. > > Examples: > gpr: iomuxc-gpr@20e0000 { > compatible = "fsl,imx6q-iomuxc-gpr", "syscon"; > reg = <0x020e0000 0x38>; > + hwlocks = <&hwlock1 1>; > +}; > + > +hwlock1: hwspinlock@40500000 { > + ... > + reg = <0x40500000 0x1000>; > + #hwlock-cells = <1>; > }; > diff --git a/drivers/mfd/syscon.c b/drivers/mfd/syscon.c > index b93fe4c..a7ae391 100644 > --- a/drivers/mfd/syscon.c > +++ b/drivers/mfd/syscon.c > @@ -13,6 +13,7 @@ > */ > > #include > +#include > #include > #include > #include > @@ -87,6 +88,30 @@ static struct syscon *of_syscon_register(struct device_node *np) > if (ret) > reg_io_width = 4; > > + ret = of_hwspin_lock_get_id(np, 0); > + if (ret > 0) { > + syscon_config.hwlock_id = ret; > + syscon_config.hwlock_mode = HWLOCK_IRQSTATE; > + } else { > + switch (ret) { > + case -ENOENT: > + /* Ignore missing hwlock, it's optional. */ > + break; > + case 0: > + /* In case of the HWSPINLOCK is not enabled. */ > + if (!IS_ENABLED(CONFIG_HWSPINLOCK)) > + break; > + > + ret = -EINVAL; > + /* fall-through */ > + default: > + pr_err("Failed to retrieve valid hwlock: %d\n", ret); > + /* fall-through */ > + case -EPROBE_DEFER: > + goto err_regmap; > + } > + } > + > syscon_config.reg_stride = reg_io_width; > syscon_config.val_bits = reg_io_width * 8; > syscon_config.max_register = resource_size(&res) - reg_io_width; -- Lee Jones Linaro Services Technical Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html