(Sorry, I spaced removing the Cc:)
On Sat, Jan 17, 2026 at 06:19:46PM +0100, Tassilo Philipp wrote:
[off-list]
Mh... ich kanns leider nicht auf riscv64 selber bauen, weil:
# cd /usr/src/sys/modules/iwlwifi
# make
make: /usr/src/sys/modules/iwlwifi/Makefile:7: Variable "KERN_OPTS" is undefined
while evaluating variable "KERN_OPTS" with value ""
ich glaub da fehlt wohl:
SYSDIR?=${SRCTOP}/sys
.include "${SYSDIR}/conf/kern.opts.mk"
Aber auch damit gehts nicht. Bin grad zwischen Tür und Angel, sonst
würd ichs weiter debuggen.
Und... in der /usr/src/sys/modules/Makefile ist der iwlwifi Teil so
drin:
.if ${MACHINE_CPUARCH} == "aarch64" || ${MACHINE_CPUARCH} == "amd64" || \
${MACHINE_CPUARCH} == "i386"
_ena= ena
_gve= gve
_iwlwifi= iwlwifi
_rtw88= rtw88
_rtw89= rtw89
_vmware= vmware
.endif
Sollte da nicht noch auf riscv getestet werden? Bin mir hier nicht
sicher, ob das nötig ist.
Tut mir leid, dass mir das alles nicht beim Review aufgefallen ist vor
ein paar Tagen.
Grüße
On Wed, Jan 14, 2026 at 10:14:23AM +0000, Bjoern A. Zeeb wrote:
On Wed, 14 Jan 2026, Tassilo Philipp wrote:
On Wed, Jan 14, 2026 at 06:43:46AM +0000, Bjoern A. Zeeb wrote:
Hi,
yeah, this looks good. Different way to make it compile, so I
guess the #ifdefs I added are not needed, but keeping the linux
headers around was the key.
Not modifying upstream Linux driver code was the goal to do this
differently than just adding the #ifdefs to iwlwifi.
Do you need any help w/ testing?
Always :) You can either pull the changes in manually or wait
until they are landed. I am not yet sure what the response to acpi
on riscv will be but given it does exist elsehwere I hope it'll be
fine.
--
Bjoern A. Zeeb r15:7
