All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Sricharan" <sricharan@codeaurora.org>
To: wsa@the-dreams.de
Cc: devicetree@vger.kernel.org, architt@codeaurora.org,
	linux-arm-msm@vger.kernel.org, ntelkar@codeaurora.org,
	galak@codeaurora.org, linux-kernel@vger.kernel.org,
	andy.gross@linaro.org, linux-i2c@vger.kernel.org,
	iivanov@mm-sol.com, agross@codeaurora.org,
	dmaengine@vger.kernel.org, linux-arm-kernel@lists.infradead.org
Subject: RE: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
Date: Tue, 19 Jan 2016 15:44:03 +0530	[thread overview]
Message-ID: <000101d152a2$23bb1da0$6b3158e0$@codeaurora.org> (raw)
In-Reply-To: <1453197766-18976-1-git-send-email-sricharan@codeaurora.org>

Hi wolfram,

> -----Original Message-----
> From: linux-arm-kernel [mailto:linux-arm-kernel-
> bounces@lists.infradead.org] On Behalf Of Sricharan R
> Sent: Tuesday, January 19, 2016 3:33 PM
> To: wsa@the-dreams.de
> Cc: devicetree@vger.kernel.org; architt@codeaurora.org; linux-arm-
> msm@vger.kernel.org; ntelkar@codeaurora.org; galak@codeaurora.org;
> linux-kernel@vger.kernel.org; andy.gross@linaro.org; linux-
> i2c@vger.kernel.org; iivanov@mm-sol.com; agross@codeaurora.org;
> dmaengine@vger.kernel.org; sricharan@codeaurora.org; linux-arm-
> kernel@lists.infradead.org
> Subject: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
> 
> 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 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
> 
> [V7] Added tested-by tags and fixed a small comment
> 
> [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
> 
> 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
> 
 Wolfram,  Does the first 4 patches looks good to be picked up  ?

Regards,
 Sricharan

WARNING: multiple messages have this Message-ID (diff)
From: sricharan@codeaurora.org (Sricharan)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
Date: Tue, 19 Jan 2016 15:44:03 +0530	[thread overview]
Message-ID: <000101d152a2$23bb1da0$6b3158e0$@codeaurora.org> (raw)
In-Reply-To: <1453197766-18976-1-git-send-email-sricharan@codeaurora.org>

Hi wolfram,

> -----Original Message-----
> From: linux-arm-kernel [mailto:linux-arm-kernel-
> bounces at lists.infradead.org] On Behalf Of Sricharan R
> Sent: Tuesday, January 19, 2016 3:33 PM
> To: wsa at the-dreams.de
> Cc: devicetree at vger.kernel.org; architt at codeaurora.org; linux-arm-
> msm at vger.kernel.org; ntelkar at codeaurora.org; galak at codeaurora.org;
> linux-kernel at vger.kernel.org; andy.gross at linaro.org; linux-
> i2c at vger.kernel.org; iivanov at mm-sol.com; agross at codeaurora.org;
> dmaengine at vger.kernel.org; sricharan at codeaurora.org; linux-arm-
> kernel at lists.infradead.org
> Subject: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
> 
> 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 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
> 
> [V7] Added tested-by tags and fixed a small comment
> 
> [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
> 
> 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
> 
 Wolfram,  Does the first 4 patches looks good to be picked up  ?

Regards,
 Sricharan

WARNING: multiple messages have this Message-ID (diff)
From: "Sricharan" <sricharan@codeaurora.org>
To: <wsa@the-dreams.de>
Cc: <devicetree@vger.kernel.org>, <architt@codeaurora.org>,
	<linux-arm-msm@vger.kernel.org>, <ntelkar@codeaurora.org>,
	<galak@codeaurora.org>, <linux-kernel@vger.kernel.org>,
	<andy.gross@linaro.org>, <linux-i2c@vger.kernel.org>,
	<iivanov@mm-sol.com>, <agross@codeaurora.org>,
	<dmaengine@vger.kernel.org>,
	<linux-arm-kernel@lists.infradead.org>
Subject: RE: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
Date: Tue, 19 Jan 2016 15:44:03 +0530	[thread overview]
Message-ID: <000101d152a2$23bb1da0$6b3158e0$@codeaurora.org> (raw)
In-Reply-To: <1453197766-18976-1-git-send-email-sricharan@codeaurora.org>

Hi wolfram,

> -----Original Message-----
> From: linux-arm-kernel [mailto:linux-arm-kernel-
> bounces@lists.infradead.org] On Behalf Of Sricharan R
> Sent: Tuesday, January 19, 2016 3:33 PM
> To: wsa@the-dreams.de
> Cc: devicetree@vger.kernel.org; architt@codeaurora.org; linux-arm-
> msm@vger.kernel.org; ntelkar@codeaurora.org; galak@codeaurora.org;
> linux-kernel@vger.kernel.org; andy.gross@linaro.org; linux-
> i2c@vger.kernel.org; iivanov@mm-sol.com; agross@codeaurora.org;
> dmaengine@vger.kernel.org; sricharan@codeaurora.org; linux-arm-
> kernel@lists.infradead.org
> Subject: [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma
> 
> 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 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
> 
> [V7] Added tested-by tags and fixed a small comment
> 
> [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
> 
> 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
> 
 Wolfram,  Does the first 4 patches looks good to be picked up  ?

Regards,
 Sricharan

  parent reply	other threads:[~2016-01-19 10:14 UTC|newest]

Thread overview: 62+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-01-19 10:02 [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma Sricharan R
2016-01-19 10:02 ` Sricharan R
2016-01-19 10:02 ` [PATCH V7 1/6] i2c: qup: Change qup_wait_writeready function to use for all timeouts Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-02-12 18:37   ` Wolfram Sang
2016-02-12 18:37     ` Wolfram Sang
2016-01-19 10:02 ` [PATCH V7 2/6] i2c: qup: Add V2 tags support Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-02-12 18:37   ` Wolfram Sang
2016-02-12 18:37     ` Wolfram Sang
2016-01-19 10:02 ` [PATCH V7 3/6] i2c: qup: Transfer each i2c_msg in i2c_msgs without a stop bit Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-01-24 11:29   ` Wolfram Sang
2016-01-24 11:29     ` Wolfram Sang
2016-01-28  4:57     ` Sricharan
2016-01-28  4:57       ` Sricharan
2016-01-28  4:57       ` Sricharan
2016-02-04 20:09       ` Wolfram Sang
2016-02-04 20:09         ` Wolfram Sang
2016-02-05  8:00         ` Sricharan
2016-02-05  8:00           ` Sricharan
2016-02-05  8:00           ` Sricharan
2016-02-12 18:38   ` Wolfram Sang
2016-02-12 18:38     ` Wolfram Sang
2016-01-19 10:02 ` [PATCH V7 4/6] i2c: qup: Add bam dma capabilities Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-02-12 18:37   ` Wolfram Sang
2016-02-12 18:37     ` Wolfram Sang
2016-02-12 18:42     ` Wolfram Sang
2016-02-12 18:42       ` Wolfram Sang
2016-02-13  6:58     ` Sricharan
2016-02-13  6:58       ` Sricharan
2016-02-13  6:58       ` Sricharan
2016-02-22 12:24     ` Sricharan
2016-02-22 12:24       ` Sricharan
2016-02-22 12:24       ` Sricharan
2016-02-22 12:32       ` Wolfram Sang
2016-02-22 12:32         ` Wolfram Sang
2016-02-22 13:41         ` Sricharan
2016-02-22 13:41           ` Sricharan
2016-02-22 13:41           ` Sricharan
2016-01-19 10:02 ` [PATCH V7 5/6] dts: msm8974: Add blsp2_bam dma node Sricharan R
2016-01-19 10:02   ` Sricharan R
     [not found]   ` <1453197766-18976-6-git-send-email-sricharan-sgV2jX0FEOL9JmXXK+q4OQ@public.gmane.org>
2016-03-25 23:17     ` Bjorn Andersson
2016-03-25 23:17       ` Bjorn Andersson
2016-03-25 23:17       ` Bjorn Andersson
2016-03-26  2:26       ` Andy Gross
2016-03-26  2:26         ` Andy Gross
2016-03-28 12:59         ` Sricharan
2016-03-28 12:59           ` Sricharan
2016-03-28 12:59           ` Sricharan
2016-01-19 10:02 ` [PATCH V7 6/6] dts: msm8974: Add dma channels for blsp2_i2c1 node Sricharan R
2016-01-19 10:02   ` Sricharan R
2016-01-19 10:14 ` Sricharan [this message]
2016-01-19 10:14   ` [PATCH V7 0/6] i2c: qup: Add support for v2 tags and bam dma Sricharan
2016-01-19 10:14   ` Sricharan
2016-01-24 11:33   ` Wolfram Sang
2016-01-24 11:33     ` Wolfram Sang
2016-01-28  5:27     ` Sricharan
2016-01-28  5:27       ` Sricharan
2016-01-28  5:27       ` Sricharan

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='000101d152a2$23bb1da0$6b3158e0$@codeaurora.org' \
    --to=sricharan@codeaurora.org \
    --cc=agross@codeaurora.org \
    --cc=andy.gross@linaro.org \
    --cc=architt@codeaurora.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=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.