[DPU PATCH] drm: Remove unused drm_connector fields.

[DPU PATCH] drm: Remove unused drm_connector fields.

[Sean Paul]

They're not used, so let's get rid of them.

Signed-off-by: Sean Paul <seanpaul@chromium.org>
---
 include/drm/drm_connector.h | 19 -------------------
 1 file changed, 19 deletions(-)

diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
index 9d9d5405ebfd..ed38df4ac204 100644
--- a/include/drm/drm_connector.h
+++ b/include/drm/drm_connector.h
@@ -754,15 +754,6 @@ struct drm_cmdline_mode {
  * @audio_latency: audio latency info from ELD, if found
  * @null_edid_counter: track sinks that give us all zeros for the EDID
  * @bad_edid_counter: track sinks that give us an EDID with invalid checksum
- * @pt_scan_info: PT scan info obtained from the VCDB of EDID
- * @it_scan_info: IT scan info obtained from the VCDB of EDID
- * @ce_scan_info: CE scan info obtained from the VCDB of EDID
- * @hdr_eotf: Electro optical transfer function obtained from HDR block
- * @hdr_metadata_type_one: Metadata type one obtained from HDR block
- * @hdr_max_luminance: desired max luminance obtained from HDR block
- * @hdr_avg_luminance: desired avg luminance obtained from HDR block
- * @hdr_min_luminance: desired min luminance obtained from HDR block
- * @hdr_supported: does the sink support HDR content
  * @edid_corrupt: indicates whether the last read EDID was corrupt
  * @debugfs_entry: debugfs directory for this connector
  * @has_tile: is this connector connected to a tiled monitor
@@ -943,16 +934,6 @@ struct drm_connector {
 	int null_edid_counter; /* needed to workaround some HW bugs where we get all 0s */
 	unsigned bad_edid_counter;
 
-	u8 pt_scan_info;
-	u8 it_scan_info;
-	u8 ce_scan_info;
-	u32 hdr_eotf;
-	bool hdr_metadata_type_one;
-	u32 hdr_max_luminance;
-	u32 hdr_avg_luminance;
-	u32 hdr_min_luminance;
-	bool hdr_supported;
-
 	/* Flag for raw EDID header corruption - used in Displayport
 	 * compliance testing - * Displayport Link CTS Core 1.2 rev1.1 4.2.2.6
 	 */
-- 
2.16.1.291.g4437f3f132-goog

_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

Re: [DPU PATCH] drm: Remove unused drm_connector fields.

[jsanka-sgV2jX0FEOL9JmXXK+q4OQ]

On 2018-02-26 08:49, Sean Paul wrote:
> They're not used, so let's get rid of them.
> 
> Signed-off-by: Sean Paul <seanpaul@chromium.org>

Reviewed-by: Jeykumar Sankaran <jsanka@codeaurora.org>

> ---
>  include/drm/drm_connector.h | 19 -------------------
>  1 file changed, 19 deletions(-)
> 
> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
> index 9d9d5405ebfd..ed38df4ac204 100644
> --- a/include/drm/drm_connector.h
> +++ b/include/drm/drm_connector.h
> @@ -754,15 +754,6 @@ struct drm_cmdline_mode {
>   * @audio_latency: audio latency info from ELD, if found
>   * @null_edid_counter: track sinks that give us all zeros for the EDID
>   * @bad_edid_counter: track sinks that give us an EDID with invalid
> checksum
> - * @pt_scan_info: PT scan info obtained from the VCDB of EDID
> - * @it_scan_info: IT scan info obtained from the VCDB of EDID
> - * @ce_scan_info: CE scan info obtained from the VCDB of EDID
> - * @hdr_eotf: Electro optical transfer function obtained from HDR 
> block
> - * @hdr_metadata_type_one: Metadata type one obtained from HDR block
> - * @hdr_max_luminance: desired max luminance obtained from HDR block
> - * @hdr_avg_luminance: desired avg luminance obtained from HDR block
> - * @hdr_min_luminance: desired min luminance obtained from HDR block
> - * @hdr_supported: does the sink support HDR content
>   * @edid_corrupt: indicates whether the last read EDID was corrupt
>   * @debugfs_entry: debugfs directory for this connector
>   * @has_tile: is this connector connected to a tiled monitor
> @@ -943,16 +934,6 @@ struct drm_connector {
>  	int null_edid_counter; /* needed to workaround some HW bugs where
> we get all 0s */
>  	unsigned bad_edid_counter;
> 
> -	u8 pt_scan_info;
> -	u8 it_scan_info;
> -	u8 ce_scan_info;
> -	u32 hdr_eotf;
> -	bool hdr_metadata_type_one;
> -	u32 hdr_max_luminance;
> -	u32 hdr_avg_luminance;
> -	u32 hdr_min_luminance;
> -	bool hdr_supported;
> -
>  	/* Flag for raw EDID header corruption - used in Displayport
>  	 * compliance testing - * Displayport Link CTS Core 1.2 rev1.1
> 4.2.2.6
>  	 */
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

Re: [DPU PATCH] drm: Remove unused drm_connector fields.

[abhinavk-sgV2jX0FEOL9JmXXK+q4OQ]

On 2018-03-02 12:06, jsanka@codeaurora.org wrote:
> On 2018-02-26 08:49, Sean Paul wrote:
>> They're not used, so let's get rid of them.
>> 
>> Signed-off-by: Sean Paul <seanpaul@chromium.org>
> 
> Reviewed-by: Jeykumar Sankaran <jsanka@codeaurora.org>
> 
>> ---
>>  include/drm/drm_connector.h | 19 -------------------
>>  1 file changed, 19 deletions(-)
>> 
>> diff --git a/include/drm/drm_connector.h b/include/drm/drm_connector.h
>> index 9d9d5405ebfd..ed38df4ac204 100644
>> --- a/include/drm/drm_connector.h
>> +++ b/include/drm/drm_connector.h
>> @@ -754,15 +754,6 @@ struct drm_cmdline_mode {
>>   * @audio_latency: audio latency info from ELD, if found
>>   * @null_edid_counter: track sinks that give us all zeros for the 
>> EDID
>>   * @bad_edid_counter: track sinks that give us an EDID with invalid
>> checksum
>> - * @pt_scan_info: PT scan info obtained from the VCDB of EDID
>> - * @it_scan_info: IT scan info obtained from the VCDB of EDID
>> - * @ce_scan_info: CE scan info obtained from the VCDB of EDID
>> - * @hdr_eotf: Electro optical transfer function obtained from HDR 
>> block
>> - * @hdr_metadata_type_one: Metadata type one obtained from HDR block
>> - * @hdr_max_luminance: desired max luminance obtained from HDR block
>> - * @hdr_avg_luminance: desired avg luminance obtained from HDR block
>> - * @hdr_min_luminance: desired min luminance obtained from HDR block
>> - * @hdr_supported: does the sink support HDR content
>>   * @edid_corrupt: indicates whether the last read EDID was corrupt
>>   * @debugfs_entry: debugfs directory for this connector
>>   * @has_tile: is this connector connected to a tiled monitor
>> @@ -943,16 +934,6 @@ struct drm_connector {
>>  	int null_edid_counter; /* needed to workaround some HW bugs where
>> we get all 0s */
>>  	unsigned bad_edid_counter;
>> 
>> -	u8 pt_scan_info;
>> -	u8 it_scan_info;
>> -	u8 ce_scan_info;
>> -	u32 hdr_eotf;
>> -	bool hdr_metadata_type_one;
>> -	u32 hdr_max_luminance;
>> -	u32 hdr_avg_luminance;
>> -	u32 hdr_min_luminance;
>> -	bool hdr_supported;
>> -
>>  	/* Flag for raw EDID header corruption - used in Displayport
>>  	 * compliance testing - * Displayport Link CTS Core 1.2 rev1.1
>> 4.2.2.6
>>  	 */
> --
> To unsubscribe from this list: send the line "unsubscribe 
> linux-arm-msm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno

Re: [DPU PATCH] drm: Remove unused drm_connector fields.

[abhinavk-sgV2jX0FEOL9JmXXK+q4OQ]

On 2018-03-02 12:17, abhinavk@codeaurora.org wrote:
> On 2018-03-02 12:06, jsanka@codeaurora.org wrote:
>> On 2018-02-26 08:49, Sean Paul wrote:
>>> They're not used, so let's get rid of them.
>>> 
>>> Signed-off-by: Sean Paul <seanpaul@chromium.org>
>> 
>> Reviewed-by: Jeykumar Sankaran <jsanka@codeaurora.org>
Reviewed-by: Abhinav Kumar <abhinavk@codeaurora.org>
>> 
>>> ---
>>>  include/drm/drm_connector.h | 19 -------------------
>>>  1 file changed, 19 deletions(-)
>>> 
>>> diff --git a/include/drm/drm_connector.h 
>>> b/include/drm/drm_connector.h
>>> index 9d9d5405ebfd..ed38df4ac204 100644
>>> --- a/include/drm/drm_connector.h
>>> +++ b/include/drm/drm_connector.h
>>> @@ -754,15 +754,6 @@ struct drm_cmdline_mode {
>>>   * @audio_latency: audio latency info from ELD, if found
>>>   * @null_edid_counter: track sinks that give us all zeros for the 
>>> EDID
>>>   * @bad_edid_counter: track sinks that give us an EDID with invalid
>>> checksum
>>> - * @pt_scan_info: PT scan info obtained from the VCDB of EDID
>>> - * @it_scan_info: IT scan info obtained from the VCDB of EDID
>>> - * @ce_scan_info: CE scan info obtained from the VCDB of EDID
>>> - * @hdr_eotf: Electro optical transfer function obtained from HDR 
>>> block
>>> - * @hdr_metadata_type_one: Metadata type one obtained from HDR block
>>> - * @hdr_max_luminance: desired max luminance obtained from HDR block
>>> - * @hdr_avg_luminance: desired avg luminance obtained from HDR block
>>> - * @hdr_min_luminance: desired min luminance obtained from HDR block
>>> - * @hdr_supported: does the sink support HDR content
>>>   * @edid_corrupt: indicates whether the last read EDID was corrupt
>>>   * @debugfs_entry: debugfs directory for this connector
>>>   * @has_tile: is this connector connected to a tiled monitor
>>> @@ -943,16 +934,6 @@ struct drm_connector {
>>>  	int null_edid_counter; /* needed to workaround some HW bugs where
>>> we get all 0s */
>>>  	unsigned bad_edid_counter;
>>> 
>>> -	u8 pt_scan_info;
>>> -	u8 it_scan_info;
>>> -	u8 ce_scan_info;
>>> -	u32 hdr_eotf;
>>> -	bool hdr_metadata_type_one;
>>> -	u32 hdr_max_luminance;
>>> -	u32 hdr_avg_luminance;
>>> -	u32 hdr_min_luminance;
>>> -	bool hdr_supported;
>>> -
>>>  	/* Flag for raw EDID header corruption - used in Displayport
>>>  	 * compliance testing - * Displayport Link CTS Core 1.2 rev1.1
>>> 4.2.2.6
>>>  	 */
>> --
>> To unsubscribe from this list: send the line "unsubscribe 
>> linux-arm-msm" in
>> the body of a message to majordomo@vger.kernel.org
>> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> --
> To unsubscribe from this list: send the line "unsubscribe 
> linux-arm-msm" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
_______________________________________________
Freedreno mailing list
Freedreno@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/freedreno