Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752858AbaKCPYU (ORCPT ); Mon, 3 Nov 2014 10:24:20 -0500 Received: from ns.mm-sol.com ([37.157.136.199]:60258 "EHLO extserv.mm-sol.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752036AbaKCPYP (ORCPT ); Mon, 3 Nov 2014 10:24:15 -0500 From: "Ivan T. Ivanov" To: Jonathan Cameron , Hartmut Knaack , Lars-Peter Clausen , Peter Meerwald Cc: "Ivan T. Ivanov" , Angelo Compagnucci , Arnd Bergmann , Grant Likely , Rob Herring , linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org Subject: [PATCH v4 0/2] Initial support for voltage ADC Date: Mon, 3 Nov 2014 17:24:28 +0200 Message-Id: <1415028270-25860-1-git-send-email-iivanov@mm-sol.com> X-Mailer: git-send-email 1.9.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, This is the forth version of this driver. V3 could be found here [1]. Changes since v3. - Addressed review comments from Hartmut Knaack and Mark Rutland: Better description of 'reg' property. Add # to address-cells and size-cells DT document. Dropped interrupt-names property, it was not actually used by driver. DT header file, which contain channels names definitions, is part of documentation patch now. Clarified 'qcom,decimation' property. Hopefully it make sense now. Make more explicit that all reference channels configuration nodes have to be defined in DT. - Promote dev_dbg to dev_err in error paths. Stan, I hope you are ok with this :-) - Use 32 bit calculations in calibration routine. - Read ADC result with one regmap call. - Show status and state registers if ADC conversin fails. Errors should not normally happen, so I will like to see content of this registers in such cases. - Use new struct iio_info::of_xlate for channel number translation. - Removed device wakeup initialization code, which didn't make sense to me for such kind of device. Patches depend on "iio: inkern: Add of_xlate function to struct iio_info" patch, which is included in IIO testing branch [2]. [1] http://comments.gmane.org/gmane.linux.ports.arm.msm/9741 [2] http://git.kernel.org/cgit/linux/kernel/git/jic23/iio.git/log/?h=testing Stanimir Varbanov (2): DT: iio: vadc: document dt binding iio: vadc: Qualcomm SPMI PMIC voltage ADC driver .../devicetree/bindings/iio/adc/qcom,spmi-vadc.txt | 129 +++ drivers/iio/adc/Kconfig | 14 + drivers/iio/adc/Makefile | 1 + drivers/iio/adc/qcom-spmi-vadc.c | 1015 ++++++++++++++++++++ include/dt-bindings/iio/qcom,spmi-vadc.h | 119 +++ 5 files changed, 1278 insertions(+) create mode 100644 Documentation/devicetree/bindings/iio/adc/qcom,spmi-vadc.txt create mode 100644 drivers/iio/adc/qcom-spmi-vadc.c create mode 100644 include/dt-bindings/iio/qcom,spmi-vadc.h -- 1.9.1 -- 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/