linux-kernel - Re: [PATCH 1/2] staging: iio: adc: ad7192: Add clock for external clock reference

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20181216100741.4e362a17@archlinux>
Date:   Sun, 16 Dec 2018 10:07:41 +0000
From:   Jonathan Cameron <jic23@...nel.org>
To:     Stephen Boyd <sboyd@...nel.org>
Cc:     Mircea Caprioru <mircea.caprioru@...log.com>,
        Michael.Hennerich@...log.com, knaack.h@....de, lars@...afoo.de,
        pmeerw@...erw.net, gregkh@...uxfoundation.org,
        linux-kernel@...r.kernel.org, linux-iio@...r.kernel.org,
        devel@...verdev.osuosl.org, Rob Herring <robh+dt@...nel.org>,
        linux-clk@...r.kernel.org
Subject: Re: [PATCH 1/2] staging: iio: adc: ad7192: Add clock for external
 clock reference

Rob, Clk experts, questions for you below.

Jonathan


On Thu, 13 Dec 2018 17:39:22 -0800
Stephen Boyd <sboyd@...nel.org> wrote:

> Quoting Jonathan Cameron (2018-12-08 07:29:54)
> > On Thu, 6 Dec 2018 11:10:51 +0200
> > Mircea Caprioru <mircea.caprioru@...log.com> wrote:
> >   
> > > This patch adds a clock to the state structure of ad7192 for getting the
> > > external clock frequency. This modifications is in accordance with clock
> > > framework dt bindings documentation.
> > > 
> > > Signed-off-by: Mircea Caprioru <mircea.caprioru@...log.com>  
> > 
> > +cc Rob and the clk list for advise on how to do the binding for this one.
> > 
> > It is basically 2 pins, you can put a clock in on one of them or connect
> > a crystal across them.  The driver has to set a register to say which is
> > the case.  
> > 
> > Current proposal is two optional clocks (fall back to internal oscillator)
> > but that doesn't seem to be commonly done, so I'm wondering if there
> > is a 'standard' way to handle this sort of thing.
> >   
> 
> I'm not sure I fully understand, but I think perhaps
> assigned-clock-parents would work? Or does that not work because either
> way some parent is assigned, either the crystal or the optional clk that
> isn't a crystal?
> 
My concern is they aren't really separate clock inputs.   They are just different
ways of providing the same fundamental clock.  So I think we may want to just
provide a single clock and have another dt binding to say what it is.

So lots of ways we could do it, but I'm not sure what the right one to
go with is!

Jonathan