On 2017-06-21 17:14, Nicolas Ferre wrote: > On 16/06/2017 at 00:40, Peter Rosin wrote: >> On 2017-06-15 11:37, Boris Brezillon wrote: >>> On Thu, 15 Jun 2017 11:24:13 +0200 >>> Peter Rosin <peda@xxxxxxxxxxxxxx> wrote: >>> >>>> From: Peter Rosin <peda@xxxxxxxxxx> >>>> >>>> Remove the layer. >>> >>> Duh. It was present in the datasheet I had. Just downloaded last >>> version of the datasheet and it's no longer there. >> >> Heh. >> >>> Nicolas, there's still a reference to OVR2 in the block diagram >>> (Section "31.3 Block Diagram"), can ask fix that (or ask someone who >>> can). >> >> Also, on page 2 (the Features chapter under peripherals) there's: >> >> "LCD TFT Controller with 4 overlays..." >> >> I think it should be three? > > Hi Peter and Boris, > > After checking with the designers here, it seems that it's an issue with > the latest datasheet. This is why Boris implemented it some time ago. > > So, the conclusion is: the OVR2 was removed from latest sama5d4 > documentation by mistake. It will be corrected in next datasheet release. > The OVR2 exists and works correctly on sama5d4. > > So, please don't remove this layer ;-) Right :-) I assume the CLUT offset for overlay 2 is at 0xe00, just as it is for sama5d3? When I know for sure, I can send a v5 of the other patch. > Peter, thanks a lot for your work with supporting the CLUT feature! I needed to get rid of some memory load... Cheers, peda _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
