Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1259793

Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533

Show all headers | View raw

On Fri 30 Oct 13:18 PDT 2015, Rob Herring wrote:

> On Fri, Oct 30, 2015 at 2:41 PM, Bjorn Andersson
> <bjorn.andersson@sonymobile.com> wrote:
> > On Fri 30 Oct 11:42 PDT 2015, Lee Jones wrote:
> >
> > Rob, please see the discussion regarding ti,boost-freq-khz below. Should
> > we both specify unit at the same time as we use standard units? (This is
> > not the first time I have to change this back and forth)
> >
> >> On Tue, 27 Oct 2015, Bjorn Andersson wrote:
> >>
[..]
> > The ti,boost-low-freq from the bq25890 binding is the only other
> > property I can find that describes the same thing. So I'm not sure I
> > follow you here.
> >
> >> Perhaps we need to create a set of generic bindings.
> >>
> >> Also, we usually measure DT bindings in HZ, not kHz.
> 
> Surprisingly, there are not enough examples to draw much conclusion.
> 
> > I thought we had defined frequencies to be in HZ and HZ only, but then
> > Rob's comment that I need to actually specify the unit doesn't make any
> > sense.
> 
> I don't think we decided, but let's decide now. Go with Hz.
> 

+1

> Really, I first prefer the property name has units and second having
> standardized units. But if there is a common property without units, I
> prefer that even more.
> 

You can find this property in a variety of hardware, but I don't think
it would make much sense to define this single property in a common
place today.

This seems to be the first case where we specify the unit on one of
these properties though.

> > Do we want these properties in a standard unit or do we want them
> > specifying the unit? Having both seems excessive.
> 
> You mean "freq" would imply the units? No, we want the actual units in
> the property.
> 

You're right, frequency can be measured in other units than Hz, I can't
think of one that would be applicable here though.

Either way, I'll slap on a unit to the property name - Hz that is...

Thanks for your input.

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

Back to linux.kernel | Previous | Next — Previous in thread | Find similar | Unroll thread

Thread

[PATCH 1/3] devicetree: mfd: Add binding for the TI LM3533 Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-25 19:20 +0100
  Re: [PATCH 1/3] devicetree: mfd: Add binding for the TI LM3533 Rob Herring <robh+dt@kernel.org> - 2015-10-27 09:00 +0100
  [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-27 17:30 +0100
    [PATCH v2 3/3] mfd: lm3533: Support initialization from Device Tree Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-27 17:30 +0100
      Re: [PATCH v2 3/3] mfd: lm3533: Support initialization from Device  Tree kbuild test robot <lkp@intel.com> - 2015-10-27 17:50 +0100
      Re: [PATCH v2 3/3] mfd: lm3533: Support initialization from Device  Tree kbuild test robot <lkp@intel.com> - 2015-10-27 18:00 +0100
      Re: [PATCH v2 3/3] mfd: lm3533: Support initialization from Device  Tree Lee Jones <lee.jones@linaro.org> - 2015-10-28 12:50 +0100
        Re: [PATCH v2 3/3] mfd: lm3533: Support initialization from Device  Tree Joe Perches <joe@perches.com> - 2015-10-28 13:00 +0100
        Re: [PATCH v2 3/3] mfd: lm3533: Support initialization from Device  Tree Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-28 19:50 +0100
    [PATCH v2 2/3] iio: light: lm3533-als: Print error message on invalid resistance Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-27 17:40 +0100
    Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Rob Herring <robh+dt@kernel.org> - 2015-10-27 20:30 +0100
    Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Lee Jones <lee.jones@linaro.org> - 2015-10-30 19:50 +0100
      Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-30 20:50 +0100
        Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Rob Herring <robh+dt@kernel.org> - 2015-10-30 21:20 +0100
          Re: [PATCH v2 1/3] devicetree: mfd: Add binding for the TI LM3533 Bjorn Andersson <bjorn.andersson@sonymobile.com> - 2015-10-30 22:20 +0100

csiph-web