lkml.org 
[lkml]   [2022]   [Apr]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v13 4/9] dt-bindings: clock: Add bindings for SP7021 clock driver
On Thu, Apr 14, 2022 at 07:06:12AM +0000, qinjian[覃健] wrote:
>
> > > +#ifndef _DT_BINDINGS_CLOCK_SUNPLUS_SP7021_H
> > > +#define _DT_BINDINGS_CLOCK_SUNPLUS_SP7021_H
> > > +
> > > +#define XTAL 27000000
> >
> > This looks like frequency which usually does not belong to bindings. Why
> > do you need it here?
> >
>
> OK, I'll remove it.

If that's the input frequency to the clock controller and it could
possibly change, then it should be defined in DT with a 'fixed-clock'
node and 'clocks' property in the clock controller.

Rob

\
 
 \ /
  Last update: 2022-04-14 17:58    [W:0.084 / U:0.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site