From patchwork Sun Dec 5 11:40:42 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Cosmin Tanislav X-Patchwork-Id: 520820 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 18376C433F5 for ; Sun, 5 Dec 2021 11:40:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232965AbhLELoU (ORCPT ); Sun, 5 Dec 2021 06:44:20 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:55020 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232600AbhLELoT (ORCPT ); Sun, 5 Dec 2021 06:44:19 -0500 Received: from mail-ed1-x52d.google.com (mail-ed1-x52d.google.com [IPv6:2a00:1450:4864:20::52d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7369CC061714; Sun, 5 Dec 2021 03:40:52 -0800 (PST) Received: by mail-ed1-x52d.google.com with SMTP id o20so31108074eds.10; Sun, 05 Dec 2021 03:40:52 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=bwuCUsjh7lJNswc/cVje4/4By3QPNRl2d07hrK1ln/8=; b=fwUfzOKrWO26ZC0xRcXplK4j2giBXWd0mGQVw6GR700WjTvCVz+6UBnKlVrPgkuPzK 3iRGFdaCphg3ccR9DFKSW2s3XFSUOIQo0WJLq3WX5/NgGi9+A7CqSiEVFVoZf05nUcNG HeJjOJBngEWE4WO9bv++LbP3ajb4CWlJLH68RhjWm+VUvB0ROSmttB+1Q17hK+eQidcB QuVJG9u0oKrzK4lSJAx9gBYueZIopE9WEXUiCiMU2oIlzeVYQu4ayeqG/kMTku2s3n91 iSwqGLqY0vlilNSXOnsA980ujstAPGkAVjmxZ6CQD/IWD0Gzxxj144m5ySQE8KIqO3/0 Nc9g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=bwuCUsjh7lJNswc/cVje4/4By3QPNRl2d07hrK1ln/8=; b=LpCXpTuxYHMJHjOi5ORmFr/pw29tq2f3kxO2gsbU9pMwSAGM2t1YUrRn425e2Mtjem THFFDI0hccnjHBA5w4Y1uiylROERxnMS0cf2ZBtynZL4naG09MsW17RTv3ikh9o2a3jO QBXcFxdclbeHiI2Ry5DuuWH4KvhUbeB5D0CvN5dmPzTsVQ7lsgPWuew0TXyDqFnEO6TX kuSthUHBFjK4isR1Af+oRmeWqhW4J9Y27Yw7HKW/IoZJe/xvxLNiPsaHhZCHXNLpgjsJ OyTbau/qoDtJsuVuZfSxJG9h4c4yltSiPzkT1tggajavkxo9dAeIDHXsk8NHHrB1AUr2 yY9w== X-Gm-Message-State: AOAM533EPK+WVKyn0Dkwynb2OrRtxXupemaXU3lzg9mg2MhNMjNg+yE2 EXJ2N+Ps4cCBRKIF4VCSK8u80dlOXF9wfA== X-Google-Smtp-Source: ABdhPJySzBiQcRsiZeWJmCokh+rOXU76ofo+Ru8NjFoJoqt4Xb3+3QbHcioO3nNNEN6ZGKmEsCwazw== X-Received: by 2002:a17:907:7d86:: with SMTP id oz6mr37384815ejc.312.1638704450979; Sun, 05 Dec 2021 03:40:50 -0800 (PST) Received: from demon-pc.localdomain ([188.24.96.74]) by smtp.gmail.com with ESMTPSA id g11sm5870469edz.53.2021.12.05.03.40.49 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 05 Dec 2021 03:40:50 -0800 (PST) From: Cosmin Tanislav X-Google-Original-From: Cosmin Tanislav Cc: cosmin.tanislav@analog.com, demonsingur@gmail.com, Lars-Peter Clausen , Michael Hennerich , Rob Herring , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Linus Walleij , Bartosz Golaszewski , linux-gpio@vger.kernel.org Subject: [PATCH v11 0/3] Add AD74413R driver Date: Sun, 5 Dec 2021 13:40:42 +0200 Message-Id: <20211205114045.173612-1-cosmin.tanislav@analog.com> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 To: unlisted-recipients:; (no To-header on input) Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org V1 -> V2 * sign off using company email V2 -> V3 * replace gpo config firmware flag with flag specifying whether gpo is in comparator mode * create two separate gpiochips, one output-only gpiochip for GPO pins not in comparator mode and one input-only for the value of digital input channels * wire up all gpo functionalities using pinconf * keep number of characters per line under 80 * rework locking * do not invalidate other chip revisions * do not set indio device parent * print probe error for refin regulator * move conversion from range register value to range / offset / raw offset into separate function * module.h -> mod_devicetable.h * use generic firmware interface functions * add comment regarding cache alignment * add comment regarding ADC channels buffered read setup * un-inline comment regarding 100us delay for conversion start * inline return statements * remove assignments to val2 where not necessary * local_channels -> chans * index -> i * channel_config -> config * IIO_ALTVOLTAGE -> IIO_VOLTAGE * .info_mask_shared_by_type_available -> .info_mask_separate_available * remove unlikely probe error messages * use an array indexed by channel function for retrieving iio channels * count iio channels while parsing * move HART rate rejection outside of setter * move channel function validation outside of setter * use SPI messages for read and write * validate DAC code earlier * simplify switches to only handle existing iio channels * pass indio_dev into functions needing access to it * pass spi into devm_regmap_init * dt-bindings: sort compatibles * dt-bindings: remove driver word from description * dt-bindings: remove refin supply description * dt-bindings: specify channel function default value * dt-bindings: remove maxItems from scalar value V3 -> v4 * remove double gpo from macro name * reset at probe * config -> chip_info and store chip name inside chip info * cacheline align every DMA buffer * simplify generation of adc samples message by caching xfer, tx_buf and rx_buf * use mask itself for writing the value of channel enable and gpo data * move reg read and write transfers to the same buffers and use local variables for transfers * merge the two for loops handling gpio configuration * let firmware decide irq edge * remove INDIO_BUFFER_SOFTWARE already set by iio framework * do not set trigger device parent * return dev_err_probe for regulator error case * do not set cs_change to 0 when not needed * do not set spi device drvdata as it is not needed * fix bug regarding wrong channels being created for resistance input, digital input, and current input with hart * use voltage input channels spec for high impedance mode * put () around macro parameters * merge AD74413R_CHANNEL macro into its uses * remove unused switch case scope * inline return IIO_VAL_INT * use {get,put}_unaligned_be16 * use proper types for reg and val * move default case handling into switch statements * pass driver state into regmap functions * use genmask for generating a 16bit max value * alphanumeric order for part numbers * dt-bindings: remove $ref from ohms value V4 -> V5 * dt-bindings: include headers necessary * dt-bindings: add IRQ_TYPE_EDGE_FALLING to interrupt flags * dt-bindings: ohm -> ohms * dt-bindings: spi0 -> spi V5 -> V6 * fix warnings regarding overflows V6 -> V7 * remove extra cache-line alignment * adi,rsense-resistance-ohms -> shunt-resistor-micro-ohms * dt-bindings: add product page links V7 -> V8 * also check DAC code lower bound * fix checkpath --strict complaints * add comment regarding mutex lock usage * propagate error when converting adc result to resistance V8 -> V9 * fix spelling mistake * undo propagate error when converting adc result to resistance * return void from adc result to resistance function * limit max adc value when doing resistance calculation to avoid a potential division-by-zero case V9 -> V10 * pick up Reviewed-By tags * fix odd alignment in header * add dev_err_probe to irq request error path V10 -> V11 * fix compile error Cosmin Tanislav (3): iio: add addac subdirectory dt-bindings: iio: add AD74413R iio: addac: add AD74413R driver .../bindings/iio/addac/adi,ad74413r.yaml | 158 ++ MAINTAINERS | 9 + drivers/iio/Kconfig | 1 + drivers/iio/Makefile | 1 + drivers/iio/addac/Kconfig | 20 + drivers/iio/addac/Makefile | 7 + drivers/iio/addac/ad74413r.c | 1475 +++++++++++++++++ include/dt-bindings/iio/addac/adi,ad74413r.h | 21 + 8 files changed, 1692 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/addac/adi,ad74413r.yaml create mode 100644 drivers/iio/addac/Kconfig create mode 100644 drivers/iio/addac/Makefile create mode 100644 drivers/iio/addac/ad74413r.c create mode 100644 include/dt-bindings/iio/addac/adi,ad74413r.h