Date: Mon, 13 Jun 2022 08:32:51 -0600 From: Warner Losh <imp@bsdimp.com> To: John Baldwin <jhb@freebsd.org> Cc: "Bjoern A. Zeeb" <bz@freebsd.org>, src-committers <src-committers@freebsd.org>, "<dev-commits-src-all@freebsd.org>" <dev-commits-src-all@freebsd.org>, dev-commits-src-main@freebsd.org Subject: Re: git: 0f7b9777f8f3 - main - rtw88: split driver up into a core and pci part Message-ID: <CANCZdfoqnsJKLtqrwpY35EMGKoQvEYnn158HQNBHEg1=X%2BKZZA@mail.gmail.com> In-Reply-To: <d514e0de-9b30-b5c8-2e2f-7e93a36747f7@FreeBSD.org> References: <202206121843.25CIhcLr014633@gitrepo.freebsd.org> <d514e0de-9b30-b5c8-2e2f-7e93a36747f7@FreeBSD.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--00000000000050146a05e1552a8a Content-Type: text/plain; charset="UTF-8" On Mon, Jun 13, 2022, 8:28 AM John Baldwin <jhb@freebsd.org> wrote: > On 6/12/22 11:43 AM, Bjoern A. Zeeb wrote: > > The branch main has been updated by bz: > > > > URL: > https://cgit.FreeBSD.org/src/commit/?id=0f7b9777f8f39fbc230b3e1de2f844d9f839adea > > > > commit 0f7b9777f8f39fbc230b3e1de2f844d9f839adea > > Author: Bjoern A. Zeeb <bz@FreeBSD.org> > > AuthorDate: 2022-06-12 18:35:58 +0000 > > Commit: Bjoern A. Zeeb <bz@FreeBSD.org> > > CommitDate: 2022-06-12 18:35:58 +0000 > > > > rtw88: split driver up into a core and pci part > > > > Split the driver up into two modules (if_rtw88_pci.ko and > rtw88_core.ko). > > This is in preparation for the hopefully eventually upcoming USB > support > > using the same driver core. > > > > Note: this changes the module name to load to if_rtw88_pci.ko > instead of > > if_rtw88.ko. If using devmatch(8) everything should stay the same > as > > the driver name (used for net.wlan.devices) stays rtw88. If using > > kld_list in rc.conf or loader.conf you will need to adjust the name. > > Update man page for this. > > > > MFC after: 3 days > > This sort of split in a .ko is kind of rare for drivers in the tree that > support > multiple bus attachments. Usually we just lump all the attachments into > the same > .ko. It's true that with the death of ISA, etc. we no longer have as many > drivers > with multiple bus attachments, but the norm has been to include them all > in a > single .ko. Is there a reason you can't follow the normal practice here? > Agreed. Furthermore, in the past when a couple of drivers did the we had issues and confusion. Please don't MFC until this discussion is done. Warner > --00000000000050146a05e1552a8a Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"auto"><div><br><br><div class=3D"gmail_quote"><div dir=3D"ltr" = class=3D"gmail_attr">On Mon, Jun 13, 2022, 8:28 AM John Baldwin <<a href= =3D"mailto:jhb@freebsd.org">jhb@freebsd.org</a>> wrote:<br></div><blockq= uote class=3D"gmail_quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc = solid;padding-left:1ex">On 6/12/22 11:43 AM, Bjoern A. Zeeb wrote:<br> > The branch main has been updated by bz:<br> > <br> > URL: <a href=3D"https://cgit.FreeBSD.org/src/commit/?id=3D0f7b9777f8f3= 9fbc230b3e1de2f844d9f839adea" rel=3D"noreferrer noreferrer" target=3D"_blan= k">https://cgit.FreeBSD.org/src/commit/?id=3D0f7b9777f8f39fbc230b3e1de2f844= d9f839adea</a><br> > <br> > commit 0f7b9777f8f39fbc230b3e1de2f844d9f839adea<br> > Author:=C2=A0 =C2=A0 =C2=A0Bjoern A. Zeeb <bz@FreeBSD.org><br> > AuthorDate: 2022-06-12 18:35:58 +0000<br> > Commit:=C2=A0 =C2=A0 =C2=A0Bjoern A. Zeeb <bz@FreeBSD.org><br> > CommitDate: 2022-06-12 18:35:58 +0000<br> > <br> >=C2=A0 =C2=A0 =C2=A0 rtw88: split driver up into a core and pci part<br= > >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 Split the driver up into two modules (if_rtw88_pci= .ko and rtw88_core.ko).<br> >=C2=A0 =C2=A0 =C2=A0 This is in preparation for the hopefully eventuall= y upcoming USB support<br> >=C2=A0 =C2=A0 =C2=A0 using the same driver core.<br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 Note: this changes the module name to load to if_r= tw88_pci.ko instead of<br> >=C2=A0 =C2=A0 =C2=A0 if_rtw88.ko.=C2=A0 If using devmatch(8) everything= should stay the same as<br> >=C2=A0 =C2=A0 =C2=A0 the driver name (used for net.wlan.devices) stays = rtw88.=C2=A0 If using<br> >=C2=A0 =C2=A0 =C2=A0 kld_list in rc.conf or loader.conf you will need t= o adjust the name.<br> >=C2=A0 =C2=A0 =C2=A0 Update man page for this.<br> >=C2=A0 =C2=A0 =C2=A0 <br> >=C2=A0 =C2=A0 =C2=A0 MFC after:=C2=A0 =C2=A0 =C2=A0 3 days<br> <br> This sort of split in a .ko is kind of rare for drivers in the tree that su= pport<br> multiple bus attachments.=C2=A0 Usually we just lump all the attachments in= to the same<br> .ko.=C2=A0 It's true that with the death of ISA, etc. we no longer have= as many drivers<br> with multiple bus attachments, but the norm has been to include them all in= a<br> single .ko.=C2=A0 Is there a reason you can't follow the normal practic= e here?<br></blockquote></div></div><div dir=3D"auto"><br></div><div dir=3D= "auto">Agreed. Furthermore, in the past when a couple of drivers did the we= had issues and confusion. Please don't MFC until this discussion is do= ne.=C2=A0</div><div dir=3D"auto"><br></div><div dir=3D"auto">Warner=C2=A0</= div><div dir=3D"auto"><div class=3D"gmail_quote"><blockquote class=3D"gmail= _quote" style=3D"margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:= 1ex"> </blockquote></div></div></div> --00000000000050146a05e1552a8a--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CANCZdfoqnsJKLtqrwpY35EMGKoQvEYnn158HQNBHEg1=X%2BKZZA>