Re: Octeon MMC driver

[Kevin Bowling <kevin.bowling%kev009.com@localhost>]

On Sat, Mar 28, 2026 at 12:49 AM Kevin Bowling <kevin.bowling%kev009.com@localhost> wrote:
>
> On Sun, Mar 22, 2026 at 8:44 AM Nick Hudson <nick.hudson%gmx.co.uk@localhost> wrote:
> >
> > On 19/03/2026 11:35, Kevin Bowling wrote:
> > > Hi,
> > > I have ported OpenBSD's Octeon MMC driver to NetBSD here:
> > > https://people.freebsd.org/~kbowling/oct_mmc.patch
> >
> > Cool. Thanks for working on this.
> >
> > > It seems stable with light testing and is now self booting on my ER4:
> > > Octeon ubnt_e100# set bootcmd 'fatload mmc 0 $loadaddr
> > > netbsd;bootoctlinux $loadaddr coremask=0x3 root=wedge:octeon-root'
> > > Octeon ubnt_e100# saveenv
> > > I would be curious if it works for others and if the code can be
> > > improved as I am not deeply familiar with Net or Open internals.
> > > There is a lag when attaching, which I suspect is caused by an SDIO
> > > check that could be improved.
> >
> > Some comments…
> >
> > The cache lock workaround is “interesting”
> >
> >
> > Don’t use splsdmmc, but instead use your sc_intr_mtx mutex or remove the splsdmmc calls.
> >
> > Something like the attached.
>
> https://people.freebsd.org/~kbowling/oct_mmc.patch

Sorry the link for my last message should be
https://people.freebsd.org/~kbowling/oct_mmc_v2.patch
to track the changes versus the first one.

> That worked.  I had to tweak octmmc_init_bus very slightly to still do
> the /* Enable the bus */ code before ocmtmmc_acquire(bus) or it hangs
> on attach.  I think it might be safe because we haven't enabled
> interrupts yet.
>
> >
> > I guess you’re getting your DTB from a vendor distribution?
>
> Hmm interesting.  I guess it's just coming from uboot?  OpenBSD does
> not have anything additional for e300.
>
> We seem to be in the same state as Linux DTS
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/arch/mips/boot/dts/cavium-octeon.
> I do see compatible files elsewhere like
> https://git.openwrt.org/openwrt/openwrt/tree/target/linux/octeon/files/arch/mips/boot/dts/cavium-octeon.
>
> >
> > ~/netbsd/nbcvs/src % grep -r  "cavium,octeon-[0-9]*-mmc" sys/external/gpl2/dts/dist
> > ~/netbsd/nbcvs/src %
> >
> > You should provide patches to
> >
> >     sys/arch/mips/dts
> >
> > See sys/arch/arm/dts for example how a board’s dts is changed compared
> > to the Linux mainline upstream
>
> I was able to reduce one timeout and quiet an error print with this
>
>         * Refuse SDIO probe. Proper SDIO operation is not possible
>         * because of a lack of card interrupt handling.
>         */
> -       if (cmd->c_opcode == SD_IO_SEND_OP_COND) {
> -               cmd->c_error = ENOTSUP;
> +       if (cmd->c_opcode == SD_IO_SEND_OP_COND ||
> +           cmd->c_opcode == SD_IO_RW_DIRECT ||
> +           cmd->c_opcode == SD_IO_RW_EXTENDED) {
> +               cmd->c_error = ETIMEDOUT;
>                return;
>        }
>
> but I have to imagine there is a better way to handle this.  Even with
> that there seems to be two silent timeout intervals delaying boot by
> 10s (vs 15s).
>
> > Nick
> >
> >
> >
> >
> >