Messages in this thread Patch in this message | | | Subject | Re: [PATCH V3 2/2] reset: img: Add Pistachio reset controller driver | From | Philipp Zabel <> | Date | Mon, 01 Feb 2016 21:42:07 +0100 |
| |
Hi James,
Thanks for the reminder. I'll apply these with a few small fixups. No need to resend if you agree.
Am Montag, den 18.01.2016, 13:12 +0000 schrieb James Hartley: > From: "Damien.Horsley" <Damien.Horsley@imgtec.com>
s/Damien.Horsley/Damien Horsley/
> > Add reset controller driver for Pistachio SoC > > Signed-off-by: Damien.Horsley <Damien.Horsley@imgtec.com>
Same here.
> Signed-off-by: James Hartley <james.hartley@imgtec.com> > --- > drivers/reset/Makefile | 1 + > drivers/reset/reset-pistachio.c | 155 ++++++++++++++++++++++++++ > include/dt-bindings/reset/pistachio-resets.h | 36 ++++++ > 3 files changed, 192 insertions(+) > create mode 100644 drivers/reset/reset-pistachio.c > create mode 100644 include/dt-bindings/reset/pistachio-resets.h [...] > diff --git a/drivers/reset/reset-pistachio.c b/drivers/reset/reset-pistachio.c > new file mode 100644 > index 0000000..8e39465 > --- /dev/null > +++ b/drivers/reset/reset-pistachio.c > @@ -0,0 +1,155 @@ [...] > +static int pistachio_reset_assert(struct reset_controller_dev *rcdev, > + unsigned long id) This should be aligned to the open parenthesis.
[...] > +static int pistachio_reset_deassert(struct reset_controller_dev *rcdev, > + unsigned long id)
And this.
[...] > +static int pistachio_reset_remove(struct platform_device *pdev) > +{ > + struct pistachio_reset_data *data = platform_get_drvdata(pdev); > + > + reset_controller_unregister(&data->rcdev); > + > + return 0; > +} > + > +
And one of these empty lines has to go.
> +static const struct of_device_id pistachio_reset_dt_ids[] = { > + { .compatible = "img,pistachio-reset", }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(of, pistachio_reset_dt_ids); > + > +static struct platform_driver pistachio_reset_driver = { > + .probe = pistachio_reset_probe, > + .remove = pistachio_reset_remove, > + .driver = { > + .name = "pistachio-reset", > + .of_match_table = pistachio_reset_dt_ids, > + }, > +}; > +module_platform_driver(pistachio_reset_driver); > + > +MODULE_AUTHOR("Damien Horsley <Damien.Horsley@imgtec.com>");
regards Philipp
| |