duplicate drivers in the linux-fslc/linux-imx tree

duplicate drivers in the linux-fslc/linux-imx tree

[Donald Carr]

[-- Attachment #1: Type: text/plain, Size: 1626 bytes --]

Top of the morning,

I am busy trying to move an imx6q device from 4.9 to 6.1. This has largely been a straight forwards process, but I have hit major hiccups trying to get my attached camera sensor (ov7251) brought up.

Part of the problem with this is that there are 2 discrete ipu-v3 implementations. There is the one under:

./drivers/gpu/imx/ipu-v3

which is required by:

drivers/staging/media/imx/imx-media-dev.c:      { .compatible = "fsl,imx-capture-subsystem" },

(which I believe is required for me to grab an image from my attached sensor)

and

./drivers/mxc/ipu3

which is required by the framebuffer device (mxc_ipuv3_fb).

this can be succinctly shown here:

└─[0] <git:(cc983d0c5a23 cc983d0c5a23✈) > git grep imx6q-ipu

arch/arm/boot/dts/imx6q.dtsi:                   compatible = "fsl,imx6q-ipu";

arch/arm/boot/dts/imx6qdl.dtsi:                 compatible = "fsl,imx6q-ipu";

arch/arm/boot/dts/imx6qp.dtsi:  compatible = "fsl,imx6qp-ipu", "fsl,imx6q-ipu";

arch/arm/boot/dts/imx6qp.dtsi:  compatible = "fsl,imx6qp-ipu", "fsl,imx6q-ipu";

drivers/gpu/drm/imx/imx-drm-core.c:     "fsl,imx6q-ipu",

drivers/gpu/imx/ipu-v3/ipu-common.c:    { .compatible = "fsl,imx6q-ipu", .data = &ipu_type_imx6q, },

drivers/mxc/ipu3/ipu_common.c:  { .compatible = "fsl,imx6q-ipu", .data = &ipu_type_imx6q, },

please note both ipu commons can not be in play simultaneously and I have thus far failed to get a working compromise.

Any help/insight as to what the path of least resistance is would be greatly appreciated.

Yours sincerely,
Donald

[-- Attachment #2: Type: text/html, Size: 1918 bytes --]

Re: [meta-freescale] duplicate drivers in the linux-fslc/linux-imx tree

[Fabio Estevam]

Hi Donald,

On Wed, Aug 16, 2023 at 3:03 PM Donald Carr <sirspudd@gmail.com> wrote:
>
> Top of the morning,
>
> I am busy trying to move an imx6q device from 4.9 to 6.1. This has largely been a straight forwards process, but I have hit major hiccups trying to get my attached camera sensor (ov7251) brought up.
>
> Part of the problem with this is that there are 2 discrete ipu-v3 implementations. There is the one under:
>
> ./drivers/gpu/imx/ipu-v3
>
> which is required by:
>
> drivers/staging/media/imx/imx-media-dev.c:      { .compatible = "fsl,imx-capture-subsystem" },
>
> (which I believe is required for me to grab an image from my attached sensor)

Correct.

>
> and
>
> ./drivers/mxc/ipu3
>
> which is required by the framebuffer device (mxc_ipuv3_fb).

This is a custom NXP driver.

I suggest you use the standard driver provided by the mainline kernel.

imx6qdl-sabresd has camera supported:

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/arch/arm/boot/dts/imx6qdl-sabresd.dtsi?h=v6.1.46#n318

Here is the documentation for it:

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/tree/Documentation/admin-guide/media/imx.rst?h=v6.1.46

Hope this helps.