ikwzm / udmabuf

User space mappable dma buffer device driver for Linux.
BSD 2-Clause "Simplified" License
560 stars 168 forks source link

[modify] Makefile for built-in Kernel #44

Closed tichkr closed 4 years ago

tichkr commented 4 years ago

Qiita共々いつも参考にさせて頂いております。 Kernelへの組み込みも選択できるようにMakefileを修正させて頂きました。 よろしければマージお願いします。

ikwzm commented 4 years ago

issue ありがとうございます。

さて、いただいた PR ですが、少し問題があります。というのも、Readme.md で少し触れていますが、Linux Kernel 5.0 以降に、この udmabuf と同じ名前でまったく別のカーネルドライバが正式に組み込まれています。そのカーネルドライバを組み込む際も CONFIG_UDMABUF を使います。

https://elixir.bootlin.com/linux/v5.4.33/source/drivers/dma-buf/Kconfig

したがって、このまま PR をマージすると、Linux Kernel 5.0 以降では CONFIG_UDMABUF を指定すると、本来の udmabuf と、この udmabuf が同時に組み込まれてしまう恐れがあります。

この udmabuf は Linux Kernel 4.20 以前までの対応のみと割り切ってしまえば良いのですが。。。

tichkr commented 4 years ago

返信ありがとうございます。 Kconfigも変更が必要になりますが、Linux Kernel 5.0 以降の対応で変更された名前を元に、UDMABUFからU_DMABUFに変更するというのは如何でしょうか? ※「U-DMA-BUF」だとエラーなりますので「」にしています。

ikwzm commented 4 years ago

少し検討させてください。

ikwzm commented 4 years ago

tichkr さんの提案通りに Kconfig の名前を UDMABUF から U_DMA_BUF に変更しました。

どうもありがとうございました。