On Fri, Mar 31, 2023 at 10:13:44PM +0200, Mark Kettenis wrote:
> > Date: Fri, 31 Mar 2023 20:07:51 +0000
> > From: Klemens Nanni <k...@openbsd.org>
> > 
> > On Fri, Mar 31, 2023 at 08:12:20PM +0200, Mark Kettenis wrote:
> > > The name BM1000 only seems to come up in unofficial device trees.
> > > There is a preliminary data sheet for a BE-M1000 SoC out there?  Is
> > > that the proper name?
> > 
> > Right, the CPU and SoC are called "Baikal-M1" and "BE-M1000", respectively.
> > 
> > > If the host bridge is the only PCIe device device on the SoC, consider
> > > naming it just "BE-M1000" (or BM1000 if that actually is the right
> > > name).  That is what I did for the Rockchip PCIe host bridges.
> > 
> > Makes sense, thanks.
> > 
> > OK?
> 
> You can't have a dash in the #define.  I think standard practive is to
> simply omit the dash, so...

Fat fingered underscore, which I'd prefer.
But now I see how other dashes are dropped;  will go with whatever is OK.

Index: pcidevs
===================================================================
RCS file: /cvs/src/sys/dev/pci/pcidevs,v
retrieving revision 1.2027
diff -u -p -r1.2027 pcidevs
--- pcidevs     31 Mar 2023 08:16:30 -0000      1.2027
+++ pcidevs     31 Mar 2023 20:21:17 -0000
@@ -344,6 +344,7 @@ vendor      SYMPHONY2       0x1c1c  Symphony Labs
 vendor SKHYNIX         0x1c5c  SK hynix
 vendor ADATA           0x1cc1  ADATA Technology
 vendor UMIS            0x1cc4  Union Memory
+vendor BAIKAL          0x1d39  Baikal Electronics
 vendor AQUANTIA        0x1d6a  Aquantia
 vendor ROCKCHIP        0x1d87  Rockchip
 vendor TEKRAM2         0x1de1  Tekram
@@ -2421,6 +2422,9 @@ product AVM T1                    0x1200  PRI T1 ISDN
 
 /* AWT products */
 product AWT RT2890             0x1059  RT2890
+
+/* Baikal products */
+product BAIKAL BEM1000         0x8060  BE-M1000
 
 /* Belkin Components products */
 product BELKIN F5D6000         0xec00  F5D6000

Reply via email to