i2c: pxa: Fixes, cleanup and support for pxa910 family
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v4 Latest11 patches 10 repliesIn review
- v311 patches 12 repliesSuperseded
- v2 posting 3/30 of 12 patches 0 repliesSuperseded
Earlier versions · 3
| Version | Posted | Patches | Replies | Status |
|---|---|---|---|---|
| v2 posting 2/3 | 0 of 12 patches | 0 replies | Superseded | |
| v2 posting 1/3 | 12 patches | 0 replies | Superseded | |
| v1 | 0 of 12 patches | 0 replies | Superseded |
Changes in v4 from the cover letter
=======
Link to V3: http://www.spinics.net/lists/devicetree/msg85904.html
- [PATCH 06/11] Removed unnecessary dev_err on devm_kzalloc() check
- [PATCH 06/11] Removed return check on platform_get_resource(), as devm_ioremap_resource() does it for us.
Also, brought up the devm_ioremap_resource() function call in the execution sequence, as no point in delaying it if we do not have resource. It make sense, after this change.
- [PATCH 04/11] Typecast changed to 'enum pxa_i2c_types'
Patches in v4 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/11 | i2c: pxa: keep i2c irq ON in suspenddrivers/ | +3 | 0 replies | |
| 2/11 | i2c: pxa: No need to set slave addr for i2c master mode resetdrivers/ | +1 | 0 replies | |
| 3/11 | i2c: pxa: Return I2C_RETRY when timeout in pio modedrivers/ | +3 | 0 replies | |
| 4/11 | i2c: pxa: Fix compile warning in 64bit modedrivers/ | +3 | 0 replies | |
| 5/11 | i2c: pxa: Update debug function to dump more info on errordrivers/ | +15 | 0 replies | |
| 6/11 | i2c:pxa: Use devm_ variants in probe functiondrivers/ | +30 | 3 replies | |
| 7/11 | i2c: pxa: enable/disable i2c module across msg xferdrivers/ | +41 | 0 replies | |
| 8/11 | Documentation: binding: add new property 'disable_after_xfer' to i2c-pxaDocumentation/ | +5 | 0 replies | |
| 9/11 | i2c: pxa: Add support for pxa910/988 & new configuration featuresdrivers/ | +41 | 0 replies | |
| 10/11 | i2c: pxa: Add ILCR (tLow & tHigh) configuration supportdrivers/ | +60 | 0 replies | |
| 11/11 | Documentation: binding: add sclk adjustment properties to i2c-pxaDocumentation/ | +13 | 0 replies |
Reviews trailers per patch
4 of 11 patches have a Reviewed-by or Acked-by.
| Patch | Robert Jarzmik |
|---|---|
| 1/11 | |
| 2/11 | A-b |
| 3/11 | A-b |
| 4/11 | A-b |
| 5/11 | |
| 6/11 | A-b |
| 7/11 | |
| 8/11 | |
| 9/11 | |
| 10/11 | |
| 11/11 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH-v4 00/11] i2c: pxa: Fixes, cleanup and support for pxa910 family
This patch series fixes bugs/warnings, cleans up the code and adds support for PXA910 family of devices to PXA I2C bus driver.
There has been one attempt made sometime back in 2012 to upstream some of the patches from below list, but did not get follow up later. I have consolidated all the patches, cleaned them up, splited into logical changes, added new patches and submitting it now.
I tried to maintain authorship & Signoff except where I did some significant changes to the code/logic.
Link to previous post:
http://permalink.gmane.org/gmane.linux.drivers.i2c/13557
Testing:
- Basic testing on PMIC device on I2C-0 interface
- Boot tested on platform based on PXA1928
- Probe is successfully passing
- Read few registers of PMIC (RTC, ID, etc...) during boot
V3 => V4
=======
Link to V3: http://www.spinics.net/lists/devicetree/msg85904.html
- [PATCH 06/11] Removed unnecessary dev_err on devm_kzalloc() check
- [PATCH 06/11] Removed return check on platform_get_resource(), as devm_ioremap_resource() does it for us.
Also, brought up the devm_ioremap_resource() function call in the execution sequence, as no point in delaying it if we do not have resource. It make sense, after this change.
- [PATCH 04/11] Typecast changed to 'enum pxa_i2c_types'
Also updated the subject line "Removed ==> Fix"
V2 => V3
=======
Link to V2: http://www.spinics.net/lists/linux-i2c/msg20059.html
- Removed PATCH [4/12] related to reset of I2C module. Suggested by "Robert Jarzmik"
- Updated commit description for,
PATCH [11/12]: Mentioned reasoning about moment of clk_get code. PATCH [12/12]: for DT property node.
- Added Acked by "Robert Jarzmik" to patched which he acked.
V1 => V2:
========
Link to V1 - http://lists.infradead.org/pipermail/linux-arm-kernel/2015-May/347012.html
- Fixed all comments from "Robert Jarzmik" and "Wolfram Sang"
- Dropped Patch
05/12: using core bus reset implementation - under work.
Will submit shortly.
08/12: NAKed and dropped
- Separated DT binding patch from driver changes, for easy merge
Leilei Shang (1):
i2c: pxa: keep i2c irq ON in suspend
Shouming Wang (1):
i2c: pxa: Return I2C_RETRY when timeout in pio mode
Vaibhav Hiremath (7):
i2c: pxa: No need to set slave addr for i2c master mode reset
i2c: pxa: Update debug function to dump more info on error i2c:pxa: Use devm_ variants in probe function
Documentation: binding: add new property 'disable_after_xfer' to
i2c-pxa
i2c: pxa: Add support for pxa910/988 & new configuration features
i2c: pxa: Add ILCR (tLow & tHigh) configuration support
Documentation: binding: add sclk adjustment properties to i2c-pxa
Yi Zhang (1):
i2c: pxa: enable/disable i2c module across msg xfer
Yipeng Yao (1):
i2c: pxa: Fix compile warning in 64bit mode
Documentation/devicetree/bindings/i2c/i2c-pxa.txt | 18 ++ drivers/i2c/busses/i2c-pxa.c | 261 ++++++++++++++++------ 2 files changed, 211 insertions(+), 68 deletions(-)
Signature · 1 line
1.9.1