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. Earlier to avoid this I had included Rob's "provide a common gralloc handle definition" patch, but build-conditionalizing the importers seemed like a sane solution. I'm of course open to alternatives, but I'm not sure how to add the hisi importer and the drmgeneric one and keep things building without also adding copies of the various gralloc headers to the project as well. thanks -john _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel
