[waffle] [PATCH 06/10] gbm: wegl_display ok in wgbm_config_get_gbm_format

Frank Henigman fjhenigman at google.com
Fri Apr 10 08:34:53 PDT 2015


Awesome, thanks.  I got a bit worried when I saw the merge commit
message mention "platform_null" but you took stuff from the right
branch, which is named just "null."  Sorry about the minefield of
confusing branch names.

On Thu, Apr 9, 2015 at 6:10 PM, Chad Versace <chad.versace at intel.com> wrote:
> On Thu 09 Apr 2015, Chad Versace wrote:
>>
>> On Mon 30 Mar 2015, Frank Henigman wrote:
>>>
>>> From: Frank Henigman <fjhenigman at gmail.com>
>>>
>>> wgbm_config_get_gbm_format() does not need to cast the display all the
>>> way to wgbm_display; wegl_display is far enough.
>>> Now wgbm_config_get_gbm_format() can be used outside of the gbm platform.
>>>
>>> Signed-off-by: Frank Henigman <fjhenigman at google.com>
>>> ---
>>> src/waffle/gbm/wgbm_config.c | 4 ++--
>>> 1 file changed, 2 insertions(+), 2 deletions(-)
>>
>>
>> Patches 2-6 are
>> Reviewed-by: Chad Versace <chad.versace at intel.com>
>>
>> I'm still thinking about your questions in the cover letter,
>> and haven't read patches 7-10 yet.
>
>
> Frank, to speed things along I added my r-b to patches 1-6 of your
> 'null1' tag, made a small change to patch 1, and pushed to master.
>
> See
> https://github.com/waffle-gl/waffle/commit/038b1dbc184054382bc89f197cf7c7811aeed5a3


More information about the waffle mailing list