On Mon, Mar 23, 2020 at 8:50 PM Jason A. Donenfeld <Jason@xxxxxxxxx> wrote: > > I would have expected your arch/x86/Kconfig.assembler file as > > arch/x86/crypto/Kconfig (source include needs to be adapted in > > arch/x86/Kconfig). > > CONFIG_AS_* is required for more than just the crypto. > OK. I was not aware of this. > > What about a commit subject like "x86: crypto: Probe assembler options > > via Kconfig instead of makefile"? > > Thanks. Will fix silly verbiage and update branch. > Just looked to what I see new in [0]. Would you mind to add the patch "Documentation/changes: Raise minimum supported binutils version to 2.23" from [1] to your series, please? For the meantime and clarification - you can drop it later (with adding Link-tag to [1]) when it landed in tip Git [2] where I am not seeing it. Thanks for taking care to you an Masah*e*ro :-). - Sedat - [0] https://git.zx2c4.com/linux-dev/log/?h=jd/kconfig-assembler-support [1] https://lore.kernel.org/lkml/20200316160259.GN26126@xxxxxxx/ [2] https://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git/
