On 05.09.24 09:16, Jan Kiszka wrote: > On 05.09.24 08:57, Krzysztof Kozlowski wrote: >> On 04/09/2024 12:00, Jan Kiszka wrote: >>> + >>> + reg-names: >>> + minItems: 4 >>> + items: >>> + - const: app >>> + - const: dbics >>> + - const: config >>> + - const: atu >>> + - const: vmap_lp >>> + - const: vmap_hp >>> + >>> + memory-region: >> >> This also did not improve. You did not address any feedback from v3. >> >> Missed feedback: >> >> This *must* be defined in top-level. > > Answered in the other reply. > >> I still think this must have some sort of maxItems. I accept your >> explanation that you could have multiple memory pools, but I don't think >> 2147000 pools is possible. Make it 4, 8 or 32. > > Can do - if you can explain the benefit of that. > It turned out during further investigations that swiotlb actually only allows a single restricted DMA pool per device. So this discussion auto-resolves, bindings and code will be adjusted in v5. Jan -- Siemens AG, Technology Linux Expert Center
