Devicetree
 help / color / mirror / Atom feed
From: Conor Dooley <conor@kernel.org>
To: Ariana Lazar <ariana.lazar@microchip.com>
Cc: "Jonathan Cameron" <jic23@kernel.org>,
	"David Lechner" <dlechner@baylibre.com>,
	"Nuno Sá" <nuno.sa@analog.com>,
	"Andy Shevchenko" <andy@kernel.org>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzk+dt@kernel.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	linux-iio@vger.kernel.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 1/2] dt-bindings: iio: dac: add support for Microchip MCP48FEB02
Date: Thu, 12 Feb 2026 18:00:06 +0000	[thread overview]
Message-ID: <20260212-germless-favoring-c27ab4c53128@spud> (raw)
In-Reply-To: <20260212-mcp48feb02-v1-1-ce5843db65db@microchip.com>

[-- Attachment #1: Type: text/plain, Size: 13024 bytes --]

On Thu, Feb 12, 2026 at 02:48:34PM +0200, Ariana Lazar wrote:
> This is the device tree schema for iio driver for Microchip
> MCP48FxBy1/2/4/8 series of buffered voltage output Digital-to-Analog
> Converters with nonvolatile or volatile memory and an SPI Interface.
> 
> The families support up to 8 output channels.
> 
> The devices can be 8-bit, 10-bit and 12-bit.
> 
> Signed-off-by: Ariana Lazar <ariana.lazar@microchip.com>

Other than the interface, what's actually different between this and the
47? Could they share the same binding?

Cheers,
Conor.

> ---
>  .../bindings/iio/dac/microchip,mcp48feb02.yaml     | 299 +++++++++++++++++++++
>  MAINTAINERS                                        |   6 +
>  2 files changed, 305 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/iio/dac/microchip,mcp48feb02.yaml b/Documentation/devicetree/bindings/iio/dac/microchip,mcp48feb02.yaml
> new file mode 100644
> index 0000000000000000000000000000000000000000..78c6bd641c6e37321e4fc056db83eb4277f429b8
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/iio/dac/microchip,mcp48feb02.yaml
> @@ -0,0 +1,299 @@
> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> +%YAML 1.2
> +---
> +$id: http://devicetree.org/schemas/iio/dac/microchip,mcp48feb02.yaml#
> +$schema: http://devicetree.org/meta-schemas/core.yaml#
> +
> +title: Microchip MCP48F(E/V)B(0/1/2)(1/2/4/8) DAC with SPI Interface Families
> +
> +maintainers:
> +  - Ariana Lazar <ariana.lazar@microchip.com>
> +
> +description: |
> +  Datasheet for MCP48FEB01, MCP48FEB02, MCP48FEB11, MCP48FEB12, MCP48FEB21,
> +  MCP48FEB22 can be found here:
> +    https://ww1.microchip.com/downloads/aemDocuments/documents/OTH/ProductDocuments/DataSheets/20005429B.pdf
> +  Datasheet for MCP48FVB01, MCP48FVB02, MCP48FVB11, MCP48FVB12, MCP48FVB21,
> +  MCP48FVB22 can be found here:
> +    https://ww1.microchip.com/downloads/aemDocuments/documents/OTH/ProductDocuments/DataSheets/20005466A.pdf
> +  Datasheet for MCP48FEB04, MCP48FEB14, MCP48FEB24, MCP48FEB08, MCP48FEB18,
> +  MCP48FEB28, MCP48FVB04, MCP48FVB14, MCP48FVB24, MCP48FVB08, MCP48FVB18,
> +  MCP48FVB28 can be found here:
> +    https://ww1.microchip.com/downloads/aemDocuments/documents/MSLD/ProductDocuments/DataSheets/MCP48FXBX4-8-Family-Data-Sheet-DS20006362A.pdf
> +
> +  +------------+--------------+-------------+-------------+------------+
> +  | Device     |  Resolution  |   Channels  | Vref number | Memory     |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FEB01 |              |      1      |      1      |   EEPROM   |
> +  | MCP48FEB02 |     8-bit    |      2      |      1      |   EEPROM   |
> +  | MCP48FEB04 |              |      4      |      2      |   EEPROM   |
> +  | MCP48FEB08 |              |      8      |      2      |   EEPROM   |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FEB11 |              |      1      |      1      |   EEPROM   |
> +  | MCP48FEB12 |    10-bit    |      2      |      1      |   EEPROM   |
> +  | MCP48FEB14 |              |      4      |      2      |   EEPROM   |
> +  | MCP48FEB18 |              |      8      |      2      |   EEPROM   |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FEB21 |              |      1      |      1      |   EEPROM   |
> +  | MCP48FEB22 |    12-bit    |      2      |      1      |   EEPROM   |
> +  | MCP48FEB24 |              |      4      |      2      |   EEPROM   |
> +  | MCP48FEB28 |              |      8      |      2      |   EEPROM   |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FVB01 |              |      1      |      1      |   RAM      |
> +  | MCP48FVB02 |     8-bit    |      2      |      1      |   RAM      |
> +  | MCP48FVB04 |              |      4      |      2      |   RAM      |
> +  | MCP48FVB08 |              |      8      |      2      |   RAM      |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FVB11 |              |      1      |      1      |   RAM      |
> +  | MCP48FVB12 |    10-bit    |      2      |      1      |   RAM      |
> +  | MCP48FVB14 |              |      4      |      2      |   RAM      |
> +  | MCP48FVB18 |              |      8      |      2      |   RAM      |
> +  |------------|--------------|-------------|-------------|------------|
> +  | MCP48FVB21 |              |      1      |      1      |   RAM      |
> +  | MCP48FVB22 |    12-bit    |      2      |      1      |   RAM      |
> +  | MCP48FVB24 |              |      4      |      2      |   RAM      |
> +  | MCP48FVB28 |              |      8      |      2      |   RAM      |
> +  +------------+--------------+-------------+-------------+------------+
> +
> +properties:
> +  compatible:
> +    enum:
> +      - microchip,mcp48feb01
> +      - microchip,mcp48feb02
> +      - microchip,mcp48feb04
> +      - microchip,mcp48feb08
> +      - microchip,mcp48feb11
> +      - microchip,mcp48feb12
> +      - microchip,mcp48feb14
> +      - microchip,mcp48feb18
> +      - microchip,mcp48feb21
> +      - microchip,mcp48feb22
> +      - microchip,mcp48feb24
> +      - microchip,mcp48feb28
> +      - microchip,mcp48fvb01
> +      - microchip,mcp48fvb02
> +      - microchip,mcp48fvb04
> +      - microchip,mcp48fvb08
> +      - microchip,mcp48fvb11
> +      - microchip,mcp48fvb12
> +      - microchip,mcp48fvb14
> +      - microchip,mcp48fvb18
> +      - microchip,mcp48fvb21
> +      - microchip,mcp48fvb22
> +      - microchip,mcp48fvb24
> +      - microchip,mcp48fvb28
> +
> +  reg:
> +    maxItems: 1
> +
> +  "#address-cells":
> +    const: 1
> +
> +  "#size-cells":
> +    const: 0
> +
> +  vdd-supply:
> +    description:
> +      Provides power to the chip and it could be used as reference voltage. The
> +      voltage is used to calculate scale. For parts without EEPROM at powerup
> +      this will be the selected as voltage reference.
> +
> +  vref-supply:
> +    description: |
> +      Vref pin (it could be found as Vref0 into the datasheet) may be used as a
> +      voltage reference when this supply is specified. The internal reference
> +      will be taken into account for voltage reference besides VDD if this supply
> +      does not exist.
> +
> +      This supply will be voltage reference for the following outputs:
> +        - for single-channel device: Vout0;
> +        - for dual-channel device: Vout0, Vout1;
> +        - for quad-channel device: Vout0, Vout2;
> +        - for octal-channel device: Vout0, Vout2, Vout4, Vout6;
> +
> +  vref1-supply:
> +    description: |
> +      Vref1 pin may be used as a voltage reference when this supply is specified.
> +      The internal reference will be taken into account for voltage reference
> +      beside VDD if this supply does not exist.
> +
> +      This supply will be voltage reference for the following outputs:
> +        - for quad-channel device: Vout1, Vout3;
> +        - for octal-channel device: Vout1, Vout3, Vout5, Vout7;
> +
> +  lat-gpios:
> +    description:
> +      LAT pin to be used as a hardware trigger to synchronously update the DAC
> +      channels. The pin is active Low. It could be also found as LAT0 in
> +      datasheet.
> +    maxItems: 1
> +
> +  lat1-gpios:
> +    description:
> +      LAT1 pin to be used as a hardware trigger to synchronously update the odd
> +      DAC channels on devices with 4 and 8 channels. The pin is active Low.
> +    maxItems: 1
> +
> +  microchip,vref-buffered:
> +    type: boolean
> +    description:
> +      Enable buffering of the external Vref/Vref0 pin in cases where the
> +      external reference voltage does not have sufficient current capability in
> +      order not to drop its voltage when connected to the internal resistor
> +      ladder circuit.
> +
> +  microchip,vref1-buffered:
> +    type: boolean
> +    description:
> +      Enable buffering of the external Vref1 pin in cases where the external
> +      reference voltage does not have sufficient current capability in order not
> +      to drop its voltage when connected to the internal resistor ladder
> +      circuit.
> +
> +patternProperties:
> +  "^channel@[0-7]$":
> +    $ref: dac.yaml
> +    type: object
> +    description: Voltage output channel.
> +
> +    properties:
> +      reg:
> +        description: The channel number.
> +        maxItems: 1
> +
> +      label:
> +        description: Unique name to identify which channel this is.
> +
> +    required:
> +      - reg
> +
> +    unevaluatedProperties: false
> +
> +required:
> +  - compatible
> +  - reg
> +  - vdd-supply
> +
> +allOf:
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - microchip,mcp48feb01
> +              - microchip,mcp48feb11
> +              - microchip,mcp48feb21
> +              - microchip,mcp48fvb01
> +              - microchip,mcp48fvb11
> +              - microchip,mcp48fvb21
> +    then:
> +      properties:
> +        lat1-gpios: false
> +        vref1-supply: false
> +        microchip,vref1-buffered: false
> +        channel@0:
> +          properties:
> +            reg:
> +              const: 0
> +      patternProperties:
> +        "^channel@[1-7]$": false
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - microchip,mcp48feb02
> +              - microchip,mcp48feb12
> +              - microchip,mcp48feb22
> +              - microchip,mcp48fvb02
> +              - microchip,mcp48fvb12
> +              - microchip,mcp48fvb22
> +    then:
> +      properties:
> +        lat1-gpios: false
> +        vref1-supply: false
> +        microchip,vref1-buffered: false
> +      patternProperties:
> +        "^channel@[0-1]$":
> +          properties:
> +            reg:
> +              enum: [0, 1]
> +        "^channel@[2-7]$": false
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - microchip,mcp48fvb04
> +              - microchip,mcp48fvb14
> +              - microchip,mcp48fvb24
> +              - microchip,mcp48feb04
> +              - microchip,mcp48feb14
> +              - microchip,mcp48feb24
> +    then:
> +      patternProperties:
> +        "^channel@[0-3]$":
> +          properties:
> +            reg:
> +              enum: [0, 1, 2, 3]
> +        "^channel@[4-7]$": false
> +  - if:
> +      properties:
> +        compatible:
> +          contains:
> +            enum:
> +              - microchip,mcp48fvb08
> +              - microchip,mcp48fvb18
> +              - microchip,mcp48fvb28
> +              - microchip,mcp48feb08
> +              - microchip,mcp48feb18
> +              - microchip,mcp48feb28
> +    then:
> +      patternProperties:
> +        "^channel@[0-7]$":
> +          properties:
> +            reg:
> +              enum: [0, 1, 2, 3, 4, 5, 6, 7]
> +  - if:
> +      not:
> +        required:
> +          - vref-supply
> +    then:
> +      properties:
> +        microchip,vref-buffered: false
> +  - if:
> +      not:
> +        required:
> +          - vref1-supply
> +    then:
> +      properties:
> +        microchip,vref1-buffered: false
> +
> +additionalProperties: false
> +
> +examples:
> +    - |
> +      spi {
> +          #address-cells = <1>;
> +          #size-cells = <0>;
> +
> +          dac@0 {
> +            compatible = "microchip,mcp48feb08";
> +            reg = <0>;
> +            vdd-supply = <&vdac_vdd>;
> +            vref-supply = <&vref_reg>;
> +
> +            #address-cells = <1>;
> +            #size-cells = <0>;
> +            channel@0 {
> +              reg = <0>;
> +              label = "Adjustable_voltage_ch0";
> +            };
> +
> +            channel@1 {
> +              reg = <0x1>;
> +              label = "Adjustable_voltage_ch1";
> +            };
> +          };
> +      };
> +...
> diff --git a/MAINTAINERS b/MAINTAINERS
> index a92290fffa163f9fe8fe3f04bf66426f9a894409..ed24fd2758ad0103dbc5191d0ec180f8ee5e8298 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -14945,6 +14945,12 @@ S:	Maintained
>  F:	Documentation/devicetree/bindings/iio/dac/microchip,mcp4821.yaml
>  F:	drivers/iio/dac/mcp4821.c
>  
> +MCP48FEB02 MICROCHIP DAC DRIVER
> +M:	Ariana Lazar <ariana.lazar@microchip.com>
> +L:	linux-iio@vger.kernel.org
> +S:	Supported
> +F:	Documentation/devicetree/bindings/iio/dac/microchip,mcp48feb02.yaml
> +
>  MCR20A IEEE-802.15.4 RADIO DRIVER
>  M:	Stefan Schmidt <stefan@datenfreihafen.org>
>  L:	linux-wpan@vger.kernel.org
> 
> -- 
> 2.43.0
> 

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

  parent reply	other threads:[~2026-02-12 18:00 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-02-12 12:48 [PATCH 0/2] Add support for Microchip MCP48FxBy1/2/4/8 DAC with an SPI Interface Ariana Lazar
2026-02-12 12:48 ` [PATCH 1/2] dt-bindings: iio: dac: add support for Microchip MCP48FEB02 Ariana Lazar
2026-02-12 17:31   ` Rob Herring (Arm)
2026-02-12 18:00   ` Conor Dooley [this message]
2026-02-12 20:04     ` Andy Shevchenko
2026-02-16 13:31       ` Ariana.Lazar
2026-02-16 15:37         ` David Lechner
2026-02-16 17:34           ` Conor Dooley
2026-02-16 18:38             ` David Lechner
2026-02-12 12:48 ` [PATCH 2/2] " Ariana Lazar
2026-02-12 14:42   ` Andy Shevchenko
2026-02-16 14:29     ` Ariana.Lazar
2026-02-17  7:54       ` Andy Shevchenko
2026-02-17 11:38         ` Ariana.Lazar
2026-02-17 12:12           ` Andy Shevchenko
2026-02-15 17:58   ` Jonathan Cameron
2026-04-24  8:01     ` Ariana.Lazar
2026-04-24 10:19       ` Jonathan Cameron
2026-04-24 13:30         ` Ariana.Lazar
2026-04-24 16:48           ` Jonathan Cameron
2026-02-12 13:39 ` [PATCH 0/2] Add support for Microchip MCP48FxBy1/2/4/8 DAC with an SPI Interface Andy Shevchenko
2026-02-12 17:58   ` Conor Dooley

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20260212-germless-favoring-c27ab4c53128@spud \
    --to=conor@kernel.org \
    --cc=andy@kernel.org \
    --cc=ariana.lazar@microchip.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=dlechner@baylibre.com \
    --cc=jic23@kernel.org \
    --cc=krzk+dt@kernel.org \
    --cc=linux-iio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nuno.sa@analog.com \
    --cc=robh@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox