Re: [RFC][PATCH 2/4 v2] drm_hwcomposer: Add platformhisi buffer importer for hikey and hikey960

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jan 24, 2018 at 11:32 AM, Rob Herring <rob.herring@xxxxxxxxxx> wrote:
> On Wed, Jan 24, 2018 at 1:05 PM, John Stultz <john.stultz@xxxxxxxxxx> wrote:
>> On Wed, Jan 24, 2018 at 7:23 AM, Sean Paul <seanpaul@xxxxxxxxxxxx> wrote:
>>> On Tue, Jan 23, 2018 at 03:16:37PM -0800, John Stultz wrote:
>>>> This allows for importing buffers allocated from the
>>>> hikey and hikey960 gralloc implelementations.
>>>>
>>>> Cc: Marissa Wall <marissaw@xxxxxxxxxx>
>>>> Cc: Sean Paul <seanpaul@xxxxxxxxxx>
>>>> Cc: Dmitry Shmidt <dimitrysh@xxxxxxxxxx>
>>>> Cc: Robert Foss <robert.foss@xxxxxxxxxxxxx>
>>>> Cc: Matt Szczesiak <matt.szczesiak@xxxxxxx>
>>>> Cc: Liviu Dudau <Liviu.Dudau@xxxxxxx>
>>>> Cc: David Hanna <david.hanna11@xxxxxxxxx>
>>>> Cc: Rob Herring <rob.herring@xxxxxxxxxx>
>>>> Signed-off-by: John Stultz <john.stultz@xxxxxxxxxx>
>>>> ---
>>>> v2:
>>>> * Make platformhisi and the generic importer exclusive in the build
>>>
>>> I actually prefer the opposite. If everything is always compiled, we reduce the
>>> chance of breaking boards when the base class is updated. I'm sure there is a
>>> good reason for this, but perhaps there's another way?
>>
>> The main reason for this was avoiding the build breaking when
>> "gralloc_priv.h" isn't supplied or present.
>>
>> Similarly the current freedesktop/master code won't build in the
>> Android environment as the platformdrmgeneric.cpp file depends on the
>> gralloc_drm_handle.h, which doesn't exist.
>
> That will be fixed soon. We're moving the handle definition to libdrm.

How soon is that eta? Do you have a current patch for libdrm I should
work against?

thanks
-john
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/dri-devel




[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux