Received: by 2002:a05:7412:d024:b0:f9:90c9:de9f with SMTP id bd36csp106054rdb; Wed, 20 Dec 2023 07:25:59 -0800 (PST) X-Google-Smtp-Source: AGHT+IG3QoEiU1YcfNFyBZYmSYxXpIg3+ZDA+InzhIxasg2UFbeXdWJuPGBpfQOwvDrlpo6bysj6 X-Received: by 2002:a05:6102:ccf:b0:464:426c:a5c with SMTP id g15-20020a0561020ccf00b00464426c0a5cmr17357061vst.18.1703085959492; Wed, 20 Dec 2023 07:25:59 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1703085959; cv=none; d=google.com; s=arc-20160816; b=QwV8W+1gyiVavJDj8RA2frZCqkA+ujKy+eg7LGMFPEtc6h5ncy7Rht1AtfaJAa8Cv1 IW6uf7gWXww7ltrlsWAeOrwx+u/SszJ5HbcdtFqrZYnWDy9Q+lGYtn3vzeAlZvSQfdKb ThkEuuVWT/Z/WH5JST9+XLs/4i26U3O1xZbiEyfOkPOk6Sc5LjvahGGT91V8XxSWZMI3 +s3pN4/TV6jEeVPopoi4ru9oSuNmv/D32wnM86Ny99Jn4kb05EkSSLAXTIrgn1/cnwci /hgA+1hvbuvCTIVcMph9S2XOc2HfQminYTu5t8LGZZcr+3mSdHV4WBx6OMF4Hjn2PKxi 8WpQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=content-transfer-encoding:mime-version:list-unsubscribe :list-subscribe:list-id:precedence:references:in-reply-to:message-id :date:subject:cc:to:from:dkim-signature; bh=BNhR0jjyUXnJtDUmjrC6hxZFLVqudwmmtFtsktcySwI=; fh=L8+qPfL3NsKGJ8orkQBFSrMyVTAehy9wbFa2Ck2J/Cs=; b=lc62/tcPtVI0e7Uiqgbl+7/dLuG/y/foe9fSEAr2YeiwKP3aBV/I6vs4zmO1YBO03y QyFSFKNaGc/etCjtuifhDR7E9avVj7f70YqDBXeFZcOob1VzRt1BXHuxG/2A+I63mQe8 YRLLxkWAcma196vbudatPkNAdVrC5wQXOjaC6IUGAy0rJiZ11OmzXhkGue/B5zG5zZN6 Lo1LgdO8Fww8WuZRF8HQjZDA5WHNTKRI8xR+QGM2dCo4i9V3e5QUehp3ulV4ZFrz24pk QOYBivEXMtrbUxRDFcCOwK1iaL0d9OcwatN6Gsd3ks9yYcp/LT45d8isYt5KTJ0oz0L3 jQeQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=ML8wwK8w; spf=pass (google.com: domain of linux-kernel+bounces-7150-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-7150-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from ny.mirrors.kernel.org (ny.mirrors.kernel.org. [2604:1380:45d1:ec00::1]) by mx.google.com with ESMTPS id l11-20020a67ba0b000000b004668ec26ebbsi1114867vsn.189.2023.12.20.07.25.59 for (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 07:25:59 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel+bounces-7150-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20230601 header.b=ML8wwK8w; spf=pass (google.com: domain of linux-kernel+bounces-7150-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) smtp.mailfrom="linux-kernel+bounces-7150-linux.lists.archive=gmail.com@vger.kernel.org"; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ny.mirrors.kernel.org (Postfix) with ESMTPS id 2E6991C20CEF for ; Wed, 20 Dec 2023 15:25:59 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 5838E481DD; Wed, 20 Dec 2023 15:22:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="ML8wwK8w" X-Original-To: linux-kernel@vger.kernel.org Received: from mail-oo1-f46.google.com (mail-oo1-f46.google.com [209.85.161.46]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id DEF85481A2; Wed, 20 Dec 2023 15:22:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Received: by mail-oo1-f46.google.com with SMTP id 006d021491bc7-59067f03282so3869891eaf.0; Wed, 20 Dec 2023 07:22:26 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1703085745; x=1703690545; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=BNhR0jjyUXnJtDUmjrC6hxZFLVqudwmmtFtsktcySwI=; b=ML8wwK8w4dC3UHwYimJZwyK1pWydK9p0+Z3nYUOMuRTOQLFFSzO8andJU3amFVBEVa QqlNZrehHzO3N5u+QtMkQC9OcsoQtbeKldO41/1IuyCMCx56ALJ7R0gzCGgqLOA5v4yk RxrM5KfKfmAxKkmPO40ZPW5OeYqj0pLC0cew7Omh3D24e27B+gm6ubmjLw6Wr5U0k8Ms FdRfioVGUfQ4R5D03Id+gsQGY6pe+7eXzgAi67tAMiqBPqa9NDt+KLtuVqjE83bBneMl vAc2WwoTkk97Km0ZGB2FMDFsBBviO06TRd4D5bUdhIERN/h0tU/iF8XXigQOyV29v7TU 0/KQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1703085745; x=1703690545; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=BNhR0jjyUXnJtDUmjrC6hxZFLVqudwmmtFtsktcySwI=; b=kmgFGlhxMq74cGzz1RgNu99fxTFQz4pOLh8NMFEiJ0IXq1GVV2fesUULK67ReREYB1 1Y8GWcysFE0x74e3u7SZccUzU2jNX/5PyuWM8u4ebl8whRvBomxdKDGBIpOTkPELs040 IahiDReQsmwOrKj8HO93vJwS2ks3VzBK+uBfNTBz/YSF64AkR1wl+Oyn+AQL0FAoUPG6 9eNYX7PqfmdnX8Oi0Bym9lz80cfNLmIS8SWuKpXdIfa/FfvHHq/VruaXA5gqk9DlPhFn TGn5D12pXwstVui/4vlMSo8R1g/6zKV8wiI9PwZ91ljkVbd8eEPxJkapwUlKB8Inv6U1 5L3g== X-Gm-Message-State: AOJu0YwzYzZBD2hrJ0Z06xV+tODz/spWK8J6bWPVIAlIglAGdK0dj6PG AnQm4Tq+YD2m3Cpr9ENoIWnJTcldnrqcVQ== X-Received: by 2002:a05:6358:7f07:b0:170:f1bd:30e2 with SMTP id p7-20020a0563587f0700b00170f1bd30e2mr14803008rwn.34.1703085745120; Wed, 20 Dec 2023 07:22:25 -0800 (PST) Received: from localhost.localdomain ([2401:4900:581e:798e:871c:98db:5638:a4]) by smtp.gmail.com with ESMTPSA id 13-20020a63164d000000b005cd835182c5sm7304724pgw.79.2023.12.20.07.22.18 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 20 Dec 2023 07:22:24 -0800 (PST) From: Anshul Dalal To: linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org Cc: Anshul Dalal , Conor Dooley , Lars-Peter Clausen , Jonathan Cameron , Rob Herring , Krzysztof Kozlowski , linux-kernel-mentees@lists.linuxfoundation.org, Shuah Khan Subject: [PATCH v5 2/2] iio: dac: driver for MCP4821 Date: Wed, 20 Dec 2023 20:49:53 +0530 Message-ID: <20231220151954.154595-2-anshulusr@gmail.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20231220151954.154595-1-anshulusr@gmail.com> References: <20231220151954.154595-1-anshulusr@gmail.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Adds driver for the MCP48xx series of DACs. Device uses a simplex SPI channel. To set the value of an output channel, a 16-bit data of following format must be written: Bit field | Description 15 [MSB] | Channel selection bit 0 -> Channel A 1 -> Channel B 13 | Output Gain Selection bit 0 -> 2x Gain (Vref = 4.096V) 1 -> 1x Gain (Vref = 2.048V) 12 | Output Shutdown Control bit 0 -> Shutdown the selected channel 1 -> Active mode operation 11-0 [LSB]| DAC Input Data bits Value's big endian representation is taken as input for the selected DAC channel. For devices with a resolution of less than 12-bits, only the x most significant bits are considered where x is the resolution of the device. Reference: Page#22 [MCP48x2 Datasheet] Supported devices: +---------+--------------+-------------+ | Device | Resolution | Channels | |---------|--------------|-------------| | MCP4801 | 8-bit | 1 | | MCP4802 | 8-bit | 2 | | MCP4811 | 10-bit | 1 | | MCP4812 | 10-bit | 2 | | MCP4821 | 12-bit | 1 | | MCP4822 | 12-bit | 2 | +---------+--------------+-------------+ Devices tested: MCP4821 [12-bit single channel] MCP4802 [8-bit dual channel] Tested on Raspberry Pi Zero 2W Datasheet: https://ww1.microchip.com/downloads/en/DeviceDoc/22244B.pdf #MCP48x1 Datasheet: https://ww1.microchip.com/downloads/en/DeviceDoc/20002249B.pdf #MCP48x2 Signed-off-by: Anshul Dalal --- Changes for v5: - Removed unnecessary mutex - Removed unused channel fields initializations - Use saperate checks for validity of val and val2 in write_raw - Use cpu_to_be16 instead of put_unaligned_be16 Changes for v3,4: - no updates Changes for v2: - Changed device ordering to numerical - Fixed ordering of headers - Added struct mcp4821_chip_info instead of relying on driver_data from spi device ids - Use scoped_guards for all mutex locks - Changed write_val from __be16 to u16 in mcp4821_write_raw Fixes sparse warning: incorrect type in assignment Previous versions: v4: https://lore.kernel.org/lkml/20231219090252.818754-2-anshulusr@gmail.com/ v3: https://lore.kernel.org/lkml/20231218164735.787199-2-anshulusr@gmail.com/ v2: https://lore.kernel.org/lkml/20231217180836.584828-2-anshulusr@gmail.com/ v1: https://lore.kernel.org/lkml/20231117073040.685860-2-anshulusr@gmail.com/ --- MAINTAINERS | 7 ++ drivers/iio/dac/Kconfig | 10 ++ drivers/iio/dac/Makefile | 1 + drivers/iio/dac/mcp4821.c | 219 ++++++++++++++++++++++++++++++++++++++ 4 files changed, 237 insertions(+) create mode 100644 drivers/iio/dac/mcp4821.c diff --git a/MAINTAINERS b/MAINTAINERS index 81d5fc0bba68..8d9274c33c6e 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -13029,6 +13029,13 @@ F: Documentation/ABI/testing/sysfs-bus-iio-potentiometer-mcp4531 F: drivers/iio/potentiometer/mcp4018.c F: drivers/iio/potentiometer/mcp4531.c +MCP4821 DAC DRIVER +M: Anshul Dalal +L: linux-iio@vger.kernel.org +S: Maintained +F: Documentation/devicetree/bindings/iio/dac/microchip,mcp4821.yaml +F: drivers/iio/dac/mcp4821.c + MCR20A IEEE-802.15.4 RADIO DRIVER M: Stefan Schmidt L: linux-wpan@vger.kernel.org diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig index 93b8be183de6..34eb40bb9529 100644 --- a/drivers/iio/dac/Kconfig +++ b/drivers/iio/dac/Kconfig @@ -400,6 +400,16 @@ config MCP4728 To compile this driver as a module, choose M here: the module will be called mcp4728. +config MCP4821 + tristate "MCP4801/02/11/12/21/22 DAC driver" + depends on SPI + help + Say yes here to build the driver for the Microchip MCP4801 + MCP4802, MCP4811, MCP4812, MCP4821 and MCP4822 DAC devices. + + To compile this driver as a module, choose M here: the module + will be called mcp4821. + config MCP4922 tristate "MCP4902, MCP4912, MCP4922 DAC driver" depends on SPI diff --git a/drivers/iio/dac/Makefile b/drivers/iio/dac/Makefile index 5b2bac900d5a..55bf89739d14 100644 --- a/drivers/iio/dac/Makefile +++ b/drivers/iio/dac/Makefile @@ -42,6 +42,7 @@ obj-$(CONFIG_MAX5522) += max5522.o obj-$(CONFIG_MAX5821) += max5821.o obj-$(CONFIG_MCP4725) += mcp4725.o obj-$(CONFIG_MCP4728) += mcp4728.o +obj-$(CONFIG_MCP4821) += mcp4821.o obj-$(CONFIG_MCP4922) += mcp4922.o obj-$(CONFIG_STM32_DAC_CORE) += stm32-dac-core.o obj-$(CONFIG_STM32_DAC) += stm32-dac.o diff --git a/drivers/iio/dac/mcp4821.c b/drivers/iio/dac/mcp4821.c new file mode 100644 index 000000000000..028983f59cda --- /dev/null +++ b/drivers/iio/dac/mcp4821.c @@ -0,0 +1,219 @@ +// SPDX-License-Identifier: GPL-2.0-or-later +/* + * Copyright (C) 2023 Anshul Dalal + * + * Driver for Microchip MCP4801, MCP4802, MCP4811, MCP4812, MCP4821 and MCP4822 + * + * Based on the work of: + * Michael Welling (MCP4922 Driver) + * + * Datasheet: + * MCP48x1: https://ww1.microchip.com/downloads/en/DeviceDoc/22244B.pdf + * MCP48x2: https://ww1.microchip.com/downloads/en/DeviceDoc/20002249B.pdf + * + * TODO: + * - Configurable gain + * - Regulator control + */ + +#include +#include +#include + +#include +#include + +#include + +#define MCP4821_ACTIVE_MODE BIT(12) +#define MCP4802_SECOND_CHAN BIT(15) + +/* DAC uses an internal Voltage reference of 4.096V at a gain of 2x */ +#define MCP4821_2X_GAIN_VREF_MV 4096 + +enum mcp4821_supported_drvice_ids { + ID_MCP4801, + ID_MCP4802, + ID_MCP4811, + ID_MCP4812, + ID_MCP4821, + ID_MCP4822, +}; + +struct mcp4821_state { + struct spi_device *spi; + u16 dac_value[2]; +}; + +struct mcp4821_chip_info { + const char *name; + int num_channels; + const struct iio_chan_spec channels[2]; +}; + +#define MCP4821_CHAN(channel_id, resolution) \ + { \ + .type = IIO_VOLTAGE, .output = 1, .indexed = 1, \ + .channel = (channel_id), \ + .info_mask_separate = BIT(IIO_CHAN_INFO_RAW), \ + .info_mask_shared_by_type = BIT(IIO_CHAN_INFO_SCALE), \ + .scan_type = { \ + .realbits = (resolution), \ + .shift = 12 - (resolution), \ + }, \ + } + +static const struct mcp4821_chip_info mcp4821_chip_info_table[6] = { + [ID_MCP4801] = { + .name = "mcp4801", + .num_channels = 1, + .channels = { MCP4821_CHAN(0, 8) } + }, + [ID_MCP4802] = { + .name = "mcp4802", + .num_channels = 2, + .channels = { MCP4821_CHAN(0, 8), + MCP4821_CHAN(1, 8) } + }, + [ID_MCP4811] = { + .name = "mcp4811", + .num_channels = 1, + .channels = { MCP4821_CHAN(0, 10) } + }, + [ID_MCP4812] = { + .name = "mcp4812", + .num_channels = 2, + .channels = { MCP4821_CHAN(0, 10), + MCP4821_CHAN(1, 10) } + }, + [ID_MCP4821] = { + .name = "mcp4821", + .num_channels = 1, + .channels = { MCP4821_CHAN(0, 12) } + }, + [ID_MCP4822] = { + .name = "mcp4822", + .num_channels = 2, + .channels = { MCP4821_CHAN(0, 12), + MCP4821_CHAN(1, 12) } + }, +}; + +static int mcp4821_read_raw(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, int *val, + int *val2, long mask) +{ + struct mcp4821_state *state; + + switch (mask) { + case IIO_CHAN_INFO_RAW: + state = iio_priv(indio_dev); + *val = state->dac_value[chan->channel]; + return IIO_VAL_INT; + case IIO_CHAN_INFO_SCALE: + *val = MCP4821_2X_GAIN_VREF_MV; + *val2 = chan->scan_type.realbits; + return IIO_VAL_FRACTIONAL_LOG2; + default: + return -EINVAL; + } +} + +static int mcp4821_write_raw(struct iio_dev *indio_dev, + struct iio_chan_spec const *chan, int val, + int val2, long mask) +{ + struct mcp4821_state *state = iio_priv(indio_dev); + u16 write_val; + __be16 write_buffer; + int ret; + + if (val2 != 0) + return -EINVAL; + if (val < 0 || val >= BIT(chan->scan_type.realbits)) + return -EINVAL; + if (mask != IIO_CHAN_INFO_RAW) + return -EINVAL; + + write_val = MCP4821_ACTIVE_MODE | val << chan->scan_type.shift; + if (chan->channel) + write_val |= MCP4802_SECOND_CHAN; + write_buffer = cpu_to_be16(write_val); + ret = spi_write(state->spi, &write_buffer, sizeof(write_buffer)); + if (ret) { + dev_err(&state->spi->dev, "Failed to write to device: %d", ret); + return ret; + } + + state->dac_value[chan->channel] = val; + return 0; +} + +static const struct iio_info mcp4821_info = { + .read_raw = &mcp4821_read_raw, + .write_raw = &mcp4821_write_raw, +}; + +static int mcp4821_probe(struct spi_device *spi) +{ + struct iio_dev *indio_dev; + struct mcp4821_state *state; + const struct mcp4821_chip_info *info; + + indio_dev = devm_iio_device_alloc(&spi->dev, sizeof(*state)); + if (indio_dev == NULL) + return -ENOMEM; + + state = iio_priv(indio_dev); + state->spi = spi; + + info = spi_get_device_match_data(spi); + indio_dev->name = info->name; + indio_dev->info = &mcp4821_info; + indio_dev->modes = INDIO_DIRECT_MODE; + indio_dev->channels = info->channels; + indio_dev->num_channels = info->num_channels; + return devm_iio_device_register(&spi->dev, indio_dev); +} + +#define MCP4821_COMPATIBLE(of_compatible, id) \ + { \ + .compatible = of_compatible, \ + .data = &mcp4821_chip_info_table[id] \ + } + +static const struct of_device_id mcp4821_of_table[] = { + MCP4821_COMPATIBLE("microchip,mcp4801", ID_MCP4801), + MCP4821_COMPATIBLE("microchip,mcp4802", ID_MCP4802), + MCP4821_COMPATIBLE("microchip,mcp4811", ID_MCP4811), + MCP4821_COMPATIBLE("microchip,mcp4812", ID_MCP4812), + MCP4821_COMPATIBLE("microchip,mcp4821", ID_MCP4821), + MCP4821_COMPATIBLE("microchip,mcp4822", ID_MCP4822), + { /* Sentinel */ } +}; +MODULE_DEVICE_TABLE(of, mcp4821_of_table); + +static const struct spi_device_id mcp4821_id_table[] = { + { "mcp4801", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4801]}, + { "mcp4802", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4802]}, + { "mcp4811", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4811]}, + { "mcp4812", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4812]}, + { "mcp4821", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4821]}, + { "mcp4822", (kernel_ulong_t)&mcp4821_chip_info_table[ID_MCP4822]}, + { /* Sentinel */ } +}; +MODULE_DEVICE_TABLE(spi, mcp4821_id_table); + +static struct spi_driver mcp4821_driver = { + .driver = { + .name = "mcp4821", + .of_match_table = mcp4821_of_table, + }, + .probe = mcp4821_probe, + .id_table = mcp4821_id_table, +}; +module_spi_driver(mcp4821_driver); + +MODULE_AUTHOR("Anshul Dalal "); +MODULE_DESCRIPTION("Microchip MCP4821 DAC Driver"); +MODULE_LICENSE("GPL"); -- 2.43.0