Re: [PATCH 5/6] driver core: lift dma_default_coherent into common code

From: Maciej W. Rozycki
Date: Sat Feb 27 2021 - 13:43:34 EST


On Mon, 22 Feb 2021, Maciej W. Rozycki wrote:

> > > I haven't booted Linux on my Malta for a while now, but it turns out to
> > > work just fine, and your patch set does not regress it booting multi-user
> > > NFS-rooted over FDDI.
> > >
> > > I note however that the system does not reboot properly:
> > >
> > > sd 0:0:0:0: [sda] Synchronizing SCSI cache
> > > reboot: Restarting system
> > > Reboot failed -- System halted
> > >
> > > which is a regression, and also the MMIO-mapped discrete CBUS UART (ttyS2)
> > > does not sign in anymore either:
> >
> > Do you mean a regression with this series, or just compared to when you
> > last tested?
>
> When last tested. Years ago, so nothing for you to be concerned. I'll
> look into it sometime unless someone beats me to. Don't hold your breath
> though. Sorry to be unclear.

For the record, Malta reboot requires:

CONFIG_POWER_RESET=y
CONFIG_POWER_RESET_SYSCON=y

to work these days, which wasn't picked automatically on an older config
regeneration for me. Sorry for the noise then, although ISTM that these
would better be picked up automatically by reverse dependencies. What's
the point of omitting reboot support?

Still looking into the CBUS UART issue.

Maciej