From: Zhiyong Tao <zhiyong.tao@mediatek.com>
To: robh+dt@kernel.org, jic23@kernel.org, knaack.h@gmx.de,
lars@metafoo.de, pmeerw@pmeerw.net
Cc: srv_heupstream@mediatek.com, liguo.zhang@mediatek.com,
yingjoe.chen@mediatek.com, eddie.huang@mediatek.com,
erin.lo@mediatek.com, dawei.chien@mediatek.com,
matthias.bgg@gmail.com, s.hauer@pengutronix.de,
devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, linux-iio@vger.kernel.org,
linux-mediatek@lists.infradead.org
Subject: [PATCH v4 0/3] AUXADC: Mediatek auxadc driver
Date: Mon, 11 Jul 2016 14:39:54 +0800 [thread overview]
Message-ID: <1468219197-9572-1-git-send-email-zhiyong.tao@mediatek.com> (raw)
This series includes three patches:
1.Change the device tree binding documentation.
2.Add auxadc driver based on linux iio.
3.Add auxadc nodes in the mediatek MT2701 dtsi file.
Changes in patch v4:
1).Change the file name 'mt65xx_auxdc.txt' to 'mt6577_auxdc.txt'.
2).Remove 'break' after 'return' in 'mt6577_auxadc_read_raw'.
3).Use 'mt2701' instead of 'auxadc' in dts commit message.
4).Change 'auxadc@' to 'adc@' in mt2701.dtsi.
Changes in patch v3:
1).Add '#' before 'io-channel-cells' and change 'auxadc@' to 'adc@' in auxadc binding document.
2).Fix CONFIG_MEDIATEK_MT65XX_AUXADC sequence as alphabetical order.
3).Change the prefix 'MT65XX' to 'MT6577' to match auxadc driver name.
4).Remove the delays stored in 'mt6577_auxadc_device' and use the defined delay directly.
5).Use '-ETIMEDOUT' instead of '-EINVAL'.
6).Return value directly in 'mt6577_auxadc_read_raw'.
7).Add power off and unprepare clock if 'iio_device_register' failed.
8).Change 'Registers' to 'Register'.
Changes in patch v2:
1).Change the file name mt65xx_auxadc.c to mt6577_auxadc.c.
2).Add 'depends on ARCH_MEDIATEK || COMPILE_TEST and HAS_IOMEM' for MEDIATEK_MT65XX_AUXADC config.
3).Add check case for return value of 'readl_poll_timeout'.
4).Remove returning 1 and 0 for SCALE and OFFSET. use channel type 'IIO_CHAN_INFO_PROCESSED' instead of 'IIO_CHAN_INFO_RAW'.
5).Remove 'IIO_CHAN_INFO_OFFSET' and 'IIO_CHAN_INFO_SCALE' in 'MT65XX_AUXADC_CHANNEL' channel type.
6).Add check case when auxadc power on failure. if auxadc power on failure, clk_disable_unprepare should be executed.
7).Put 'platform_set_drvdata' before 'iio_device_register'.
8).Fix coding style and spelling error.
Zhiyong Tao (3):
dt-bindings: auxadc: Add binding document for Mediatek auxadc.
iio: adc: mt2701: Add Mediatek auxadc driver for mt2701.
arm: dts: mt2701: Add auxadc node.
.../devicetree/bindings/iio/adc/mt6577_auxadc.txt | 27 ++
.../devicetree/bindings/soc/mediatek/auxadc.txt | 21 --
arch/arm/boot/dts/mt2701.dtsi | 9 +
drivers/iio/adc/Kconfig | 13 +
drivers/iio/adc/Makefile | 1 +
drivers/iio/adc/mt6577_auxadc.c | 294 ++++++++++++++++++++
6 files changed, 344 insertions(+), 21 deletions(-)
create mode 100644 Documentation/devicetree/bindings/iio/adc/mt6577_auxadc.txt
delete mode 100644 Documentation/devicetree/bindings/soc/mediatek/auxadc.txt
create mode 100644 drivers/iio/adc/mt6577_auxadc.c
next reply other threads:[~2016-07-11 6:39 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-07-11 6:39 Zhiyong Tao [this message]
2016-07-11 6:39 ` [PATCH v4 1/3] dt-bindings: auxadc: Add binding document for Mediatek auxadc Zhiyong Tao
2016-07-12 2:43 ` zhiyong tao
2016-07-16 19:31 ` Rob Herring
[not found] ` <1468219197-9572-1-git-send-email-zhiyong.tao-NuS5LvNUpcJWk0Htik3J/w@public.gmane.org>
2016-07-11 6:39 ` [PATCH v4 2/3] iio: adc: mt2701: Add Mediatek auxadc driver for mt2701 Zhiyong Tao
2016-07-18 17:12 ` Jonathan Cameron
[not found] ` <82fa5ffa-52be-e1ec-83dd-dd92b3f3ef5d-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
2016-07-20 6:02 ` zhiyong tao
2016-07-20 6:24 ` zhiyong tao
2016-07-11 6:39 ` [PATCH v4 3/3] arm: dts: mt2701: Add auxadc node Zhiyong Tao
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=1468219197-9572-1-git-send-email-zhiyong.tao@mediatek.com \
--to=zhiyong.tao@mediatek.com \
--cc=dawei.chien@mediatek.com \
--cc=devicetree@vger.kernel.org \
--cc=eddie.huang@mediatek.com \
--cc=erin.lo@mediatek.com \
--cc=jic23@kernel.org \
--cc=knaack.h@gmx.de \
--cc=lars@metafoo.de \
--cc=liguo.zhang@mediatek.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=matthias.bgg@gmail.com \
--cc=pmeerw@pmeerw.net \
--cc=robh+dt@kernel.org \
--cc=s.hauer@pengutronix.de \
--cc=srv_heupstream@mediatek.com \
--cc=yingjoe.chen@mediatek.com \
/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;
as well as URLs for NNTP newsgroup(s).