From mboxrd@z Thu Jan  1 00:00:00 1970
From: Frank Wunderlich <frank-w@public-files.de>
Date: Wed, 19 Aug 2020 12:58:24 +0200
Subject: Aw: Re: [PATCH v2 09/11] arm: dts: mt7622: add sata- and asm_sel nodes
In-Reply-To: <1597827268.23067.5.camel@mhfsdcap03>
References: <20200819080225.4267-1-linux@fw-web.de>
 <20200819080225.4267-10-linux@fw-web.de>
 <1597827268.23067.5.camel@mhfsdcap03>
Message-ID: <trinity-7d33a3e8-3e7f-42c3-a191-c98cc5c1eaec-1597834704540@3c-app-gmx-bap53>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi

> Gesendet: Mittwoch, 19. August 2020 um 10:54 Uhr
> Von: "Chunfeng Yun" <chunfeng.yun@mediatek.com>
> > +	sata_phy: sata-phy at 1a243000 {
> > +		compatible = "mediatek,generic-tphy-v1";
> > +		#address-cells = <1>;
> > +		#size-cells = <1>;
> > +		ranges;
> we can use ranges with parameters if you send next version, otherwise no
> change is also fine to me.
>
> ranges=<0 0x1a243000 0x100>;

i took the dts-part from linux where ranges is also empty

https://elixir.bootlin.com/linux/latest/source/arch/arm64/boot/dts/mediatek/mt7622.dtsi#L880

why should ranges have a value when i recently removed the reg (after making it optional)?

> For phy part,
> Reviewed-by: Chunfeng Yun <chunfeng.yun@mediatek.com>

any issues in sata-node/gpio-hog?

regards Frank