From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.6 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_2 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7FC42C10DCE for ; Sun, 15 Mar 2020 13:06:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4165E205C9 for ; Sun, 15 Mar 2020 13:06:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1584277593; bh=bXTzl89vOQzkCNSNfVMXf5mh6a/68EqvlMu7nnmuKNc=; h=Date:From:To:Cc:Subject:In-Reply-To:References:List-ID:From; b=rupOT6gqAZb8sl0GHMu9bzfk2r7RE8tIpyEDWOqxzIQUlCmqOgBr/zc2NX/DMvE+5 bhnDUe17KLGV1knZ3dUM5yb6bzxb3a0GL8dJaeY4A5lvZv41uo5Nf74ympjIWZmAi4 n/rAs9fYt3dJZ7MeM+dz6Ds2nGjx+zclbeX6EJ+M= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728571AbgCONGc (ORCPT ); Sun, 15 Mar 2020 09:06:32 -0400 Received: from mail.kernel.org ([198.145.29.99]:46296 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728541AbgCONGc (ORCPT ); Sun, 15 Mar 2020 09:06:32 -0400 Received: from archlinux (cpc149474-cmbg20-2-0-cust94.5-4.cable.virginm.net [82.4.196.95]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 2B60020575; Sun, 15 Mar 2020 13:06:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1584277591; bh=bXTzl89vOQzkCNSNfVMXf5mh6a/68EqvlMu7nnmuKNc=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=hE7n/BSoGNQtcA2MCHyINiyu4i7ORQ/UM1EIpWoDyaeoYFIDiQhnlndjV86RxWged k1cawE6QKA3KkNefAxMfQhR+c0vg7j+NREs5ZJHmchlK4MOhcKfEZrxyJzIwfmwL0u IwF8cnGkjsC3REIpDO6iUrNsEMFANQ66tE2uqNtY= Date: Sun, 15 Mar 2020 13:06:25 +0000 From: Jonathan Cameron To: Alexandru Ardelean Cc: linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, robh+dt@kernel.org, Laszlo.Nagy@analog.com, Andrei.Grozav@analog.com, Michael.Hennerich@analog.com, Istvan.Csomortani@analog.com, Adrian.Costina@analog.com, Dragos.Bogdan@analog.com, Alexandru Ardelean Subject: Re: [PATCH v9 0/8] iio: adi-axi-adc,ad9647: Add support for AD9467 ADC Message-ID: <20200315130625.705548ce@archlinux> In-Reply-To: <20200312083511.28832-1-alexandru.ardelean@analog.com> References: <20200312083511.28832-1-alexandru.ardelean@analog.com> X-Mailer: Claws Mail 3.17.4 (GTK+ 2.24.32; x86_64-pc-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-iio-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-iio@vger.kernel.org On Thu, 12 Mar 2020 10:35:03 +0200 Alexandru Ardelean wrote: > This changeset adds support for the AD9467 LVDS High-Speed ADC. > In order to support it, support for an FPGA ADI AXI ADC is added in this > set. > This uses the current support for IIO buffer DMAEngine. I took a look through this series and I'm think I'm now happy with it. However, I'd like to leave a bit of time for Rob, the DT binding Maintainer to take a look at the bindings. Long term I sort of wonder if we might get some alternative for the ADI-AXI interface, either because Analog decide to rewrite it, or because someone else puts together similar IP. At that point we may want a more sophisticated registration framework etc. I guess we can wait and see if anyone ever does that though. Jonathan > > Changelog v8 -> v9: > * adding more Analog people to the list; predominantly HDL people; this > should help me sync people about the details of regs/reg-names > * added 'Acked-by: Moritz Fischer ' tag to fpga patches > - we can always re-update these patches if something else is decided about > the location of the 'adi-axi-common.h' header; I'm not insisting about > where to put it; I'm open to other proposals > * patch 'iio: adc: adi-axi-adc: add support for AXI ADC IP core' > - prefixed regs ADI_AXI_ ; I tried ADI_AXI_ADC_, but that seemed to make > them too long > - dropped unused regs; will add them as stuff gets added in the upstream > driver; in the meantime, reg-names can be reworked > - dropped generic LOWERXY_SET/GET macros > - update reg-names a bit; will update them in the docs and HDL > - order in adi_axi_adc_conv_unregister() should now be symmetrically > oppposite now to the register function > - implemented 'is_visible()' callback to adi_axi_adc_attributes[] so that > attrs can be made invisible to userspace if needed; > - 'indio_dev->name = "adi-axi-adc";' > - added kernel doc-string for @reg_access > * patch 'iio: adc: ad9467: add support AD9467 ADC' > - ad9467_spi_read() split in 2 buffers; tbuf & rbuf > - removed 'if (chan->extend_name)' test ; left-over from initial driver > - removed 'if (!st->clk)' check; driver will fail probe without a clock > - removed 'if (!spi->dev.of_node)' in probe; shouldn't be needed > - using 'of_device_get_match_data()' in probe to get data; moved chip > info table entry as data on the of_device_id table > > Changelog v7 -> v8: > * in 'iio: adc: adi-axi-adc: add support for AXI ADC IP core' > - updated register definitions and bits to newer format/docs; the ref driver wasn't really up-to-date > -- prefixed bit names with reg-name to avoid bit definition colisions; that makes some macros longer, but at least the format is consistent > - using dev_name(&pdev->dev) for indio_dev->name > - moved reset to own axi_adc_reset() function; may be re-used later > - some re-formatting/alignment changes > - address ENOSYS checkpatch complaint; changed with EOPNOTSUPP > > Changelog v6 -> v7: > * Fixed dt-schema build for adi,axi-adc.yaml based on Rob's suggestion > - added '$ref: /schemas/types.yaml#/definitions/phandle' to 'adi,adc-dev' > - dropped 'maxItems' from 'adi,adc-dev' > > Changelog v5 -> v6 > * fix URLs; got changed during rename > https://wiki.analog.com/resources/fpga/docs/adi_axi_adc_ip -> > https://wiki.analog.com/resources/fpga/docs/axi_adc_ip > - noticed while working on the AXI DAC driver > > Changelog v4 -> v5: > * update drivers/iio/adc/Kconfig note about module name; omitted during first rename > - 'module will be called axi-adc.' -> 'module will be called adi-axi-adc.' > > Changelog v3 -> v4: > * addressed Rob's dt-remarks > - change 'adi-axi-adc-client' prop to 'adi,adc-dev' > > Changelog v2 -> v3: > * addressed compiler warning > > Changelog v1 -> v2: > * first series was added a bit hastily > * addressed 'make dt_binding_check' complaints; seems I missed a few when running the check; > * added missing patches to include/linux/fpga/adi-axi-common.h > - 'include: fpga: adi-axi-common.h: fixup whitespace tab -> space' > - 'include: fpga: adi-axi-common.h: add version helper macros' > * patch 'iio: buffer-dmaengine: add dev-managed calls for buffer alloc/free' > - remove copy+pasted comment for 'devm_iio_dmaengine_buffer_alloc()' > - removed devm_iio_dmaengine_buffer_free() ; hopefully it might never be needed > - fix-up alignment for devm_iio_dmaengine_buffer_alloc() in header > * patch 'iio: adc: adi-axi-adc: add support for AXI ADC IP core' > - renamed axi-adc.c -> adi-axi-adc.c & Kconfig symbol > - prefix all axi_adc -> adi_axi_adc > - removed switch statement in axi_adc_read_raw() & axi_adc_write_raw() > - remove axi_adc_chan_spec ; replaced with iio_chan_spec directly ; will think of a simpler solution for extra chan params > - removed left-over 'struct axi_adc_cleanup_data' > - moved 'devm_add_action_or_reset()' call right after 'adi_axi_adc_attach_client()' > - switched to using 'devm_platform_ioremap_resource()' > * patch 'iio: adc: ad9467: add support AD9467 ADC' > - renamed ADI_ADC reg prefixes to AN877_ADC > - dropped 'info_mask_separate' field in AD9467_CHAN - will be re-added later when driver gets more features; was left-over from the initial ref driver > - remove .shift = 0, in AD9467_CHAN > - renamed 'sample-clock' -> 'adc-clock' > - direct returns in ad9467_read_raw() & ad9467_write_raw() & ad9467_setup() switch statements > - removed blank line after devm_axi_adc_conv_register() > - removed ad9467_id & reworked to use ad9467_of_match > > Alexandru Ardelean (6): > include: fpga: adi-axi-common.h: fixup whitespace tab -> space > include: fpga: adi-axi-common.h: add version helper macros > iio: buffer-dmaengine: use %zu specifier for sprintf(align) > iio: buffer-dmaengine: add dev-managed calls for buffer alloc > dt-bindings: iio: adc: add bindings doc for AXI ADC driver > dt-bindings: iio: adc: add bindings doc for AD9467 ADC > > Michael Hennerich (2): > iio: adc: adi-axi-adc: add support for AXI ADC IP core > iio: adc: ad9467: add support AD9467 ADC > > .../bindings/iio/adc/adi,ad9467.yaml | 65 +++ > .../bindings/iio/adc/adi,axi-adc.yaml | 63 +++ > drivers/iio/adc/Kconfig | 35 ++ > drivers/iio/adc/Makefile | 2 + > drivers/iio/adc/ad9467.c | 420 ++++++++++++++ > drivers/iio/adc/adi-axi-adc.c | 518 ++++++++++++++++++ > .../buffer/industrialio-buffer-dmaengine.c | 41 +- > include/linux/fpga/adi-axi-common.h | 6 +- > include/linux/iio/adc/adi-axi-adc.h | 64 +++ > include/linux/iio/buffer-dmaengine.h | 3 + > 10 files changed, 1215 insertions(+), 2 deletions(-) > create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,ad9467.yaml > create mode 100644 Documentation/devicetree/bindings/iio/adc/adi,axi-adc.yaml > create mode 100644 drivers/iio/adc/ad9467.c > create mode 100644 drivers/iio/adc/adi-axi-adc.c > create mode 100644 include/linux/iio/adc/adi-axi-adc.h >