This reverts commit 887ba96a.

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Signed-off-by: Tomeu Vizoso <tomeu.vizoso@collabora.com>

Nowadays mali_kbase tells us when we need to mmap() things, not the
other way around. Unfortunately it's a little rude and doesn't seem to
specify the right output flag for requiring an mmap, so we just tell if
an mmap is required by checking for the magic number that's always the
first cookie. Let's hope we never try multithreading with this!

This might have something to do with the kbase API we're reporting.

Lyude Paul authored Nov 18, 2018 and Tomeu Vizoso committed Dec 04, 2018

When we started this project, I had 'interesting' ideas. Mainly, I had
made our own custom headers for interacting with mali_kbase because the
headers they had were a humongous mess. I wasn't aware the burden of
maintainership it'd incur, and nowadays the kbase headers are a lot more
acceptable anyway.

Since we're updating to the latest mali_kbase uapi, I decided to just
throw them out and start using the actual headers. Now we do that using
a simple bridging header, panfrost-mali-base.h, which handles the
various defines and other things needed to make including the kbase
headers 'just work'.

Additionally-move panfrost-int.h to panfrost-misc.h so we can stick some
other required types into it.

This commit doesn't get things onto the screen on my T82x board quite
yet, but it does communicate with the kernel properly!

Lyude Paul authored Nov 18, 2018 and Tomeu Vizoso committed Dec 04, 2018

Since we're about to pull in all of ARM's official headers for their
kernel driver instead of our own, and all of those are prefixed with
mali-* or mali_*, let's rename ours to panfrost-* so there isn't any
confusion.

No functional changes.

This was missed when I added the new enum.

Cc: 18.3 <mesa-stable@lists.freedesktop.org>
Reviewed-by: Bas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Reviewed-by: Leo Liu <leo.liu@amd.com>

Andrii Simiklit authored Nov 13, 2018 and Eric Engestrom committed Nov 14, 2018

1. brw_pipe_control.c:311:34: warning:
    unused variable ‘devinfo’
2. brw_program_binary.c:209:19: warning:
    unused variable ‘gen_size’
3. brw_program_binary.c:216:19: warning:
    unused variable ‘nir_size’

v2: Changes for unreproducible issues were removed

Signed-off-by: Andrii Simiklit <andrii.simiklit@globallogic.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>

Andrii Simiklit authored Nov 13, 2018 and Eric Engestrom committed Nov 14, 2018

1. nir/nir_lower_vars_to_ssa.c:691:21: warning:
       unused variable ‘var’
       nir_variable *var = path->path[0]->var;

v2: Changes for some part of 'may be used uninitialized'
    warnings were removed, seems like it is a compiler issue.
        ( Eric Engestrom <eric.engestrom@intel.com> )
    Possible like this one:
    https://gcc.gnu.org/bugzilla/show_bug.cgi?id=46684


    This issue is flagged as duplicate but an
    original one is not closed yet.

Signed-off-by: Andrii Simiklit <andrii.simiklit@globallogic.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>

Andrii Simiklit authored Nov 13, 2018 and Eric Engestrom committed Nov 14, 2018

1. tools/aub_read.c:271:31: warning: unused variable ‘end’
    const uint32_t *p = data, *end = data + data_len, *next;

2. tools/aub_mem.c:292:13: warning: unused variable ‘res’
       void *res = mmap((uint8_t *)bo.map + map_offset, 4096, PROT_READ,
   tools/aub_mem.c:357:13: warning: unused variable ‘res’
       void *res = mmap((uint8_t *)bo.map + (page - bo.addr), 4096, PROT_READ,

v2: The i965_disasm.c changes was moved into a separate patch
    The 'end' variable declared separately with MAYBE_UNUSED
    to avoid effect of it to other variables.
       ( Eric Engestrom <eric.engestrom@intel.com> )

Signed-off-by: Andrii Simiklit <andrii.simiklit@globallogic.com>
Reviewed-by: Eric Engestrom <eric.engestrom@intel.com>

Bump minor to signal support for new formats and higher precision
solid pictures.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>

Support Component Alpha for those composite operations that do not require
per-channel alpha blending.

Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>

constify function arguments to clean up the code a bit.

Reported-by: Brian Paul <brianp@vmware.com>
Signed-off-by: Thomas Hellstrom <thellstrom@vmware.com>
Reviewed-by: Brian Paul <brianp@vmware.com>
Reviewed-by: Sinclair Yeh <syeh@vmware.com>