linux-kernel - Re: [PATCH] misc: add CS2000 Fractional-N driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87vbbdkbcm.wl%kuninori.morimoto.gx@renesas.com>
Date:	Mon, 14 Sep 2015 02:05:58 +0000
From:	Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
To:	Arnd Bergmann <arnd@...db.de>
CC:	Simon <horms@...ge.net.au>,
	Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
	<linux-kernel@...r.kernel.org>, Magnus <magnus.damm@...il.com>,
	<linux-sh@...r.kernel.org>
Subject: Re: [PATCH] misc: add CS2000 Fractional-N driver


Hi Arnd

Thank you for your feedback

> > From: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> > 
> > This patch adds CS2000 Fractional-N driver as clock provider.
> > It is useful if it supports runtime clock setting, but it supports
> > fixed clock rate at this point.
> > 
> > Signed-off-by: Kuninori Morimoto <kuninori.morimoto.gx@...esas.com>
> > ---
> >  .../devicetree/bindings/misc/cs2000-cp.txt         |  20 ++
> >  drivers/misc/Kconfig                               |   6 +
> >  drivers/misc/Makefile                              |   1 +
> >  drivers/misc/cs2000-cp.c                           | 341 +++++++++++++++++++++
> >  4 files changed, 368 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/misc/cs2000-cp.txt
> >  create mode 100644 drivers/misc/cs2000-cp.c
> 
> I think the driver should be in drivers/clk/ if it provides a clk to
> other devices.
> 
> Please also split out the binding document into a separate patch and
> Cc the devicetree mailing list.

OK, thanks
I will do it, and send to clock ML

> > +       ret = cs2000_get_clk(client, &rate_in, &rate_out);
> > +       if (ret < 0)
> > +               return ret;
> > +
> > +       ret = cs2000_enable_dev_config(client);
> > +       if (ret < 0)
> > +               return ret;
> > +
> > +       ret = cs2000_clk_in_bound_rate(client, rate_in);
> > +       if (ret < 0)
> > +               return ret;
> > +
> > +       ret = cs2000_ratio_set(client, ch, rate_in, rate_out);
> > +       if (ret < 0)
> > +               return ret;
> 
> The probe function lacks unwinding if anything goes wrong, so you end
> up with clocks that are registered but the driver being absent.
> For most things, you can use the devm_*() interfaces here.

I got it.
Thanks

Best regards
---
Kuninori Morimoto
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/