On Tue, 2020-09-29 at 13:55 +1000, Alexey Kardashevskiy wrote:

On 12/09/2020 03:07, Leonardo Bras wrote:

quoted

Cc: linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org,

A previous change introduced the usage of DDW as a bigger indirect DMA
mapping when the DDW available size does not map the whole partition.

As most of the code that manipulates direct mappings was reused for
indirect mappings, it's necessary to rename all names and debug/info
messages to reflect that it can be used for both kinds of mapping.

Also, defines DEFAULT_DMA_WIN as "ibm,dma-window" to document that
it's the name of the default DMA window.

"ibm,dma-window" is so old so it does not need a macro (which btw would 
be DMA_WIN_PROPNAME to match the other names) :)

Thanks for bringing that to my attention!
In fact, DMA_WIN_PROPNAME makes more sense, but it's still generic and
doesn't look to point to a generic one.

Would that be ok to call it DEFAULT_WIN_PROPNAME ?

quoted

Those changes are not supposed to change how the code works in any
way, just adjust naming.

I simply have this in my .vimrc for the cases like this one:

===
This should cause no behavioural change.
===

Great tip! I will make sure to have this saved here :)

Thank you!