mbox series

[0/5] starfive reboot cleanup & fix

Message ID cover.1706705649.git.namcao@linutronix.de
Headers show
Series starfive reboot cleanup & fix | expand

Message

Nam Cao Jan. 31, 2024, 1:09 p.m. UTC
This series:
  - add gracefully error handling when reboot devices are not present
  - correct a device tree compatibility string
  - some minor cleanup

This series complements my other series:
http://lists.infradead.org/pipermail/opensbi/2024-January/006300.html

Nam Cao (5):
  platform: starfive: remove redundant compatibility check in pmic_ops
  platform: starfive: rename "stf,axp15060-regulator" ->
    "x-powers,axp15060"
  platform: starfive: drop "compatible" from struct pmic
  platform: starfive: return error if needed devices are not present
  platform: starfive: call starfive_jh7110_inst_init() in
    pm_reset_init()

 platform/generic/starfive/jh7110.c | 49 ++++++++++++++++--------------
 1 file changed, 27 insertions(+), 22 deletions(-)

Comments

Anup Patel Feb. 1, 2024, 3:52 a.m. UTC | #1
On Wed, Jan 31, 2024 at 6:39 PM Nam Cao <namcao@linutronix.de> wrote:
>
> This series:
>   - add gracefully error handling when reboot devices are not present
>   - correct a device tree compatibility string
>   - some minor cleanup
>
> This series complements my other series:
> http://lists.infradead.org/pipermail/opensbi/2024-January/006300.html
>
> Nam Cao (5):
>   platform: starfive: remove redundant compatibility check in pmic_ops
>   platform: starfive: rename "stf,axp15060-regulator" ->
>     "x-powers,axp15060"
>   platform: starfive: drop "compatible" from struct pmic
>   platform: starfive: return error if needed devices are not present
>   platform: starfive: call starfive_jh7110_inst_init() in
>     pm_reset_init()

Can someone from StarFive review and test this series ?

Regards,
Anup

>
>  platform/generic/starfive/jh7110.c | 49 ++++++++++++++++--------------
>  1 file changed, 27 insertions(+), 22 deletions(-)
>
> --
> 2.39.2
>
>
> --
> opensbi mailing list
> opensbi@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/opensbi
Minda Chen Feb. 1, 2024, 5:38 a.m. UTC | #2
> On Wed, Jan 31, 2024 at 6:39 PM Nam Cao <namcao@linutronix.de> wrote:
> >
> > This series:
> >   - add gracefully error handling when reboot devices are not present
> >   - correct a device tree compatibility string
> >   - some minor cleanup
> >
> > This series complements my other series:
> > http://lists.infradead.org/pipermail/opensbi/2024-January/006300.html
> >
> > Nam Cao (5):
> >   platform: starfive: remove redundant compatibility check in pmic_ops
> >   platform: starfive: rename "stf,axp15060-regulator" ->
> >     "x-powers,axp15060"
> >   platform: starfive: drop "compatible" from struct pmic
> >   platform: starfive: return error if needed devices are not present
> >   platform: starfive: call starfive_jh7110_inst_init() in
> >     pm_reset_init()
> 
> Can someone from StarFive review and test this series ?
> 
> Regards,
> Anup
> 
Let me review these. But I don't receive this series patch.

Minda
> >
> >  platform/generic/starfive/jh7110.c | 49 ++++++++++++++++--------------
> >  1 file changed, 27 insertions(+), 22 deletions(-)
> >
> > --
> > 2.39.2
> >
> >
> > --
> > opensbi mailing list
> > opensbi@lists.infradead.org
> > http://lists.infradead.org/mailman/listinfo/opensbi
Nam Cao Feb. 1, 2024, 7:01 a.m. UTC | #3
On Thu, 1 Feb 2024 05:38:16 +0000 Minda Chen <minda.chen@starfivetech.com> wrote:
> > On Wed, Jan 31, 2024 at 6:39 PM Nam Cao <namcao@linutronix.de> wrote:  
> > >
> > > This series:
> > >   - add gracefully error handling when reboot devices are not present
> > >   - correct a device tree compatibility string
> > >   - some minor cleanup
> > >
> > > This series complements my other series:
> > > http://lists.infradead.org/pipermail/opensbi/2024-January/006300.html
> > >
> > > Nam Cao (5):
> > >   platform: starfive: remove redundant compatibility check in pmic_ops
> > >   platform: starfive: rename "stf,axp15060-regulator" ->
> > >     "x-powers,axp15060"
> > >   platform: starfive: drop "compatible" from struct pmic
> > >   platform: starfive: return error if needed devices are not present
> > >   platform: starfive: call starfive_jh7110_inst_init() in
> > >     pm_reset_init()  
> > 
> > Can someone from StarFive review and test this series ?
> > 
> > Regards,
> > Anup
> >   
> Let me review these. But I don't receive this series patch.

I only sent it to the list. Can you grab the series from there?

Or I can also resend the series and include you as a recipient, if that's
more convenient.

Best regards,
Nam
Minda Chen Feb. 2, 2024, 10:47 a.m. UTC | #4
> 
> On Thu, 1 Feb 2024 05:38:16 +0000 Minda Chen
> <minda.chen@starfivetech.com> wrote:
> > > On Wed, Jan 31, 2024 at 6:39 PM Nam Cao <namcao@linutronix.de> wrote:
> > > >
> > > > This series:
> > > >   - add gracefully error handling when reboot devices are not present
> > > >   - correct a device tree compatibility string
> > > >   - some minor cleanup
> > > >
> > > > This series complements my other series:
> > > > http://lists.infradead.org/pipermail/opensbi/2024-January/006300.h
> > > > tml
> > > >
> > > > Nam Cao (5):
> > > >   platform: starfive: remove redundant compatibility check in pmic_ops
> > > >   platform: starfive: rename "stf,axp15060-regulator" ->
> > > >     "x-powers,axp15060"
> > > >   platform: starfive: drop "compatible" from struct pmic
> > > >   platform: starfive: return error if needed devices are not present
> > > >   platform: starfive: call starfive_jh7110_inst_init() in
> > > >     pm_reset_init()
> > >
> > > Can someone from StarFive review and test this series ?
> > >
> > > Regards,
> > > Anup
> > >
> > Let me review these. But I don't receive this series patch.
> 
> I only sent it to the list. Can you grab the series from there?
> 
> Or I can also resend the series and include you as a recipient, if that's more
> convenient.
> 
> Best regards,
> Nam

Resend the patchset is better. Thanks
Minda