All of lore.kernel.org
 help / color / mirror / Atom feed
From: Marco Felsch <m.felsch@pengutronix.de>
To: Greg Ungerer <gerg@linux-m68k.org>
Cc: peng.fan@nxp.com,
	Alexander Stein <alexander.stein@ew.tq-group.com>,
	abailon@baylibre.com, krzysztof.kozlowski+dt@linaro.org,
	festevam@gmail.com, abelvesa@kernel.org, marex@denx.de,
	Markus.Niebel@ew.tq-group.com,
	Ahmad Fatoum <a.fatoum@pengutronix.de>,
	paul.elder@ideasonboard.com, gerg@kernel.org, linux-imx@nxp.com,
	devicetree@vger.kernel.org,
	"Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
	linux-pm@vger.kernel.org, s.hauer@pengutronix.de,
	robh+dt@kernel.org, aford173@gmail.com,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
	laurent.pinchart@ideasonboard.com, linux-kernel@vger.kernel.org,
	kernel@pengutronix.de, djakov@kernel.org, shawnguo@kernel.org,
	l.stach@pengutronix.de
Subject: Re: [PATCH V3 7/7] arm64: dts: imx8mp: add interconnect for hsio blk ctrl
Date: Tue, 28 Mar 2023 17:11:00 +0200	[thread overview]
Message-ID: <20230328151100.msl46qupstwplkgw@pengutronix.de> (raw)
In-Reply-To: <c368a0f8-41f0-69ac-04f4-459e5fc8b9d6@linux-m68k.org>

Hi Greg,

On 23-03-29, Greg Ungerer wrote:
> Hi Marco,

...

