[PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7

Vinod Koul posted 2 patches 4 years, 4 months ago
There is a newer version of this series
.../bindings/spmi/qcom,spmi-pmic-arb.yaml     |  11 +
drivers/spmi/spmi-pmic-arb.c                  | 233 ++++++++++++++++--
2 files changed, 225 insertions(+), 19 deletions(-)
[PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Vinod Koul 4 years, 4 months ago
Hello,

The is version 3 of support for PMIC v7. I have added a new property
qcom,bus-id for supporting v7 and then add driver changes for v7

This depends on yaml conversion patch:
https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/

Changes since v2:
 - Drop yaml conversion patch
 - Fix author for spmi patch
Changes since v1:
 - Add yaml conversion patch and new binding
 - fix driver bug report by Jonathan

David Collins (1):
  spmi: pmic-arb: Add support for PMIC v7

Vinod Koul (1):
  dt-bindings: spmi: Add qcom,bus-id

 .../bindings/spmi/qcom,spmi-pmic-arb.yaml     |  11 +
 drivers/spmi/spmi-pmic-arb.c                  | 233 ++++++++++++++++--
 2 files changed, 225 insertions(+), 19 deletions(-)

-- 
2.31.1

Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Vinod Koul 4 years, 4 months ago
On 01-02-22, 19:11, Vinod Koul wrote:
> Hello,
> 
> The is version 3 of support for PMIC v7. I have added a new property
> qcom,bus-id for supporting v7 and then add driver changes for v7
> 
> This depends on yaml conversion patch:
> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/

Any feedback on this...

> 
> Changes since v2:
>  - Drop yaml conversion patch
>  - Fix author for spmi patch
> Changes since v1:
>  - Add yaml conversion patch and new binding
>  - fix driver bug report by Jonathan
> 
> David Collins (1):
>   spmi: pmic-arb: Add support for PMIC v7
> 
> Vinod Koul (1):
>   dt-bindings: spmi: Add qcom,bus-id
> 
>  .../bindings/spmi/qcom,spmi-pmic-arb.yaml     |  11 +
>  drivers/spmi/spmi-pmic-arb.c                  | 233 ++++++++++++++++--
>  2 files changed, 225 insertions(+), 19 deletions(-)
> 
> -- 
> 2.31.1

-- 
~Vinod
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Dmitry Baryshkov 4 years, 1 month ago
On 22/02/2022 19:53, Vinod Koul wrote:
> On 01-02-22, 19:11, Vinod Koul wrote:
>> Hello,
>>
>> The is version 3 of support for PMIC v7. I have added a new property
>> qcom,bus-id for supporting v7 and then add driver changes for v7
>>
>> This depends on yaml conversion patch:
>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/
> 
> Any feedback on this...

Another gracious reminder about these patches. At this moment this is 
one of the important pieces lacking for the full SM8450 support in the 
upstream kernel.

> 
>>
>> Changes since v2:
>>   - Drop yaml conversion patch
>>   - Fix author for spmi patch
>> Changes since v1:
>>   - Add yaml conversion patch and new binding
>>   - fix driver bug report by Jonathan
>>
>> David Collins (1):
>>    spmi: pmic-arb: Add support for PMIC v7
>>
>> Vinod Koul (1):
>>    dt-bindings: spmi: Add qcom,bus-id
>>
>>   .../bindings/spmi/qcom,spmi-pmic-arb.yaml     |  11 +
>>   drivers/spmi/spmi-pmic-arb.c                  | 233 ++++++++++++++++--
>>   2 files changed, 225 insertions(+), 19 deletions(-)
>>
>> -- 
>> 2.31.1
> 


-- 
With best wishes
Dmitry
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Dmitry Baryshkov 4 years ago
On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> On 22/02/2022 19:53, Vinod Koul wrote:
>> On 01-02-22, 19:11, Vinod Koul wrote:
>>> Hello,
>>>
>>> The is version 3 of support for PMIC v7. I have added a new property
>>> qcom,bus-id for supporting v7 and then add driver changes for v7
>>>
>>> This depends on yaml conversion patch:
>>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/ 
>>>
>>
>> Any feedback on this...
> 
> Another gracious reminder about these patches. At this moment this is 
> one of the important pieces lacking for the full SM8450 support in the 
> upstream kernel.

Stephen, yet another ping. This is the blocking point for the further 
SM8450 progress.

>>> Changes since v2:
>>>   - Drop yaml conversion patch
>>>   - Fix author for spmi patch
>>> Changes since v1:
>>>   - Add yaml conversion patch and new binding
>>>   - fix driver bug report by Jonathan
>>>
>>> David Collins (1):
>>>    spmi: pmic-arb: Add support for PMIC v7
>>>
>>> Vinod Koul (1):
>>>    dt-bindings: spmi: Add qcom,bus-id
>>>
>>>   .../bindings/spmi/qcom,spmi-pmic-arb.yaml     |  11 +
>>>   drivers/spmi/spmi-pmic-arb.c                  | 233 ++++++++++++++++--
>>>   2 files changed, 225 insertions(+), 19 deletions(-)
>>>
>>> -- 
>>> 2.31.1
>>
> 
> 


-- 
With best wishes
Dmitry
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Stephen Boyd 3 years, 10 months ago
Quoting Dmitry Baryshkov (2022-06-18 08:29:58)
> On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> > On 22/02/2022 19:53, Vinod Koul wrote:
> >> On 01-02-22, 19:11, Vinod Koul wrote:
> >>> Hello,
> >>>
> >>> The is version 3 of support for PMIC v7. I have added a new property
> >>> qcom,bus-id for supporting v7 and then add driver changes for v7
> >>>
> >>> This depends on yaml conversion patch:
> >>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/ 
> >>>
> >>
> >> Any feedback on this...
> > 
> > Another gracious reminder about these patches. At this moment this is 
> > one of the important pieces lacking for the full SM8450 support in the 
> > upstream kernel.
> 
> Stephen, yet another ping. This is the blocking point for the further 
> SM8450 progress.
> 

Sorry I completely missed this one as it fell off the end of my inbox
into the abyss.
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Vinod Koul 3 years, 10 months ago
On 29-08-22, 15:25, Stephen Boyd wrote:
> Quoting Dmitry Baryshkov (2022-06-18 08:29:58)
> > On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> > > On 22/02/2022 19:53, Vinod Koul wrote:
> > >> On 01-02-22, 19:11, Vinod Koul wrote:
> > >>> Hello,
> > >>>
> > >>> The is version 3 of support for PMIC v7. I have added a new property
> > >>> qcom,bus-id for supporting v7 and then add driver changes for v7
> > >>>
> > >>> This depends on yaml conversion patch:
> > >>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/ 
> > >>>
> > >>
> > >> Any feedback on this...
> > > 
> > > Another gracious reminder about these patches. At this moment this is 
> > > one of the important pieces lacking for the full SM8450 support in the 
> > > upstream kernel.
> > 
> > Stephen, yet another ping. This is the blocking point for the further 
> > SM8450 progress.
> > 
> 
> Sorry I completely missed this one as it fell off the end of my inbox
> into the abyss.

Thanks for the reply. Is this applied now or you have some feedback for
me to address..

Thanks
-- 
~Vinod
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Stephen Boyd 3 years, 10 months ago
Quoting Vinod Koul (2022-08-29 21:36:00)
> On 29-08-22, 15:25, Stephen Boyd wrote:
> > Quoting Dmitry Baryshkov (2022-06-18 08:29:58)
> > > On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> > > > On 22/02/2022 19:53, Vinod Koul wrote:
> > > >> On 01-02-22, 19:11, Vinod Koul wrote:
> > > >>> Hello,
> > > >>>
> > > >>> The is version 3 of support for PMIC v7. I have added a new property
> > > >>> qcom,bus-id for supporting v7 and then add driver changes for v7
> > > >>>
> > > >>> This depends on yaml conversion patch:
> > > >>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/ 
> > > >>>
> > > >>
> > > >> Any feedback on this...
> > > > 
> > > > Another gracious reminder about these patches. At this moment this is 
> > > > one of the important pieces lacking for the full SM8450 support in the 
> > > > upstream kernel.
> > > 
> > > Stephen, yet another ping. This is the blocking point for the further 
> > > SM8450 progress.
> > > 
> > 
> > Sorry I completely missed this one as it fell off the end of my inbox
> > into the abyss.
> 
> Thanks for the reply. Is this applied now or you have some feedback for
> me to address..
> 

Does it apply along with the series from qcom[1]? I have to check and
make sure they both work together.

[1] https://lore.kernel.org/r/1651108369-11059-1-git-send-email-quic_fenglinw@quicinc.com
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Vinod Koul 3 years, 9 months ago
On 30-08-22, 14:11, Stephen Boyd wrote:
> Quoting Vinod Koul (2022-08-29 21:36:00)
> > On 29-08-22, 15:25, Stephen Boyd wrote:
> > > Quoting Dmitry Baryshkov (2022-06-18 08:29:58)
> > > > On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> > > > > On 22/02/2022 19:53, Vinod Koul wrote:
> > > > >> On 01-02-22, 19:11, Vinod Koul wrote:
> > > > >>> Hello,
> > > > >>>
> > > > >>> The is version 3 of support for PMIC v7. I have added a new property
> > > > >>> qcom,bus-id for supporting v7 and then add driver changes for v7
> > > > >>>
> > > > >>> This depends on yaml conversion patch:
> > > > >>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/ 
> > > > >>>
> > > > >>
> > > > >> Any feedback on this...
> > > > > 
> > > > > Another gracious reminder about these patches. At this moment this is 
> > > > > one of the important pieces lacking for the full SM8450 support in the 
> > > > > upstream kernel.
> > > > 
> > > > Stephen, yet another ping. This is the blocking point for the further 
> > > > SM8450 progress.
> > > > 
> > > 
> > > Sorry I completely missed this one as it fell off the end of my inbox
> > > into the abyss.
> > 
> > Thanks for the reply. Is this applied now or you have some feedback for
> > me to address..
> > 
> 
> Does it apply along with the series from qcom[1]? I have to check and
> make sure they both work together.

There were conflicts, I have resolved, tested on top of spmi/next again
and posted the v4 now.. Hope this would get in.

-- 
~Vinod
Re: [PATCH v3 0/2] spmi: pmic-arb: Add support for PMIC v7
Posted by Linus Walleij 4 years ago
On Sat, Jun 18, 2022 at 5:30 PM Dmitry Baryshkov
<dmitry.baryshkov@linaro.org> wrote:
> On 01/05/2022 22:41, Dmitry Baryshkov wrote:
> > On 22/02/2022 19:53, Vinod Koul wrote:
> >> On 01-02-22, 19:11, Vinod Koul wrote:
> >>> Hello,
> >>>
> >>> The is version 3 of support for PMIC v7. I have added a new property
> >>> qcom,bus-id for supporting v7 and then add driver changes for v7
> >>>
> >>> This depends on yaml conversion patch:
> >>> https://lore.kernel.org/linux-arm-msm/20211227170151.73116-1-david@ixit.cz/
> >>>
> >>
> >> Any feedback on this...
> >
> > Another gracious reminder about these patches. At this moment this is
> > one of the important pieces lacking for the full SM8450 support in the
> > upstream kernel.
>
> Stephen, yet another ping. This is the blocking point for the further
> SM8450 progress.

Pending since february!

I would rather suspect something is wrong and it somehow goes
into the spam or wrong inbox on Stephen's end.

Sounds like something Bjorn could solve if Stephen don't have time
to manage the SPMI subsystem right now? If it's OK with Stephen
or he simply don't respond we can just queue the stuff in the SoC
tree I assume?

Yours,
Linus Walleij