Am 11.08.20 um 10:49 schrieb Dave Airlie:
On Tue, 11 Aug 2020 at 18:42, Christian König ckoenig.leichtzumerken@gmail.com wrote:
I've totally missed those and still don't see any reference in any inbox to the original mail or patch #2 in this series.
I forgot to cc you on the original posting, but they should be on dri-devel
https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fpatchwork....
Reviewed-by: Christian König christian.koenig@amd.com for both.
But this patch at least looks like it makes a lot of sense.
BTW: Does anybody know why we separate base and offset here? That distinction seems to be superfluous as well.
I think base is only used for premapped objects,
Yeah, but take a look how it is used. As far as I know we don't have a single case where we don't do "mem->bus.base + mem->bus.offset".
If that's true (needs double checking) then why the heck do we have two fields for this?
Regards, Christian.
Dave.