Re: [PATCH] clk: si5341: Add sysfs property to check selected input

From: Stephen Boyd
Date: Thu Dec 16 2021 - 20:47:40 EST


Quoting wujek dev (2021-12-16 06:25:58)
>
>
> ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
>
> On Thursday, December 16th, 2021 at 07:32, Stephen Boyd <sboyd@xxxxxxxxxx> wrote:
>
> > Quoting wujek dev (2021-12-09 04:04:48)
> >
> > > ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
> > >
> > > On Wednesday, December 8th, 2021 at 05:30, Stephen Boyd sboyd@xxxxxxxxxx wrote:
> > >
> > > > Quoting wujek dev (2021-12-06 15:08:03)
> > > >
> > > > > Other sysfs properties used by this driver are not documented so I didn't add for this one. Even more not a single property from clk subsystem is described. Shall I the add description of this single property?
> > > >
> > > > Please don't top post. sysfs properties are supposed to be single value
> > > >
> > > > and for machine consumption. Is this a debugfs property?
> > > >
> > > > No, I think this is rather sysfs than debugfs. The type of information is similar to other reported by this driver. The values in the added property are the same, this is just a different representation (input selected as an int, as string based on manual and as string as described in the DTB).
> >
> > Ok what program is going to check the selected input?
> So far, only the project specific program. I'm not aware about a standard tool that check other (already available) properties from this driver.

Why doesn't clk debugfs that indicates the parents of a clk work here?