There's quite a lot of code that does:
if (ppc_md.progress)
ppc_md.progress(...)
So move that idiom into a wrapper. Having a wrapper also allows us
to have a fallback to printk if no progress routine is specified.
Signed-off-by: Michael Ellerman
---
arch/powerpc/include/asm/machdep.h |
On Wed, Jan 14, 2009 at 11:43 PM, Michael Ellerman
wrote:
> There's quite a lot of code that does:
>
> if (ppc_md.progress)
>ppc_md.progress(...)
>
> So move that idiom into a wrapper. Having a wrapper also allows us
> to have a fallback to printk if no progress routine is specified.
>
> S
Michael Ellerman writes:
> There's quite a lot of code that does:
>
> if (ppc_md.progress)
> ppc_md.progress(...)
>
> So move that idiom into a wrapper. Having a wrapper also allows us
> to have a fallback to printk if no progress routine is specified.
It certainly used to be the case on
> It certainly used to be the case on ppc32 (and may still be) that
> ppc_md.progress was called very very early, in some cases before the
> MMU was set up, so we don't want to call printk from it. Falling back
> to udbg_printf would be more appropriate.
>
> The idea of ppc_md.progress was that
On Thu, 2009-01-15 at 22:34 +1100, Paul Mackerras wrote:
> Michael Ellerman writes:
>
> > There's quite a lot of code that does:
> >
> > if (ppc_md.progress)
> > ppc_md.progress(...)
> >
> > So move that idiom into a wrapper. Having a wrapper also allows us
> > to have a fallback to printk i