krzk changed the topic of #linux-exynos to: Linux Samsung SoC Exynos and Family | https://exynos.wiki.kernel.org | This channel is logged: https://libera.irclog.whitequark.org/linux-exynos
Peng_Fan has joined #linux-exynos
tambarus has joined #linux-exynos
<tambarus> krzk: Hi! Can I change to optional a dt property that's currently marked as mandatory? I'm talking about the dma property in Documentation/devicetree/bindings/spi/samsung,spi.yaml. SPI does not have an internal DMA, it uses DMA channels from a DMA controller that feeds other IPs as well.
<tambarus> krzk: the driver switches to PIO mode when dmas is not specified in DT, we shall be fine
<krzk> tambarus: in general property can be made un-required, but I don't know why it was made required in the first place
<tambarus> krzk: okay, thanks for the hint. I'll study it a bit and come up with a proposal
GNUtoo has quit [Ping timeout: 260 seconds]
<tambarus> krzk: dmas was marked as mandatory because the driver failed to probe if the property was absent. It's not a hardware requirement, it's just that the driver was poorly written. Since the driver was updated to make dmas optional just in 2022, I can't guarantee that older exynos platforms don't need the DMA. I assume they don't, but I can't verify. I'll make it optional just for gs101. Others can
<tambarus> follow.
GNUtoo has joined #linux-exynos
<krzk> tambarus: both approaches work for me
GNUtoo has quit [Remote host closed the connection]
GNUtoo has joined #linux-exynos
tambarus has quit [Ping timeout: 246 seconds]