[media] platform: Make use of media_bus_format enum
In order to have subsytem agnostic media bus format definitions we've moved media bus definition to include/uapi/linux/media-bus-format.h and prefixed values with MEDIA_BUS_FMT instead of V4L2_MBUS_FMT. Reference new definitions in all platform drivers. Signed-off-by: Boris Brezillon <boris.brezillon@free-electrons.com> Acked-by: Hans Verkuil <hans.verkuil@cisco.com> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Acked-by: Sekhar Nori <nsekhar@ti.com> Acked-by: Lad, Prabhakar <prabhakar.csengg@gmail.com> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
This commit is contained in:

committad av
Mauro Carvalho Chehab

förälder
43ba464182
incheckning
27ffaeb0ab
@@ -964,18 +964,16 @@ static void preview_setup_hw(struct isp_prev_device *prev, u32 update,
|
||||
* @prev: pointer to previewer private structure
|
||||
* @pixelcode: pixel code
|
||||
*/
|
||||
static void
|
||||
preview_config_ycpos(struct isp_prev_device *prev,
|
||||
enum v4l2_mbus_pixelcode pixelcode)
|
||||
static void preview_config_ycpos(struct isp_prev_device *prev, u32 pixelcode)
|
||||
{
|
||||
struct isp_device *isp = to_isp_device(prev);
|
||||
enum preview_ycpos_mode mode;
|
||||
|
||||
switch (pixelcode) {
|
||||
case V4L2_MBUS_FMT_YUYV8_1X16:
|
||||
case MEDIA_BUS_FMT_YUYV8_1X16:
|
||||
mode = YCPOS_CrYCbY;
|
||||
break;
|
||||
case V4L2_MBUS_FMT_UYVY8_1X16:
|
||||
case MEDIA_BUS_FMT_UYVY8_1X16:
|
||||
mode = YCPOS_YCrYCb;
|
||||
break;
|
||||
default:
|
||||
@@ -1028,16 +1026,16 @@ static void preview_config_input_format(struct isp_prev_device *prev,
|
||||
ISPPRV_PCR_WIDTH);
|
||||
|
||||
switch (info->flavor) {
|
||||
case V4L2_MBUS_FMT_SGRBG8_1X8:
|
||||
case MEDIA_BUS_FMT_SGRBG8_1X8:
|
||||
prev->params.cfa_order = 0;
|
||||
break;
|
||||
case V4L2_MBUS_FMT_SRGGB8_1X8:
|
||||
case MEDIA_BUS_FMT_SRGGB8_1X8:
|
||||
prev->params.cfa_order = 1;
|
||||
break;
|
||||
case V4L2_MBUS_FMT_SBGGR8_1X8:
|
||||
case MEDIA_BUS_FMT_SBGGR8_1X8:
|
||||
prev->params.cfa_order = 2;
|
||||
break;
|
||||
case V4L2_MBUS_FMT_SGBRG8_1X8:
|
||||
case MEDIA_BUS_FMT_SGBRG8_1X8:
|
||||
prev->params.cfa_order = 3;
|
||||
break;
|
||||
default:
|
||||
@@ -1078,8 +1076,8 @@ static void preview_config_input_size(struct isp_prev_device *prev, u32 active)
|
||||
unsigned int elv = prev->crop.top + prev->crop.height - 1;
|
||||
u32 features;
|
||||
|
||||
if (format->code != V4L2_MBUS_FMT_Y8_1X8 &&
|
||||
format->code != V4L2_MBUS_FMT_Y10_1X10) {
|
||||
if (format->code != MEDIA_BUS_FMT_Y8_1X8 &&
|
||||
format->code != MEDIA_BUS_FMT_Y10_1X10) {
|
||||
sph -= 2;
|
||||
eph += 2;
|
||||
slv -= 2;
|
||||
@@ -1709,21 +1707,21 @@ __preview_get_crop(struct isp_prev_device *prev, struct v4l2_subdev_fh *fh,
|
||||
|
||||
/* previewer format descriptions */
|
||||
static const unsigned int preview_input_fmts[] = {
|
||||
V4L2_MBUS_FMT_Y8_1X8,
|
||||
V4L2_MBUS_FMT_SGRBG8_1X8,
|
||||
V4L2_MBUS_FMT_SRGGB8_1X8,
|
||||
V4L2_MBUS_FMT_SBGGR8_1X8,
|
||||
V4L2_MBUS_FMT_SGBRG8_1X8,
|
||||
V4L2_MBUS_FMT_Y10_1X10,
|
||||
V4L2_MBUS_FMT_SGRBG10_1X10,
|
||||
V4L2_MBUS_FMT_SRGGB10_1X10,
|
||||
V4L2_MBUS_FMT_SBGGR10_1X10,
|
||||
V4L2_MBUS_FMT_SGBRG10_1X10,
|
||||
MEDIA_BUS_FMT_Y8_1X8,
|
||||
MEDIA_BUS_FMT_SGRBG8_1X8,
|
||||
MEDIA_BUS_FMT_SRGGB8_1X8,
|
||||
MEDIA_BUS_FMT_SBGGR8_1X8,
|
||||
MEDIA_BUS_FMT_SGBRG8_1X8,
|
||||
MEDIA_BUS_FMT_Y10_1X10,
|
||||
MEDIA_BUS_FMT_SGRBG10_1X10,
|
||||
MEDIA_BUS_FMT_SRGGB10_1X10,
|
||||
MEDIA_BUS_FMT_SBGGR10_1X10,
|
||||
MEDIA_BUS_FMT_SGBRG10_1X10,
|
||||
};
|
||||
|
||||
static const unsigned int preview_output_fmts[] = {
|
||||
V4L2_MBUS_FMT_UYVY8_1X16,
|
||||
V4L2_MBUS_FMT_YUYV8_1X16,
|
||||
MEDIA_BUS_FMT_UYVY8_1X16,
|
||||
MEDIA_BUS_FMT_YUYV8_1X16,
|
||||
};
|
||||
|
||||
/*
|
||||
@@ -1742,7 +1740,7 @@ static void preview_try_format(struct isp_prev_device *prev,
|
||||
struct v4l2_mbus_framefmt *fmt,
|
||||
enum v4l2_subdev_format_whence which)
|
||||
{
|
||||
enum v4l2_mbus_pixelcode pixelcode;
|
||||
u32 pixelcode;
|
||||
struct v4l2_rect *crop;
|
||||
unsigned int i;
|
||||
|
||||
@@ -1774,7 +1772,7 @@ static void preview_try_format(struct isp_prev_device *prev,
|
||||
|
||||
/* If not found, use SGRBG10 as default */
|
||||
if (i >= ARRAY_SIZE(preview_input_fmts))
|
||||
fmt->code = V4L2_MBUS_FMT_SGRBG10_1X10;
|
||||
fmt->code = MEDIA_BUS_FMT_SGRBG10_1X10;
|
||||
break;
|
||||
|
||||
case PREV_PAD_SOURCE:
|
||||
@@ -1782,13 +1780,13 @@ static void preview_try_format(struct isp_prev_device *prev,
|
||||
*fmt = *__preview_get_format(prev, fh, PREV_PAD_SINK, which);
|
||||
|
||||
switch (pixelcode) {
|
||||
case V4L2_MBUS_FMT_YUYV8_1X16:
|
||||
case V4L2_MBUS_FMT_UYVY8_1X16:
|
||||
case MEDIA_BUS_FMT_YUYV8_1X16:
|
||||
case MEDIA_BUS_FMT_UYVY8_1X16:
|
||||
fmt->code = pixelcode;
|
||||
break;
|
||||
|
||||
default:
|
||||
fmt->code = V4L2_MBUS_FMT_YUYV8_1X16;
|
||||
fmt->code = MEDIA_BUS_FMT_YUYV8_1X16;
|
||||
break;
|
||||
}
|
||||
|
||||
@@ -1843,8 +1841,8 @@ static void preview_try_crop(struct isp_prev_device *prev,
|
||||
* and no columns in other modes. Increase the margins based on the sink
|
||||
* format.
|
||||
*/
|
||||
if (sink->code != V4L2_MBUS_FMT_Y8_1X8 &&
|
||||
sink->code != V4L2_MBUS_FMT_Y10_1X10) {
|
||||
if (sink->code != MEDIA_BUS_FMT_Y8_1X8 &&
|
||||
sink->code != MEDIA_BUS_FMT_Y10_1X10) {
|
||||
left += 2;
|
||||
right -= 2;
|
||||
top += 2;
|
||||
@@ -2092,7 +2090,7 @@ static int preview_init_formats(struct v4l2_subdev *sd,
|
||||
memset(&format, 0, sizeof(format));
|
||||
format.pad = PREV_PAD_SINK;
|
||||
format.which = fh ? V4L2_SUBDEV_FORMAT_TRY : V4L2_SUBDEV_FORMAT_ACTIVE;
|
||||
format.format.code = V4L2_MBUS_FMT_SGRBG10_1X10;
|
||||
format.format.code = MEDIA_BUS_FMT_SGRBG10_1X10;
|
||||
format.format.width = 4096;
|
||||
format.format.height = 4096;
|
||||
preview_set_format(sd, fh, &format);
|
||||
|
Referens i nytt ärende
Block a user