The enum v4l2_pixel_encoding can be used instead of rkisp1_fmt_pix_type. This allows cleanup of the enum rkisp1_fmt_pix_type. Also, since the v4l2_pixel_encoding can be retrieved from the info->pixel_enc of the capture formats, the field fmt_type can be dropped from the struct rkisp1_capture_fmt_cfg. Patch 1: replace rkisp1_fmt_pix_type with v4l2_pixel_encoding Patch 2: drop the field fmt_type from the struct rkisp1_capture_fmt_cfg Patch 2: rename the fields fmt_type to the more informative name pixel_enc Changes from v1: v1 was a single patch that only drops the fmt_type from the struct rkisp1_capture_fmt_cfg and adds a function rkisp1_pixel_enc_to_fmt_pix which is in v2 not needed. Changes from v2: Add patch 3 that rename the fields fmt_type to pixel_enc Dafna Hirschfeld (3): media: staging: rkisp1: replace rkisp1_fmt_pix_type with v4l2_pixel_encoding media: staging: rkisp1: cap: remove field fmt_type from struct rkisp1_capture_fmt_cfg media: staging: rkisp1: change fields names from fmt_type to pixel_enc drivers/staging/media/rkisp1/rkisp1-capture.c | 51 +------------------ drivers/staging/media/rkisp1/rkisp1-common.h | 11 +--- drivers/staging/media/rkisp1/rkisp1-isp.c | 42 +++++++-------- drivers/staging/media/rkisp1/rkisp1-resizer.c | 12 ++--- 4 files changed, 31 insertions(+), 85 deletions(-) -- 2.20.1