mbox series

[v3,0/3] Add Richtek RT5120 PMIC support

Message ID 1657780937-20891-1-git-send-email-u0084500@gmail.com (mailing list archive)
Headers show
Series Add Richtek RT5120 PMIC support | expand

Message

ChiYuan Huang July 14, 2022, 6:42 a.m. UTC
From: ChiYuan Huang <cy_huang@richtek.com>

This patch series is to add Richtek RT5120 PMIC support.
In RT5120, it integrates four channels of buck converter, one channel of LDO,
and one external enable channel to control the external power source.

rt5120-regulator can be referred in the below link
Link: https://lore.kernel.org/lkml/165599062767.127406.12571486984170083213.b4-ty@kernel.org/

Since v3:
- Use a 'dev' variable and dev_err_probe to decrease the LOC in mfd.
- Simplify the power key irq handler key report
- Since press and release irq not needed to keep in private data, change 'press',
  'release' irq as local variable only.
- Fix Kconfig typo for pwrkey.

Since v2:
- Add 'unevaluatedProperties: false' for regulator buck1~4/ldo/exten.
- Fix indention and remove status for powerkey.
- Refine powerkey description for more HW details.
- For the regulator property parsing, use of_property_read_u32 instead.
- Not to overwrite regulator constraint.
- Refine regulator desc coding.

ChiYuan Huang (3):
  dt-binding: mfd: Add Richtek RT5120 PMIC support
  mfd: rt5120: Add Richtek PMIC support
  input: misc: rt5120: Add power key support

 .../devicetree/bindings/mfd/richtek,rt5120.yaml    | 178 +++++++++++++++++++++
 drivers/input/misc/Kconfig                         |   9 ++
 drivers/input/misc/Makefile                        |   1 +
 drivers/input/misc/rt5120-pwrkey.c                 | 105 ++++++++++++
 drivers/mfd/Kconfig                                |  12 ++
 drivers/mfd/Makefile                               |   1 +
 drivers/mfd/rt5120.c                               | 121 ++++++++++++++
 7 files changed, 427 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/richtek,rt5120.yaml
 create mode 100644 drivers/input/misc/rt5120-pwrkey.c
 create mode 100644 drivers/mfd/rt5120.c

Comments

ChiYuan Huang July 26, 2022, 3:45 a.m. UTC | #1
cy_huang <u0084500@gmail.com> 於 2022年7月14日 週四 下午2:42寫道:
>
> From: ChiYuan Huang <cy_huang@richtek.com>
>
> This patch series is to add Richtek RT5120 PMIC support.
> In RT5120, it integrates four channels of buck converter, one channel of LDO,
> and one external enable channel to control the external power source.
ping ......
>
> rt5120-regulator can be referred in the below link
> Link: https://lore.kernel.org/lkml/165599062767.127406.12571486984170083213.b4-ty@kernel.org/
>
> Since v3:
> - Use a 'dev' variable and dev_err_probe to decrease the LOC in mfd.
> - Simplify the power key irq handler key report
> - Since press and release irq not needed to keep in private data, change 'press',
>   'release' irq as local variable only.
> - Fix Kconfig typo for pwrkey.
>
> Since v2:
> - Add 'unevaluatedProperties: false' for regulator buck1~4/ldo/exten.
> - Fix indention and remove status for powerkey.
> - Refine powerkey description for more HW details.
> - For the regulator property parsing, use of_property_read_u32 instead.
> - Not to overwrite regulator constraint.
> - Refine regulator desc coding.
>
> ChiYuan Huang (3):
>   dt-binding: mfd: Add Richtek RT5120 PMIC support
>   mfd: rt5120: Add Richtek PMIC support
>   input: misc: rt5120: Add power key support
>
>  .../devicetree/bindings/mfd/richtek,rt5120.yaml    | 178 +++++++++++++++++++++
>  drivers/input/misc/Kconfig                         |   9 ++
>  drivers/input/misc/Makefile                        |   1 +
>  drivers/input/misc/rt5120-pwrkey.c                 | 105 ++++++++++++
>  drivers/mfd/Kconfig                                |  12 ++
>  drivers/mfd/Makefile                               |   1 +
>  drivers/mfd/rt5120.c                               | 121 ++++++++++++++
>  7 files changed, 427 insertions(+)
>  create mode 100644 Documentation/devicetree/bindings/mfd/richtek,rt5120.yaml
>  create mode 100644 drivers/input/misc/rt5120-pwrkey.c
>  create mode 100644 drivers/mfd/rt5120.c
>
> --
> 2.7.4
>
Krzysztof Kozlowski July 26, 2022, 9:59 a.m. UTC | #2
On 26/07/2022 05:45, ChiYuan Huang wrote:
> cy_huang <u0084500@gmail.com> 於 2022年7月14日 週四 下午2:42寫道:
>>
>> From: ChiYuan Huang <cy_huang@richtek.com>
>>
>> This patch series is to add Richtek RT5120 PMIC support.
>> In RT5120, it integrates four channels of buck converter, one channel of LDO,
>> and one external enable channel to control the external power source.
> ping ......

