Re: Marvell sk/msk driver NetBSD 8.0_RC1

[Jaromír Doleček <jaromir.dolecek%gmail.com@localhost>]

Sure, that should be enough - I misread the email thread, I thought
the boot message is still that the pci device is not configured.

So yeah, apparently it's some newer variant, which needs some more
changes in the driver to actually work.

Jaromir

2018-06-08 15:59 GMT+02:00 John Halfpenny <jsh%sdf.org@localhost>:
> On Thu, Jun 07, 2018 at 07:56:20PM +0200, Jaromir Dolecek wrote:
>> You also need to add the id into the list in the driver, used for matching.
>
> I thought I already had by adding an entry in if_msk.c?
>
> If I grep out my changes-
>
> ./if_msk.c:     { PCI_VENDOR_MARVELL,           PCI_PRODUCT_MARVELL_YUKONII_8071 },
> ./pcidevs:product MARVELL YUKONII_8071  0x436b  Yukon-II 88E8071
> ./pcidevs_data.h:           PCI_VENDOR_MARVELL, PCI_PRODUCT_MARVELL_YUKONII_8071,
> ./pcidevs.h:#define     PCI_PRODUCT_MARVELL_YUKONII_8071        0x436b          /* Yukon-II 88E8071 */
>
> (pcidevs.h and pcidevs_data.h were both generated using Makefile.pcidevs)
>
> # pcictl pci0 list | grep -i marvell
> 002:00:0: Marvell product 436b (ethernet network, revision 0x16)
>
> Have I missed something somewhere?
>
> Bw
> John
>
> --
> jsh%sdf.org@localhost
> SDF Public Access UNIX System - http://sdf.org