| Submitter | Datta, Shubhrajyoti |
|---|---|
| Date | Aug. 16, 2012, 2:03 p.m. |
| Message ID | <1345125835-15524-1-git-send-email-shubhrajyoti@ti.com> |
| Download | mbox |
| Permalink | /patch/177996/ |
| State | New |
| Headers | show |
Pull-request
git://gitorious.org/linus-tree/linus-tree.git for_3.6/i2c/big_cleanupComments
On Thu, Aug 16, 2012 at 07:33:31PM +0530, Shubhrajyoti D wrote: > Changes since v1: > - removed tabification on patch 6/17 > - removed dev_err() which was introduced on patch 09/17 > Changes since v2: > - do not set full fifo depth in the RDR interrupt. > - some changelog updates. > - rebase to the Wolfram's tree. > Changes since v3: > - Remove a redundant read of status register > - Read the dev->buf_len variable instead of the register > as the information of the remaining bytes is there. > Changes since v4: > - Ack the arbitration lost. > - Rebase to the i2c-embedded/for-next branch. > Changes since v5: > - Rebase to latest mainline > - Added some more cleanup patches so as have a consolidated series. > Changes since v6: > - Fix comments on setting the pdev to NULL. > - Trivial changelog update > > Tested on omap4sdp and 3430sdp. > > The following changes since commit 3bf671af14d591ede9251acb0085e8017f3705e7: > > Merge branch 'fixes-for-3.6' of git://git.kernel.org/pub/scm/linux/kernel/git/cooloney/linux-leds (2012-08-13 09:59:04 +0300) > > are available in the git repository at: > > git://gitorious.org/linus-tree/linus-tree.git for_3.6/i2c/big_cleanup A GIT tree, thanks for that! Since the series is quite huge, I'd appreciate it if you could wait for reviews a little bit (a few days) and collect the issues raised there before resending. I know "release early & often", yet the traffic causes some overhead here :) Thanks, Wolfram
Changes since v1: - removed tabification on patch 6/17 - removed dev_err() which was introduced on patch 09/17 Changes since v2: - do not set full fifo depth in the RDR interrupt. - some changelog updates. - rebase to the Wolfram's tree. Changes since v3: - Remove a redundant read of status register - Read the dev->buf_len variable instead of the register as the information of the remaining bytes is there. Changes since v4: - Ack the arbitration lost. - Rebase to the i2c-embedded/for-next branch. Changes since v5: - Rebase to latest mainline - Added some more cleanup patches so as have a consolidated series. Changes since v6: - Fix comments on setting the pdev to NULL. - Trivial changelog update Tested on omap4sdp and 3430sdp. The following changes since commit 3bf671af14d591ede9251acb0085e8017f3705e7: Merge branch 'fixes-for-3.6' of git://git.kernel.org/pub/scm/linux/kernel/git/cooloney/linux-leds (2012-08-13 09:59:04 +0300) are available in the git repository at: git://gitorious.org/linus-tree/linus-tree.git for_3.6/i2c/big_cleanup Felipe Balbi (22): i2c: omap: switch to devm_* API i2c: omap: simplify num_bytes handling i2c: omap: decrease indentation level on data handling i2c: omap: add blank lines i2c: omap: simplify omap_i2c_ack_stat() i2c: omap: split out [XR]DR and [XR]RDY i2c: omap: improve i462 errata handling i2c: omap: re-factor receive/transmit data loop i2c: omap: switch over to do {} while loop i2c: omap: ack IRQ in parts i2c: omap: switch to platform_get_irq() i2c: omap: bus: add a receiver flag i2c: omap: simplify errata check i2c: omap: always return IRQ_HANDLED i2c: omap: simplify IRQ exit path i2c: omap: resize fifos before each message i2c: omap: get rid of the "complete" label i2c: omap: always return IRQ_HANDLED i2c: omap: switch to threaded IRQ support i2c: omap: remove unnecessary pm_runtime_suspended check i2c: omap: switch over to autosuspend API i2c: omap: sanitize exit path Shubhrajyoti D (2): i2c: omap: remove redundant status read i2c: omap: Prevent NULL pointer dereference in remove drivers/i2c/busses/i2c-omap.c | 446 +++++++++++++++++++++++++---------------- 1 files changed, 271 insertions(+), 175 deletions(-)