Skip to content

Commit 4e39ade

Browse files
Dr. David Alan Gilberthdeller
authored andcommitted
video: hdmi: Remove unused hdmi_infoframe_check
hdmi_infoframe_check() has been unused since it was added in commit c5e69ab ("video/hdmi: Constify infoframe passed to the pack functions") Remove it. Note that the individual check functions for each type are actually used, so they're staying. Signed-off-by: Dr. David Alan Gilbert <[email protected]> Signed-off-by: Helge Deller <[email protected]>
1 parent dcbcf52 commit 4e39ade

File tree

2 files changed

+0
-29
lines changed

2 files changed

+0
-29
lines changed

drivers/video/hdmi.c

Lines changed: 0 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -894,34 +894,6 @@ hdmi_vendor_any_infoframe_pack(union hdmi_vendor_any_infoframe *frame,
894894
return hdmi_vendor_any_infoframe_pack_only(frame, buffer, size);
895895
}
896896

897-
/**
898-
* hdmi_infoframe_check() - check a HDMI infoframe
899-
* @frame: HDMI infoframe
900-
*
901-
* Validates that the infoframe is consistent and updates derived fields
902-
* (eg. length) based on other fields.
903-
*
904-
* Returns 0 on success or a negative error code on failure.
905-
*/
906-
int
907-
hdmi_infoframe_check(union hdmi_infoframe *frame)
908-
{
909-
switch (frame->any.type) {
910-
case HDMI_INFOFRAME_TYPE_AVI:
911-
return hdmi_avi_infoframe_check(&frame->avi);
912-
case HDMI_INFOFRAME_TYPE_SPD:
913-
return hdmi_spd_infoframe_check(&frame->spd);
914-
case HDMI_INFOFRAME_TYPE_AUDIO:
915-
return hdmi_audio_infoframe_check(&frame->audio);
916-
case HDMI_INFOFRAME_TYPE_VENDOR:
917-
return hdmi_vendor_any_infoframe_check(&frame->vendor);
918-
default:
919-
WARN(1, "Bad infoframe type %d\n", frame->any.type);
920-
return -EINVAL;
921-
}
922-
}
923-
EXPORT_SYMBOL(hdmi_infoframe_check);
924-
925897
/**
926898
* hdmi_infoframe_pack_only() - write a HDMI infoframe to binary buffer
927899
* @frame: HDMI infoframe

include/linux/hdmi.h

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -445,7 +445,6 @@ ssize_t hdmi_infoframe_pack(union hdmi_infoframe *frame, void *buffer,
445445
size_t size);
446446
ssize_t hdmi_infoframe_pack_only(const union hdmi_infoframe *frame,
447447
void *buffer, size_t size);
448-
int hdmi_infoframe_check(union hdmi_infoframe *frame);
449448
int hdmi_infoframe_unpack(union hdmi_infoframe *frame,
450449
const void *buffer, size_t size);
451450
void hdmi_infoframe_log(const char *level, struct device *dev,

0 commit comments

Comments
 (0)