Whom are you pinging? Everyone in To list?

Best regards,
Krzysztof
ChiYuan Huang July 26, 2022, 10:03 a.m. UTC | #3
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> 於 2022年7月26日 週二 下午5:59寫道:
>
> On 26/07/2022 05:45, ChiYuan Huang wrote:
> > cy_huang <u0084500@gmail.com> 於 2022年7月14日 週四 下午2:42寫道:
> >>
> >> From: ChiYuan Huang <cy_huang@richtek.com>
> >>
> >> This patch series is to add Richtek RT5120 PMIC support.
> >> In RT5120, it integrates four channels of buck converter, one channel of LDO,
> >> and one external enable channel to control the external power source.
> > ping ......
>
> Whom are you pinging? Everyone in To list?
>
Sorry, forget to specify the part.
I'm pining the 'mfd' patch and 'power key' patch.

> Best regards,
> Krzysztof
Lee Jones Aug. 1, 2022, 2:42 p.m. UTC | #4
On Tue, 26 Jul 2022, ChiYuan Huang wrote:

> Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> 於 2022年7月26日 週二 下午5:59寫道:
> >
> > On 26/07/2022 05:45, ChiYuan Huang wrote:
> > > cy_huang <u0084500@gmail.com> 於 2022年7月14日 週四 下午2:42寫道:
> > >>
> > >> From: ChiYuan Huang <cy_huang@richtek.com>
> > >>
> > >> This patch series is to add Richtek RT5120 PMIC support.
> > >> In RT5120, it integrates four channels of buck converter, one channel of LDO,
> > >> and one external enable channel to control the external power source.
> > > ping ......
> >
> > Whom are you pinging? Everyone in To list?
> >
> Sorry, forget to specify the part.
> I'm pining the 'mfd' patch and 'power key' patch.

Don't ping any{thing,one}!  If you think your patch has fallen through
the gaps (it hasn't), then submit a [RESEND].

Your patch was submitted at the very end of the development cycle,
which means that it is low priority and is unlikely to receive
attention until after -rc1 is out.
ChiYuan Huang Aug. 1, 2022, 3:20 p.m. UTC | #5
Lee Jones <lee.jones@linaro.org> 於 2022年8月1日 週一 晚上10:42寫道:
>
> On Tue, 26 Jul 2022, ChiYuan Huang wrote:
>
> > Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> 於 2022年7月26日 週二 下午5:59寫道:
> > >
> > > On 26/07/2022 05:45, ChiYuan Huang wrote:
> > > > cy_huang <u0084500@gmail.com> 於 2022年7月14日 週四 下午2:42寫道:
> > > >>
> > > >> From: ChiYuan Huang <cy_huang@richtek.com>
> > > >>
> > > >> This patch series is to add Richtek RT5120 PMIC support.
> > > >> In RT5120, it integrates four channels of buck converter, one channel of LDO,
> > > >> and one external enable channel to control the external power source.
> > > > ping ......
> > >
> > > Whom are you pinging? Everyone in To list?
> > >
> > Sorry, forget to specify the part.
> > I'm pining the 'mfd' patch and 'power key' patch.
>
> Don't ping any{thing,one}!  If you think your patch has fallen through
> the gaps (it hasn't), then submit a [RESEND].
>
> Your patch was submitted at the very end of the development cycle,
> which means that it is low priority and is unlikely to receive
> attention until after -rc1 is out.
>
Got it.
Because I'm not very clear about the linux development cycle.
Now I know. I'll just keep waiting for the reviewing.

Very apologized to ping it.
> --
> Lee Jones [李琼斯]
> Principal Technical Lead - Developer Services
> Linaro.org │ Open source software for Arm SoCs
> Follow Linaro: Facebook | Twitter | Blog