> > I forgot to ask: Does your i.MX8MP have a VPU? There are i.MX8MP devices
> > (don't know the name) which don't have support for certain IPs. If this
> 
> The hardware platform I have is using the MIMX8ML4CVNKZAB "i.MX 8M Plus QuadLite"
> (https://www.nxp.com/part/MIMX8ML4CVNKZAB#/) which does not have the hardware
> video encode/decoder module (like the "i.MX 8M Plus Quad" parts).

and that's the problem :) You need to update your bootloader to a
version which support disabling the VPU nodes else you will always see
the errors.

> > is the case the bootloader will fixup your devicetree by disable the
> > corresponding nodes, we call this feature-controller:
> > 
> > https://elixir.bootlin.com/barebox/latest/source/arch/arm/dts/imx8mp.dtsi
> > 
> > As you can see the imx8mp.dtsi is missing the feature bits for the VPU
> > but you can check the i.mx8mm.dtsi. Here you can see that barebox will
> > check the availability of the vpu:
> > 
> > https://elixir.bootlin.com/barebox/latest/source/arch/arm/dts/imx8mm.dtsi
> 
> Ok, thanks, I'll take a look.

Patches are welcome if you use barebox :)

Regards,
  Marco

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

WARNING: multiple messages have this Message-ID (diff)
From: Marco Felsch <m.felsch@pengutronix.de>
To: Greg Ungerer <gerg@linux-m68k.org>
Cc: peng.fan@nxp.com,
	Alexander Stein <alexander.stein@ew.tq-group.com>,
	abailon@baylibre.com, krzysztof.kozlowski+dt@linaro.org,
	festevam@gmail.com, abelvesa@kernel.org, marex@denx.de,
	Markus.Niebel@ew.tq-group.com,
	Ahmad Fatoum <a.fatoum@pengutronix.de>,
	paul.elder@ideasonboard.com, gerg@kernel.org, linux-imx@nxp.com,
	devicetree@vger.kernel.org,
	"Peng Fan (OSS)" <peng.fan@oss.nxp.com>,
	linux-pm@vger.kernel.org, s.hauer@pengutronix.de,
	robh+dt@kernel.org, aford173@gmail.com,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	laurent.pinchart@ideasonboard.com, linux-kernel@vger.kernel.org,
	kernel@pengutronix.de, djakov@kernel.org, shawnguo@kernel.org,
	l.stach@pengutronix.de
Subject: Re: [PATCH V3 7/7] arm64: dts: imx8mp: add interconnect for hsio blk ctrl
Date: Tue, 28 Mar 2023 17:11:00 +0200	[thread overview]
Message-ID: <20230328151100.msl46qupstwplkgw@pengutronix.de> (raw)
In-Reply-To: <c368a0f8-41f0-69ac-04f4-459e5fc8b9d6@linux-m68k.org>

Hi Greg,

On 23-03-29, Greg Ungerer wrote:
> Hi Marco,

...

> > I forgot to ask: Does your i.MX8MP have a VPU? There are i.MX8MP devices
> > (don't know the name) which don't have support for certain IPs. If this
> 
> The hardware platform I have is using the MIMX8ML4CVNKZAB "i.MX 8M Plus QuadLite"
> (https://www.nxp.com/part/MIMX8ML4CVNKZAB#/) which does not have the hardware
> video encode/decoder module (like the "i.MX 8M Plus Quad" parts).

and that's the problem :) You need to update your bootloader to a
version which support disabling the VPU nodes else you will always see
the errors.

> > is the case the bootloader will fixup your devicetree by disable the
> > corresponding nodes, we call this feature-controller:
> > 
> > https://elixir.bootlin.com/barebox/latest/source/arch/arm/dts/imx8mp.dtsi
> > 
> > As you can see the imx8mp.dtsi is missing the feature bits for the VPU
> > but you can check the i.mx8mm.dtsi. Here you can see that barebox will
> > check the availability of the vpu:
> > 
> > https://elixir.bootlin.com/barebox/latest/source/arch/arm/dts/imx8mm.dtsi
> 
> Ok, thanks, I'll take a look.

Patches are welcome if you use barebox :)

Regards,
  Marco

  reply	other threads:[~2023-03-28 15:12 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-07-03  9:14 [PATCH V3 0/7] imx: blk-ctrl: Add interconnect for i.MX8MP Peng Fan (OSS)
2022-07-03  9:14 ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 1/7] dt-bindings: soc: imx: add interconnect property for i.MX8MP media blk ctrl Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 2/7] dt-bindings: soc: imx: add interconnect property for i.MX8MP hdmi " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 3/7] dt-bindings: soc: imx: add interconnect property for i.MX8MP hsio " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 4/7] soc: imx: add icc paths for i.MX8MP media " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 5/7] soc: imx: add icc paths for i.MX8MP hsio/hdmi " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-03  9:14 ` [PATCH V3 6/7] arm64: dts: imx8mp: add interconnects for media " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2022-07-07 11:38   ` Marco Felsch
2022-07-07 11:38     ` Marco Felsch
2022-07-08  0:06     ` Peng Fan
2022-07-08  0:06       ` Peng Fan
2022-07-03  9:14 ` [PATCH V3 7/7] arm64: dts: imx8mp: add interconnect for hsio " Peng Fan (OSS)
2022-07-03  9:14   ` Peng Fan (OSS)
2023-03-27  4:50   ` Greg Ungerer
2023-03-27  4:50     ` Greg Ungerer
2023-03-27  6:27     ` Alexander Stein
2023-03-27  6:27       ` Alexander Stein
2023-03-27  7:16       ` Ahmad Fatoum
2023-03-27  7:16         ` Ahmad Fatoum
2023-03-27  8:07         ` Greg Ungerer
2023-03-27  8:07           ` Greg Ungerer
2023-03-28  7:33           ` Marco Felsch
2023-03-28  7:33             ` Marco Felsch
2023-03-28 12:58             ` Greg Ungerer
2023-03-28 12:58               ` Greg Ungerer
2023-03-28 13:42               ` Marco Felsch
2023-03-28 13:42                 ` Marco Felsch
2023-03-28 13:51                 ` Marco Felsch
2023-03-28 13:51                   ` Marco Felsch
2023-03-28 14:34                   ` Greg Ungerer
2023-03-28 14:34                     ` Greg Ungerer
2023-03-28 15:11                     ` Marco Felsch [this message]
2023-03-28 15:11                       ` Marco Felsch
2023-03-31  5:55                       ` Greg Ungerer
2023-03-31  5:55                         ` Greg Ungerer
2023-03-31  7:45                         ` Markus Niebel
2023-03-31  7:45                           ` Markus Niebel
2023-03-31  8:11                           ` Ahmad Fatoum
2023-03-31  8:11                             ` Ahmad Fatoum
2023-03-31 15:09                             ` Greg Ungerer
2023-03-31 15:09                               ` Greg Ungerer
2023-03-27  7:51       ` Greg Ungerer
2023-03-27  7:51         ` Greg Ungerer

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20230328151100.msl46qupstwplkgw@pengutronix.de \
    --to=m.felsch@pengutronix.de \
    --cc=Markus.Niebel@ew.tq-group.com \
    --cc=a.fatoum@pengutronix.de \
    --cc=abailon@baylibre.com \
    --cc=abelvesa@kernel.org \
    --cc=aford173@gmail.com \
    --cc=alexander.stein@ew.tq-group.com \
    --cc=devicetree@vger.kernel.org \
    --cc=djakov@kernel.org \
    --cc=festevam@gmail.com \
    --cc=gerg@kernel.org \
    --cc=gerg@linux-m68k.org \
    --cc=kernel@pengutronix.de \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=l.stach@pengutronix.de \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-imx@nxp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pm@vger.kernel.org \
    --cc=marex@denx.de \
    --cc=paul.elder@ideasonboard.com \
    --cc=peng.fan@nxp.com \
    --cc=peng.fan@oss.nxp.com \
    --cc=robh+dt@kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=shawnguo@kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.