18.03.2021 12:31, Michał Mirosław пишет:
>>  static const struct tegra_windowgroup_soc tegra194_dc_wgrps[] = {
>> @@ -2430,6 +2781,7 @@ static const struct tegra_dc_soc_info 
>> tegra194_dc_soc_info = {
>>      .has_nvdisplay = true,
>>      .wgrps = tegra194_dc_wgrps,
>>      .num_wgrps = ARRAY_SIZE(tegra194_dc_wgrps),
>> +    .plane_tiled_memory_bandwidth_x2 = false,
>>  };
> For globals you will have .x = false by default; I'm not sure those entries
> add much value.
> 
> Reviewed-by: Michał Mirosław <mirq-li...@rere.qmqm.pl>

IIRC, in the past Thierry preferred to add the defaults to this driver
in order to ease reading/understanding of the code. So I added them for
consistency.

Thank you very much for helping with the review!

Reply via email to