From: Archit Taneja <architt@codeaurora.org>
To: Sricharan R <sricharan@codeaurora.org>,
devicetree@vger.kernel.org, linux-arm-msm@vger.kernel.org,
agross@codeaurora.org, linux-kernel@vger.kernel.org,
linux-i2c@vger.kernel.org, iivanov@mm-sol.com,
galak@codeaurora.org, dmaengine@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, andy.gross@linaro.org,
ntelkar@codeaurora.org, wsa@the-dreams.de
Cc: srichara@codeaurora.org
Subject: Re: [PATCH V6 0/6] i2c: qup: Add support for v2 tags and bam dma
Date: Tue, 12 Jan 2016 15:49:43 +0530 [thread overview]
Message-ID: <5694D33F.7040205@codeaurora.org> (raw)
In-Reply-To: <1452588233-1803-1-git-send-email-sricharan@codeaurora.org>
On 01/12/2016 02:13 PM, Sricharan R wrote:
> i2c: qup: Add support for v2 tags and bam dma
>
> QUP from version 2.1.1 onwards, supports a new format of i2c command tags.
> Tag codes instructs the controller to perform a operation like read/write.
> This new tagging version supports and is required for adding bam dma
> capabilities. V2 tags supports transfer of more than 256 bytes in a single i2c
> transaction. Also adding bam dma support facilitates transferring each i2c_msg
> in i2c_msgs without a 'stop' bit in between which is required for some of
> the clients.
>
> Tested this series on apq8074 dragon board eeprom client on i2c bus1
>
> [V6] Added review tags and fixed a checkpatch warning in patch 4/6.
>
> [V5] Addressed few more comments from Ivan T. Ivanov.
> Squashed patch 2 and 3 as no point in having only few lines of
> common code between v1 and v2 tags for increased complexity.
> Couple of non functional review comments fixes in patch 3, 4.
> Added a change in patch 4 to have proper transfer completion in
> a corner case. patch 5, 6 unchanged.
>
> [V4] Added a patch to factor out some common code.
> Removed support for freq > 400KHZ as per comments.
> Addressed comments from Ivan T. Ivanov to keep the code for
> V2 support in a separate path.
> Changed the authorship of V2 tags support patch.
>
> [V3] Added support to coalesce each i2c_msg in i2c_msgs for fifo and
> block mode in Patch 2. Also addressed further code comments.
>
> http://comments.gmane.org/gmane.linux.drivers.i2c/22497
>
> [V2] Addressed comments from Ivan T. Ivanov, Andy Gross [v1] Initial Version
For the series:
Tested-by: Archit Taneja <architt@codeaurora.org>
>
> Sricharan R (6):
> i2c: qup: Change qup_wait_writeready function to use for all timeouts
> i2c: qup: Add V2 tags support
> i2c: qup: Transfer each i2c_msg in i2c_msgs without a stop bit
> i2c: qup: Add bam dma capabilities
> dts: msm8974: Add blsp2_bam dma node
> dts: msm8974: Add dma channels for blsp2_i2c1 node
>
> arch/arm/boot/dts/qcom-msm8974.dtsi | 14 +-
> drivers/i2c/busses/i2c-qup.c | 922 ++++++++++++++++++++++++++++++++++--
> 2 files changed, 887 insertions(+), 49 deletions(-)
>
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
Forum, hosted by The Linux Foundation
WARNING: multiple messages have this Message-ID (diff)
From: architt@codeaurora.org (Archit Taneja)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V6 0/6] i2c: qup: Add support for v2 tags and bam dma
Date: Tue, 12 Jan 2016 15:49:43 +0530 [thread overview]
Message-ID: <5694D33F.7040205@codeaurora.org> (raw)
In-Reply-To: <1452588233-1803-1-git-send-email-sricharan@codeaurora.org>
On 01/12/2016 02:13 PM, Sricharan R wrote:
> i2c: qup: Add support for v2 tags and bam dma
>
> QUP from version 2.1.1 onwards, supports a new format of i2c command tags.
> Tag codes instructs the controller to perform a operation like read/write.
> This new tagging version supports and is required for adding bam dma
> capabilities. V2 tags supports transfer of more than 256 bytes in a single i2c
> transaction. Also adding bam dma support facilitates transferring each i2c_msg
> in i2c_msgs without a 'stop' bit in between which is required for some of
> the clients.
>
> Tested this series on apq8074 dragon board eeprom client on i2c bus1
>
> [V6] Added review tags and fixed a checkpatch warning in patch 4/6.
>
> [V5] Addressed few more comments from Ivan T. Ivanov.
> Squashed patch 2 and 3 as no point in having only few lines of
> common code between v1 and v2 tags for increased complexity.
> Couple of non functional review comments fixes in patch 3, 4.
> Added a change in patch 4 to have proper transfer completion in
> a corner case. patch 5, 6 unchanged.
>
> [V4] Added a patch to factor out some common code.
> Removed support for freq > 400KHZ as per comments.
> Addressed comments from Ivan T. Ivanov to keep the code for
> V2 support in a separate path.
> Changed the authorship of V2 tags support patch.
>
> [V3] Added support to coalesce each i2c_msg in i2c_msgs for fifo and
> block mode in Patch 2. Also addressed further code comments.
>
> http://comments.gmane.org/gmane.linux.drivers.i2c/22497
>
> [V2] Addressed comments from Ivan T. Ivanov, Andy Gross [v1] Initial Version
For the series:
Tested-by: Archit Taneja <architt@codeaurora.org>
>
> Sricharan R (6):
> i2c: qup: Change qup_wait_writeready function to use for all timeouts
> i2c: qup: Add V2 tags support
> i2c: qup: Transfer each i2c_msg in i2c_msgs without a stop bit
> i2c: qup: Add bam dma capabilities
> dts: msm8974: Add blsp2_bam dma node
> dts: msm8974: Add dma channels for blsp2_i2c1 node
>
> arch/arm/boot/dts/qcom-msm8974.dtsi | 14 +-
> drivers/i2c/busses/i2c-qup.c | 922 ++++++++++++++++++++++++++++++++++--
> 2 files changed, 887 insertions(+), 49 deletions(-)
>
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora
Forum, hosted by The Linux Foundation
next prev parent reply other threads:[~2016-01-12 10:19 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-01-12 8:43 [PATCH V6 0/6] i2c: qup: Add support for v2 tags and bam dma Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 1/6] i2c: qup: Change qup_wait_writeready function to use for all timeouts Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 2/6] i2c: qup: Add V2 tags support Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 3/6] i2c: qup: Transfer each i2c_msg in i2c_msgs without a stop bit Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 4/6] i2c: qup: Add bam dma capabilities Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 5/6] dts: msm8974: Add blsp2_bam dma node Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 8:43 ` [PATCH V6 6/6] dts: msm8974: Add dma channels for blsp2_i2c1 node Sricharan R
2016-01-12 8:43 ` Sricharan R
2016-01-12 10:19 ` Archit Taneja [this message]
2016-01-12 10:19 ` [PATCH V6 0/6] i2c: qup: Add support for v2 tags and bam dma Archit Taneja
2016-01-19 7:48 ` ntelkar
2016-01-19 7:48 ` ntelkar at codeaurora.org
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=5694D33F.7040205@codeaurora.org \
--to=architt@codeaurora.org \
--cc=agross@codeaurora.org \
--cc=andy.gross@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=dmaengine@vger.kernel.org \
--cc=galak@codeaurora.org \
--cc=iivanov@mm-sol.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=ntelkar@codeaurora.org \
--cc=srichara@codeaurora.org \
--cc=sricharan@codeaurora.org \
--cc=wsa@the-dreams.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.