Messages in this thread | | | Date | Tue, 9 Jun 2020 18:04:48 +0200 | From | Maxime Ripard <> | Subject | Re: [PATCH 1/2] mmc: sunxi-mmc: do not hide address in sunxi_mmc_irq() |
| |
On Mon, Jun 08, 2020 at 08:54:51PM +0800, Frank Lee wrote: > On Fri, Jun 5, 2020 at 11:18 PM Maxime Ripard <maxime@cerno.tech> wrote: > > > > Hi, > > > > On Fri, Jun 05, 2020 at 08:55:44PM +0800, Frank Lee wrote: > > > From: Yangtao Li <tiny.windzz@gmail.com> > > > > > > Using %px to show the actual address in sunxi_mmc_irq() > > > to help us to debug issue. > > > > > > Signed-off-by: Yangtao Li <tiny.windzz@gmail.com> > > > Signed-off-by: Frank Lee <frank@allwinnertech.com> > > > > I'm fairly sure this hash is on purpose, and both the commit log of the > > %px introduction (ad67b74d2469) and the checkpatch message are pretty > > explicit about it. > > > > What issue were you trying to solve here? > > There doesn't seem to be a strong requirement to force this to be printed out, > I just think that ____ptrval____ is very unpleasant.
That should happen only at boot when the system doesn't have enough entropy iirc, after a while it prints out hashes.
> Or delete %p directly?
You mean the message in the driver? I'd be fine with that too.
Maxime [unhandled content-type:application/pgp-signature] | |