Re: iwlwifi(4) on riscv64 (intel ax200)

(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
>>