Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1404266
| From | Carlo Caione <carlo@caione.org> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller |
| Date | 2016-05-20 11:10 +0200 |
| Message-ID | <rAOBr-3AY-1@gated-at.bofh.it> (permalink) |
| References | <rANYJ-38Z-5@gated-at.bofh.it> <rANYJ-38Z-3@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
On 20/05/16 10:27, Neil Armstrong wrote:
> This patch adds the platform driver for the Amlogic Meson GXBB Reset
> Controller.
>
> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> ---
> drivers/reset/Kconfig | 6 ++
> drivers/reset/Makefile | 1 +
> drivers/reset/reset-meson-gxbb.c | 129 +++++++++++++++++++++++++++++++++++++++
> 3 files changed, 136 insertions(+)
> create mode 100644 drivers/reset/reset-meson-gxbb.c
Do we really need to be that specific (-gxbb)? This driver looks generic
and simple enough to be used for several Amlogic families. You are
already differentiating between them with the include file defining the
reset indexes for the SoC.
> diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig
> index df37212..4ac5c4d 100644
> --- a/drivers/reset/Kconfig
> +++ b/drivers/reset/Kconfig
> @@ -12,5 +12,11 @@ menuconfig RESET_CONTROLLER
>
> If unsure, say no.
>
> +config MESON_GXBB_RESET
> + tristate "Amlogic Meson GXBB Reset Driver"
> + depends on (ARCH_MESON && RESET_CONTROLLER)
> + help
> + Build the Amlogic Meson GxBB reset driver.
> +
> source "drivers/reset/sti/Kconfig"
> source "drivers/reset/hisilicon/Kconfig"
> diff --git a/drivers/reset/Makefile b/drivers/reset/Makefile
> index a1fc8ed..5ff83a1 100644
> --- a/drivers/reset/Makefile
> +++ b/drivers/reset/Makefile
> @@ -3,6 +3,7 @@ obj-$(CONFIG_ARCH_LPC18XX) += reset-lpc18xx.o
> obj-$(CONFIG_ARCH_SOCFPGA) += reset-socfpga.o
> obj-$(CONFIG_ARCH_BERLIN) += reset-berlin.o
> obj-$(CONFIG_MACH_PISTACHIO) += reset-pistachio.o
> +obj-$(CONFIG_MESON_GXBB_RESET) += reset-meson-gxbb.o
obj-$(CONFIG_ARCH_MESON) ?
[...]
> +static const struct reset_control_ops meson_gxbb_reset_ops = {
> + .reset = meson_gxbb_reset_reset,
> +};
nit: unfortunate name :)
Any reason to not have also .assert / .deassert?
[...]
> +static struct platform_driver meson_gxbb_reset_driver = {
> + .probe = meson_gxbb_reset_probe,
> + .remove = meson_gxbb_reset_remove,
> + .driver = {
> + .name = "meson_gxbb_reset",
> + .of_match_table = meson_gxbb_reset_dt_ids,
> + },
> +};
> +
> +module_platform_driver(meson_gxbb_reset_driver);
No MODULE_AUTHOR, MODULE_LICENSE, etc... ?
--
Carlo Caione
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH 0/3] Amlogic: GXBB: Add reset controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 10:30 +0200
[PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 10:30 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Carlo Caione <carlo@caione.org> - 2016-05-20 11:10 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 11:20 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Philipp Zabel <p.zabel@pengutronix.de> - 2016-05-20 11:30 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Carlo Caione <carlo@caione.org> - 2016-05-20 12:10 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 14:30 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Carlo Caione <carlo@caione.org> - 2016-05-20 14:30 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Philipp Zabel <p.zabel@pengutronix.de> - 2016-05-20 11:30 +0200
Re: [PATCH 1/3] reset: Add support for the Amlogic Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 14:20 +0200
[PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 10:30 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Carlo Caione <carlo@caione.org> - 2016-05-20 10:50 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 11:00 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Carlo Caione <carlo@caione.org> - 2016-05-20 11:20 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 14:20 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Carlo Caione <carlo@caione.org> - 2016-05-20 14:30 +0200
Re: [PATCH 3/3] ARM64: dts: amlogic: Enable Reset Controller on GXBB-based platforms Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 14:30 +0200
[PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 10:30 +0200
Re: [PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Carlo Caione <carlo@caione.org> - 2016-05-20 10:50 +0200
Re: [PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 11:00 +0200
Re: [PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Carlo Caione <carlo@caione.org> - 2016-05-20 11:10 +0200
Re: [PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Neil Armstrong <narmstrong@baylibre.com> - 2016-05-20 14:20 +0200
Re: [PATCH 2/3] dt-bindings: reset: Add bindings for the Meson GXBB Reset Controller Rob Herring <robh@kernel.org> - 2016-05-23 23:10 +0200
csiph-web