RE: [PATCH 1/1] Serial: imx: add dev_pm_ops to support suspend to ram/disk

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> -----Original Message-----
> From: Greg KH [mailto:gregkh@xxxxxxxxxxxxxxxxxxx]
> Sent: 2015年7月23日 19:37
> To: Wang Shenwei-B38339
> Cc: linux-serial@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/1] Serial: imx: add dev_pm_ops to support suspend to
> ram/disk
> > +	sport->saved_reg[4] = readl(sport->port.membase + UFCR);
> > +	sport->saved_reg[5] = readl(sport->port.membase + UESC);
> > +	sport->saved_reg[6] = readl(sport->port.membase + UTIM);
> > +	sport->saved_reg[7] = readl(sport->port.membase + UBIR);
> > +	sport->saved_reg[8] = readl(sport->port.membase + UBMR);
> > +	sport->saved_reg[9] = readl(sport->port.membase + IMX21_UTS);
> > +	clk_disable(sport->clk_ipg);
> > +
> > +	pr_debug("0x%p (%d)\r\n", sport->port.membase, sport->port.line);
> 
> You have a struct device, please use dev_dbg() instead.

Adopted in PATCH v2.

> 
> > +
> > +	return 0;
> > +}
> > +
> > +	writel(sport->saved_reg[2], sport->port.membase + UCR3);
> > +	writel(sport->saved_reg[3], sport->port.membase + UCR4);
> > +	clk_disable(sport->clk_ipg);
> > +
> > +	pr_debug("0x%p (%d)\r\n", sport->port.membase, sport->port.line);
> 
> Same here, please use dev_dbg()

Adopted in PATCH v2.

Thanks,
Shenwei

?韬{.n?????%??檩??w?{.n???{炳谦??{ay????j?f"??????_璁(????"??m???G??⒏??璀?x??




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux