Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1690051 > unrolled thread
| Started by | Kiran Gunda <kgunda@codeaurora.org> |
|---|---|
| First post | 2017-07-18 12:30 +0200 |
| Last post | 2017-07-20 09:00 +0200 |
| Articles | 3 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH V4 0/4]: spmi: pmic-arb: support for V5 HW and bug fixes Kiran Gunda <kgunda@codeaurora.org> - 2017-07-18 12:30 +0200
Re: [PATCH V4 0/4]: spmi: pmic-arb: support for V5 HW and bug fixes Stephen Boyd <sboyd@codeaurora.org> - 2017-07-19 00:20 +0200
Re: [PATCH V4 0/4]: spmi: pmic-arb: support for V5 HW and bug fixes kgunda@codeaurora.org - 2017-07-20 09:00 +0200
| From | Kiran Gunda <kgunda@codeaurora.org> |
|---|---|
| Date | 2017-07-18 12:30 +0200 |
| Subject | [PATCH V4 0/4]: spmi: pmic-arb: support for V5 HW and bug fixes |
| Message-ID | <u4xVo-1aY-5@gated-at.bofh.it> |
v4:
* spmi: pmic-arb: add support for HW version 5
Clean-up as per Stephen's comments
v3:
* spmi: pmic-arb: add support for HW version 5
Modified #define INVALID (-1) to
#define INVALID_EE 0xFF.
v2:
* spmi: pmic-arb: return __iomem pointer instead of offset
Added Stephen's reviewed-by tag.
* spmi: pmic-arb: fix a possible null pointer dereference
Added Stephen's reviewed-by tag.
* spmi: pmic-arb: add support for HW version 5
Modified the pmic_arb_offset_v5 function to return the offset instead
of passed by a pointer.
* spmi: pmic-arb: Remove checking opc value not less than 0
Added Stephen's reviewed-by tag.
Added my sign-off tag.
v1:
This patch series add the support for pmic arbiter hardware v5 along with
the few bug fixes and code cleanup.
This patch series is dependent on the below patches and can be merged
cleanly only after picking the below patches in to the tree.
https://patchwork.kernel.org/patch/9810723/
https://patchwork.kernel.org/patch/9822597/
https://patchwork.kernel.org/patch/9822601/
https://patchwork.kernel.org/patch/9822607/
https://patchwork.kernel.org/patch/9822609/
https://patchwork.kernel.org/patch/9822613/
David Collins (1):
spmi: pmic-arb: add support for HW version 5
Fenglin Wu (1):
spmi: pmic-arb: Remove checking opc value not less than 0
Kiran Gunda (2):
spmi: pmic-arb: return __iomem pointer instead of offset
spmi: pmic-arb: fix a possible null pointer dereference
drivers/spmi/spmi-pmic-arb.c | 331 ++++++++++++++++++++++++++++++++++---------
1 file changed, 266 insertions(+), 65 deletions(-)
--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project
[toc] | [next] | [standalone]
| From | Stephen Boyd <sboyd@codeaurora.org> |
|---|---|
| Date | 2017-07-19 00:20 +0200 |
| Message-ID | <u4J0u-8bs-11@gated-at.bofh.it> |
| In reply to | #1690051 |
On 07/18, Kiran Gunda wrote: > v4: > * spmi: pmic-arb: add support for HW version 5 > Clean-up as per Stephen's comments > > v3: > * spmi: pmic-arb: add support for HW version 5 > Modified #define INVALID (-1) to > #define INVALID_EE 0xFF. > > v2: > * spmi: pmic-arb: return __iomem pointer instead of offset > Added Stephen's reviewed-by tag. > > * spmi: pmic-arb: fix a possible null pointer dereference > Added Stephen's reviewed-by tag. > > * spmi: pmic-arb: add support for HW version 5 > Modified the pmic_arb_offset_v5 function to return the offset instead > of passed by a pointer. > > * spmi: pmic-arb: Remove checking opc value not less than 0 > Added Stephen's reviewed-by tag. > Added my sign-off tag. > > v1: > > This patch series add the support for pmic arbiter hardware v5 along with > the few bug fixes and code cleanup. > > This patch series is dependent on the below patches and can be merged > cleanly only after picking the below patches in to the tree. Can you combine the two series? It's really confusing why there are two patch series from you for the same driver. Presumably one of the series needs to be applied before the other, so putting them into one series makes that clear what the order is. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project
[toc] | [prev] | [next] | [standalone]
| From | kgunda@codeaurora.org |
|---|---|
| Date | 2017-07-20 09:00 +0200 |
| Message-ID | <u5dBf-3AP-7@gated-at.bofh.it> |
| In reply to | #1690762 |
On 2017-07-19 03:47, Stephen Boyd wrote: > On 07/18, Kiran Gunda wrote: >> v4: >> * spmi: pmic-arb: add support for HW version 5 >> Clean-up as per Stephen's comments >> >> v3: >> * spmi: pmic-arb: add support for HW version 5 >> Modified #define INVALID (-1) to >> #define INVALID_EE 0xFF. >> >> v2: >> * spmi: pmic-arb: return __iomem pointer instead of offset >> Added Stephen's reviewed-by tag. >> >> * spmi: pmic-arb: fix a possible null pointer dereference >> Added Stephen's reviewed-by tag. >> >> * spmi: pmic-arb: add support for HW version 5 >> Modified the pmic_arb_offset_v5 function to return the offset >> instead >> of passed by a pointer. >> >> * spmi: pmic-arb: Remove checking opc value not less than 0 >> Added Stephen's reviewed-by tag. >> Added my sign-off tag. >> >> v1: >> >> This patch series add the support for pmic arbiter hardware v5 along >> with >> the few bug fixes and code cleanup. >> >> This patch series is dependent on the below patches and can be merged >> cleanly only after picking the below patches in to the tree. > > Can you combine the two series? It's really confusing why there > are two patch series from you for the same driver. Presumably one > of the series needs to be applied before the other, so putting > them into one series makes that clear what the order is. I had two series just to give the information about the fix-up patches for the previously merged patches and other new patches. Anyways, having a single series will be easy and clear to everyone. Will combine them.
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web