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

Groups > linux.kernel > #1224994

Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors

Show all headers | View raw

On Tue, 2015-09-15 at 10:37 +0100, Punit Agrawal wrote:
> "Jon Medhurst (Tixy)" <tixy@linaro.org> writes:
> 
> > On Mon, 2015-09-14 at 15:38 +0100, Punit Agrawal wrote:
> >> Mark Rutland <mark.rutland@arm.com> writes:
> >> 
[...]
> >> The way the SCP interface is defined, the sensor identifiers are
> >> contiguous,
> >
> > Is there any documentation other than DUI0922A? [1] From what I can seen
> > that just says it's a 16-bit value and doesn't put any particular
> > constraints on its value.
> 
> Although not explicitly stated, if you look at the Get Sensor Capability
> [2] and Get Sensor Info [3] commands you can indirectly infer that the
> Sensor IDs are contiguous.

I personally wouldn't even indirectly infer they are contiguous from
what the document says. If I were implementing the firmware I would feel
quite in my rights to, for example, use the top 8 bits of the ID for a
sensor type and the bottom 8 for an index, if that made dispatching of
requests more efficient. Or if some optional hardware was detected as
missing, leaving some holes in ID space.

As a specification of a 'standard' the document seems to be rather
lacking. So, Sensor ID should be documented as being "an unsigned
integer less than then number of sensors returned by the Get Sensor
Capability command", or something like that. I guess clocks and other
devices suffer from similar lack of specificity.

>  Not the strongest guarantee I know.
> 
> All platforms currently using SCP (Juno R0 and R1) do indeed expose
> contiguous identifiers.

IMO, Linux drivers should be coded to the standard or written
specification (where they are available) not the particular
implementations available.

> >
> > [1] http://community.arm.com/servlet/JiveServlet/download/8401-40-18262/DUI0922A_scp_message_interface.pdf
> [2] http://arminfo.emea.arm.com/help/index.jsp?topic=/com.arm.doc.dui0922b/ch03s02s21.html
> [3] http://arminfo.emea.arm.com/help/index.jsp?topic=/com.arm.doc.dui0922b/BABCCCJJ.html

I think those links are on ARM's intranet, they return NXDOMAIN for me.

-- 
Tixy

--
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 | Next in thread | Find similar | Unroll thread

Thread

[PATCH v3 0/5] SCPI Sensor support Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:10 +0200
  [PATCH v3 3/5] hwmon: Support sensors exported via ARM SCP interface Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:10 +0200
  [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:10 +0200
    Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors Mark Rutland <mark.rutland@arm.com> - 2015-09-14 15:20 +0200
      Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:40 +0200
        Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors Mark Rutland <mark.rutland@arm.com> - 2015-09-14 16:00 +0200
          Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 16:40 +0200
            Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors Mark Rutland <mark.rutland@arm.com> - 2015-09-14 16:50 +0200
              Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 17:10 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors Mark Rutland <mark.rutland@arm.com> - 2015-09-14 17:20 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 18:10 +0200
            Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors "Jon Medhurst (Tixy)" <tixy@linaro.org> - 2015-09-14 20:20 +0200
              Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-15 11:40 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors "Jon Medhurst (Tixy)" <tixy@linaro.org> - 2015-09-15 12:50 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors Mark Rutland <mark.rutland@arm.com> - 2015-09-15 13:10 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors "Jon Medhurst (Tixy)" <tixy@linaro.org> - 2015-09-15 13:40 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-15 18:10 +0200
                Re: [PATCH v3 1/5] Documentation: add DT bindings for ARM SCPI  sensors "Jon Medhurst (Tixy)" <tixy@linaro.org> - 2015-09-15 18:40 +0200
  [PATCH v3 5/5] arm64: dts: Add sensor node to Juno dt Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:20 +0200
  [PATCH v3 4/5] hwmon: Support thermal zones registration for SCP temperature sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:20 +0200
  [Patch v3 4/5] hwmon: Support registration of thermal zones for SCP temperature sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:20 +0200
    Re: [Patch v3 4/5] hwmon: Support registration of thermal zones for SCP temperature sensors Punit Agrawal <punit.agrawal@arm.com> - 2015-09-14 15:20 +0200

csiph-web