lkml.org 
[lkml]   [2015]   [Sep]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] soc: mediatek: Move the initial setting of pmic wrap interrupt before requesting irq.
From
Date
Hi Sascha,

On Mon, 2015-09-07 at 07:39 +0200, Sascha Hauer wrote:
> On Fri, Sep 04, 2015 at 12:05:12PM +0800, Henry Chen wrote:
> > Seperate the initialization of pmic wrap interrupt from pmic wrap init to make sure setting is correct before requesting irq.
>
> It seems the problem this patch solves is the missing watchdog
> initialization when the pwrap is already initialized but the watchdog is
> not, so how about a commit log like:
>
> "The watchdog may not be initialized by the bootloader, even if the rest
> of the pwrap is. Move the watchdog initialization out of pwrap_init()
> to make sure the watchdog is always initialized and not only when the
> pwrap is uninitialized."
>
> Also please add linebreaks to the commit log.
>

Yes, thanks for explanation, I will add this on commit log make it more
clearly to understand.

Henry

> >
> > Signed-off-by: Henry Chen <henryc.chen@mediatek.com>
> > ---
> > drivers/soc/mediatek/mtk-pmic-wrap.c | 14 ++++++++++----
> > 1 file changed, 10 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/soc/mediatek/mtk-pmic-wrap.c b/drivers/soc/mediatek/mtk-pmic-wrap.c
> > index f432291..c91d629 100644
> > --- a/drivers/soc/mediatek/mtk-pmic-wrap.c
> > +++ b/drivers/soc/mediatek/mtk-pmic-wrap.c
> > @@ -725,10 +725,6 @@ static int pwrap_init(struct pmic_wrapper *wrp)
> > pwrap_writel(wrp, 0x1, PWRAP_WACS2_EN);
> > pwrap_writel(wrp, 0x5, PWRAP_STAUPD_PRD);
> > pwrap_writel(wrp, 0xff, PWRAP_STAUPD_GRPEN);
> > - pwrap_writel(wrp, 0xf, PWRAP_WDT_UNIT);
> > - pwrap_writel(wrp, 0xffffffff, PWRAP_WDT_SRC_EN);
> > - pwrap_writel(wrp, 0x1, PWRAP_TIMER_EN);
> > - pwrap_writel(wrp, ~((1 << 31) | (1 << 1)), PWRAP_INT_EN);
> >
> > if (pwrap_is_mt8135(wrp)) {
> > /* enable pwrap events and pwrap bridge in AP side */
> > @@ -896,6 +892,16 @@ static int pwrap_probe(struct platform_device *pdev)
> > return -ENODEV;
> > }
> >
> > + /*
> > + * watch dog@pwrap,
> > + * The bootloader may not initial the wdt/interrupt setting of pwrap,
> > + * set it before reguesting the irq.
> > + */
>
> /* initialize watchdog, may not be done by the bootloader */
>
> Sascha
>




\
 
 \ /
  Last update: 2015-09-09 16:21    [W:0.051 / U:2.384 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site