Aw: Re: [PATCH v2 09/11] arm: dts: mt7622: add sata- and asm_sel nodes

Chunfeng Yun chunfeng.yun at mediatek.com
Thu Aug 20 03:24:52 CEST 2020


On Wed, 2020-08-19 at 12:58 +0200, Frank Wunderlich wrote:
> Hi
> 
> > Gesendet: Mittwoch, 19. August 2020 um 10:54 Uhr
> > Von: "Chunfeng Yun" <chunfeng.yun at 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
> 
I think it follows the early dt-binding doc of tphy, I modify it later.

> why should ranges have a value when i recently removed the reg (after making it optional)?
Both are ok, but prefer to use translation way:)

Without shared registers, it's simple and clear to use address
translation from parent-bus-address to child-bus-address. 
e.g. subnode: usb-phy at 0, usb-phy at 1000, they tell us the offset at the
same time.

> 
> > For phy part,
> > Reviewed-by: Chunfeng Yun <chunfeng.yun at mediatek.com>
> 
> any issues in sata-node/gpio-hog?
Also looks good to me

> 
> regards Frank



More information about the U-Boot mailing list