linux-kernel - RE: [EXTERNAL] Re: [PATCH v10 1/1] misc: mrvl-cn10k-dpi: add Octeon CN10K DPI administrative driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: 
 <SJ0PR18MB52467C544972534F6DECDC48A6A62@SJ0PR18MB5246.namprd18.prod.outlook.com>
Date: Fri, 12 Jul 2024 08:44:22 +0000
From: Vamsi Krishna Attunuru <vattunuru@...vell.com>
To: Greg KH <gregkh@...uxfoundation.org>
CC: "arnd@...db.de" <arnd@...db.de>, Jerin Jacob <jerinj@...vell.com>,
        Srujana
 Challa <schalla@...vell.com>,
        "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>
Subject: RE: [EXTERNAL] Re: [PATCH v10 1/1] misc: mrvl-cn10k-dpi: add Octeon
 CN10K DPI administrative driver


>> >> >>
>> >> >> +config MARVELL_CN10K_DPI
>> >> >> +	tristate "Octeon CN10K DPI driver"
>> >> >> +	depends on ARM64 && PCI
>> >> >
>> >> >Why does ARM64 matter here?  I don't see any dependency required
>of it.
>> >> >
>> >> Thanks, Greg, for your time. This DPI device is an on-chip PCIe
>> >> device and only present on Marvell's CN10K platforms(which are
>> >> 64-bit ARM SoC
>> >processors), so added those dependency.
>> >
>> >Then perhaps keep the ARM64 and add a COMPILE_TEST option as well so
>> >that we can build this as part of normal testing?
>> >
>> >So that would be:
>> >	depends on PCI && (ARM64 || COMPILE_TEST) right?
>> >
>> Yes, it makes sense to add. Can I send this fix as next version now so
>> that it will show up in next release, please suggest.
>
>Send it as a follow-on patch on top of my tree, doing what Arnd suggested.

Hi Greg, I submitted the follow-on patch on top of your char-misc-testing.
Please let me know if there is any additional action required on my part.

Regards
Vamsi
>
>thanks,
>
>greg k-h