Message ID | 1473248229-5540-8-git-send-email-stanimir.varbanov@linaro.org (mailing list archive) |
---|---|
State | Not Applicable, archived |
Delegated to: | Andy Gross |
Headers | show |
On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: > Makefile and Kconfig files to build the video codec driver. > > Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> > --- > drivers/media/platform/qcom/Kconfig | 8 ++++++++ > drivers/media/platform/qcom/Makefile | 6 ++++++ > drivers/media/platform/qcom/vidc/Makefile | 15 +++++++++++++++ > 3 files changed, 29 insertions(+) > create mode 100644 drivers/media/platform/qcom/Kconfig > create mode 100644 drivers/media/platform/qcom/Makefile > create mode 100644 drivers/media/platform/qcom/vidc/Makefile > > diff --git a/drivers/media/platform/qcom/Kconfig b/drivers/media/platform/qcom/Kconfig > new file mode 100644 > index 000000000000..4bad5c0f68e4 > --- /dev/null > +++ b/drivers/media/platform/qcom/Kconfig > @@ -0,0 +1,8 @@ > +comment "Qualcomm V4L2 drivers" > + > +menuconfig QCOM_VIDC > + tristate "Qualcomm V4L2 encoder/decoder driver" > + depends on ARCH_QCOM && VIDEO_V4L2 > + depends on IOMMU_DMA > + depends on QCOM_VENUS_PIL > + select VIDEOBUF2_DMA_SG If at all possible, please depend on COMPILE_TEST as well! Also missing: a patch adding an entry to the MAINTAINERS file. Regards, Hans > diff --git a/drivers/media/platform/qcom/Makefile b/drivers/media/platform/qcom/Makefile > new file mode 100644 > index 000000000000..150892f6533b > --- /dev/null > +++ b/drivers/media/platform/qcom/Makefile > @@ -0,0 +1,6 @@ > +# > +# Makefile for the QCOM spcific video device drivers > +# based on V4L2. > +# > + > +obj-$(CONFIG_QCOM_VIDC) += vidc/ > diff --git a/drivers/media/platform/qcom/vidc/Makefile b/drivers/media/platform/qcom/vidc/Makefile > new file mode 100644 > index 000000000000..f8b5f9a438ee > --- /dev/null > +++ b/drivers/media/platform/qcom/vidc/Makefile > @@ -0,0 +1,15 @@ > +# Makefile for Qualcomm vidc driver > + > +vidc-objs += \ > + core.o \ > + helpers.o \ > + vdec.o \ > + vdec_ctrls.o \ > + venc.o \ > + venc_ctrls.o \ > + hfi_venus.o \ > + hfi_msgs.o \ > + hfi_cmds.o \ > + hfi.o \ > + > +obj-$(CONFIG_QCOM_VIDC) += vidc.o I recommend renaming the module to qcom-vidc. 'vidc' is too generic. Regards, Hans > -- 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
Hi Hans, On 09/19/2016 01:35 PM, Hans Verkuil wrote: > On 09/07/2016 01:37 PM, Stanimir Varbanov wrote: >> Makefile and Kconfig files to build the video codec driver. >> >> Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> >> --- >> drivers/media/platform/qcom/Kconfig | 8 ++++++++ >> drivers/media/platform/qcom/Makefile | 6 ++++++ >> drivers/media/platform/qcom/vidc/Makefile | 15 +++++++++++++++ >> 3 files changed, 29 insertions(+) >> create mode 100644 drivers/media/platform/qcom/Kconfig >> create mode 100644 drivers/media/platform/qcom/Makefile >> create mode 100644 drivers/media/platform/qcom/vidc/Makefile >> >> diff --git a/drivers/media/platform/qcom/Kconfig b/drivers/media/platform/qcom/Kconfig >> new file mode 100644 >> index 000000000000..4bad5c0f68e4 >> --- /dev/null >> +++ b/drivers/media/platform/qcom/Kconfig >> @@ -0,0 +1,8 @@ >> +comment "Qualcomm V4L2 drivers" >> + >> +menuconfig QCOM_VIDC >> + tristate "Qualcomm V4L2 encoder/decoder driver" >> + depends on ARCH_QCOM && VIDEO_V4L2 >> + depends on IOMMU_DMA >> + depends on QCOM_VENUS_PIL >> + select VIDEOBUF2_DMA_SG > > If at all possible, please depend on COMPILE_TEST as well! OK, I will add it. > > Also missing: a patch adding an entry to the MAINTAINERS file. I will add such a patch in next submission.
diff --git a/drivers/media/platform/qcom/Kconfig b/drivers/media/platform/qcom/Kconfig new file mode 100644 index 000000000000..4bad5c0f68e4 --- /dev/null +++ b/drivers/media/platform/qcom/Kconfig @@ -0,0 +1,8 @@ +comment "Qualcomm V4L2 drivers" + +menuconfig QCOM_VIDC + tristate "Qualcomm V4L2 encoder/decoder driver" + depends on ARCH_QCOM && VIDEO_V4L2 + depends on IOMMU_DMA + depends on QCOM_VENUS_PIL + select VIDEOBUF2_DMA_SG diff --git a/drivers/media/platform/qcom/Makefile b/drivers/media/platform/qcom/Makefile new file mode 100644 index 000000000000..150892f6533b --- /dev/null +++ b/drivers/media/platform/qcom/Makefile @@ -0,0 +1,6 @@ +# +# Makefile for the QCOM spcific video device drivers +# based on V4L2. +# + +obj-$(CONFIG_QCOM_VIDC) += vidc/ diff --git a/drivers/media/platform/qcom/vidc/Makefile b/drivers/media/platform/qcom/vidc/Makefile new file mode 100644 index 000000000000..f8b5f9a438ee --- /dev/null +++ b/drivers/media/platform/qcom/vidc/Makefile @@ -0,0 +1,15 @@ +# Makefile for Qualcomm vidc driver + +vidc-objs += \ + core.o \ + helpers.o \ + vdec.o \ + vdec_ctrls.o \ + venc.o \ + venc_ctrls.o \ + hfi_venus.o \ + hfi_msgs.o \ + hfi_cmds.o \ + hfi.o \ + +obj-$(CONFIG_QCOM_VIDC) += vidc.o
Makefile and Kconfig files to build the video codec driver. Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org> --- drivers/media/platform/qcom/Kconfig | 8 ++++++++ drivers/media/platform/qcom/Makefile | 6 ++++++ drivers/media/platform/qcom/vidc/Makefile | 15 +++++++++++++++ 3 files changed, 29 insertions(+) create mode 100644 drivers/media/platform/qcom/Kconfig create mode 100644 drivers/media/platform/qcom/Makefile create mode 100644 drivers/media/platform/qcom/vidc/Makefile