Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: quoted-printable
In-Reply-To: <Qk2RUY9T8QCX1un25vsEYOAnlYlkMQpywbz05_FpVSYq8Lek4EStNPkA_vt_BBH1HKuwsbNw_F8Ec73fM2Kk9Zdl1wvjLtwaHE5QKSvV6Hs=@wujek.eu>
References: <20211203141515.2448129-1-dev_public@wujek.eu> <20211206225354.BF0AFC341C6@smtp.kernel.org> <jgJW1u3RfTQcv5ZBcVQt4fz8_sA4gvyXsN5x18zzLo_nTR5nfLpZtdAjy-WlEaCMnmnPGZjeFGV5OoiWNtA4Tn5wLhqNJdQEvPw5Cqs1z3I=@wujek.eu> <20211208043054.CEFD6C00446@smtp.kernel.org> <XedF_WXH1abo9n2bzY4gaRIjq-R1O6ha5QedK8e1W9-R_1p_zo90aAgaKSSfDYMW3WUaMdY0pjONnKWj4zNMh5TbHlbZbAVAe4KG_Zq8Vvk=@wujek.eu> <20211216063245.603F1C36AE4@smtp.kernel.org> <Qk2RUY9T8QCX1un25vsEYOAnlYlkMQpywbz05_FpVSYq8Lek4EStNPkA_vt_BBH1HKuwsbNw_F8Ec73fM2Kk9Zdl1wvjLtwaHE5QKSvV6Hs=@wujek.eu>
Subject: Re: [PATCH] clk: si5341: Add sysfs property to check selected input
From:   Stephen Boyd <sboyd@kernel.org>
Cc:     Michael Turquette <mturquette@baylibre.com>,
        linux-clk@vger.kernel.org, linux-kernel@vger.kernel.org
To:     wujek dev <dev_public@wujek.eu>
Date:   Thu, 16 Dec 2021 17:47:36 -0800
User-Agent: alot/0.9.1
Message-Id: <20211217014737.4BA49C36AE0@smtp.kernel.org>
Precedence: bulk

Quoting wujek dev (2021-12-16 06:25:58)
>=20
>=20
> =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 Original =
Message =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90
>=20
> On Thursday, December 16th, 2021 at 07:32, Stephen Boyd <sboyd@kernel.org=
> wrote:
>=20
> > Quoting wujek dev (2021-12-09 04:04:48)
> >
> > > =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90 Origi=
nal Message =E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90=E2=80=90
> > >
> > > On Wednesday, December 8th, 2021 at 05:30, Stephen Boyd sboyd@kernel.=
org 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 subsyst=
em is described. Shall I the add description of this single property?
> > > >
> > > > Please don't top post. sysfs properties are supposed to be single v=
alue
> > > >
> > > > and for machine consumption. Is this a debugfs property?
> > > >
> > > > No, I think this is rather sysfs than debugfs. The type of informat=
ion is similar to other reported by this driver. The values in the added pr=
operty are the same, this is just a different representation (input selecte=
d as an int, as string based on manual and as string as described in the DT=
B).
> >
> > 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?