Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1571289

Re: [PATCH 1/2] ARM: dts: i.MX25: add AIPS control registers

Show all headers | View raw

On Tue, Jan 31, 2017 at 08:20:01PM +0100, Martin Kaiser wrote:
> The i.MX25 contains two AHB to IP bridges (AIPS), each of which has a set of
> control registers. Add the memory regions for the control registers to
> the Device Tree.
> 
> All the registers from the i.MX53 AIPSTZ are present on the i.MX25 as
> well. Indicate that we are compatible to fsl,imx53-aipstz. However,
> don't use aipstz for our name, this seems to be specific to i.MX53.
> 
> Signed-off-by: Martin Kaiser <martin@kaiser.cx>
> ---
>  arch/arm/boot/dts/imx25.dtsi | 10 ++++++++++
>  1 file changed, 10 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/imx25.dtsi b/arch/arm/boot/dts/imx25.dtsi
> index e0ba550..4c4768f 100644
> --- a/arch/arm/boot/dts/imx25.dtsi
> +++ b/arch/arm/boot/dts/imx25.dtsi
> @@ -93,6 +93,11 @@
>  			reg = <0x43f00000 0x100000>;
>  			ranges;
>  
> +			aips1: bridge@43f00000 {
> +				compatible = "fsl,imx25-aips", "fsl,imx53-aipstz";
> +				reg = <0x43f00000 0x60>;
> +			};

The additional compatible properties should indicate if there is an
older ip block this one is compatible to. Since i.MX53 is newer than
i.MX25 this goes into the wrong direction. If anything you could add the
"fsl,imx25-aips" compatible to the i.MX53 dtsi file, but I don't think
that's worth it.

Sascha


-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 1/2] ARM: dts: i.MX25: add AIPS control registers Martin Kaiser <martin@kaiser.cx> - 2017-01-31 21:10 +0100
  Re: [PATCH 1/2] ARM: dts: i.MX25: add AIPS control registers Sascha Hauer <s.hauer@pengutronix.de> - 2017-02-01 08:10 +0100
  [PATCH v2] ARM: dts: i.MX25: add AIPS control registers Martin Kaiser <martin@kaiser.cx> - 2017-02-01 23:20 +0100
    Re: [PATCH v2] ARM: dts: i.MX25: add AIPS control registers Sascha Hauer <s.hauer@pengutronix.de> - 2017-02-02 08:20 +0100
  [PATCH v3] ARM: dts: i.MX25: add AIPS control registers Martin Kaiser <martin@kaiser.cx> - 2017-02-03 14:10 +0100
    Re: [PATCH v3] ARM: dts: i.MX25: add AIPS control registers Sascha Hauer <s.hauer@pengutronix.de> - 2017-02-03 18:30 +0100

csiph-web