DMARC-Filter: OpenDMARC Filter v1.3.2 mx1.redhat.com EAC71C0567A3
Message-ID: <1493208671.23739.19.camel@redhat.com>
Subject: Re: [PATCH 3/6] drm: fourcc byteorder: add bigendian support to
 drm_mode_legacy_fb_format
From: Gerd Hoffmann <kraxel@redhat.com>
To: Michel =?ISO-8859-1?Q?D=E4nzer?= <michel@daenzer.net>
Cc: Daniel Vetter <daniel.vetter@intel.com>, amd-gfx@lists.freedesktop.org,
        open list <linux-kernel@vger.kernel.org>,
        dri-devel@lists.freedesktop.org
Date: Wed, 26 Apr 2017 14:11:11 +0200
In-Reply-To: <8f91cc58-16dc-5899-66b6-06d430a18801@daenzer.net>
References: <20170424062532.26722-1-kraxel@redhat.com>
         <20170424062532.26722-4-kraxel@redhat.com>
         <3b872a56-80b5-0c44-712f-a9517489eb24@daenzer.net>
         <1493185990.23739.7.camel@redhat.com>
         <8f91cc58-16dc-5899-66b6-06d430a18801@daenzer.net>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 8BIT
Mime-Version: 1.0
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 819
Lines: 23

  Hi,

> >> Just to reiterate, this won't work for the radeon driver, which programs
> >> the GPU to use (effectively, per the current definition that these are
> >> little endian GPU formats) DRM_FORMAT_XRGB8888 with pre-R600 and
> >> DRM_FORMAT_BGRX8888 with >= R600.
> > 
> > Hmm, ok, how does bigendian fbdev emulation work on pre-R600 then?
> 
> Using a GPU byte swapping mechanism which only affects CPU access to
> video RAM.

That is done using the RADEON_TILING_SWAP_{16,32}BIT flag mentioned in
another thread?

Ok, so the cpu view to fbdev is DRM_FORMAT_BGRX8888 in all cases.

What about dumb bos?  You've mentioned the swap flag isn't used for
those.  Which implies they are in little endian byte order (both gpu and
cpu view).  Does the modesetting driver work correctly on that hardware?

cheers,
  Gerd