RE: [PATCH 3/3] ARM: dts: am335x: add support for Moxa UC-2102 open platform

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

<snip>
 
> The 2102 is a superset of the 2101? If so, the 2102 should come first.
> If not, this should probably be 2 separate entries.
> 

Nope, they use different SoC models and PMICs. I'll resend them separately in next version.

Thanks for guidance.

> > +&tps {
> > +	vcc1-supply = <&vbat>;
> > +	vcc2-supply = <&vbat>;
> > +	vcc3-supply = <&vbat>;
> > +	vcc4-supply = <&vbat>;
> > +	vcc5-supply = <&vbat>;
> > +	vcc6-supply = <&vbat>;
> > +	vcc7-supply = <&vbat>;
> > +	vccio-supply = <&vbat>;
> > +
> > +	regulators {
> > +		vrtc_reg: regulator@0 {
> 
> Without a reg property, you shouldn't have unit addresses.

I would like to ask why shouldn't we keep unit address with "regulator-always-on;" reg property?

SZ

> 
> 
> > +			regulator-always-on;
> > +		};
> > +
> > +		vio_reg: regulator@1 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdd1_reg: regulator@2 {
> > +			/* VDD_MPU voltage limits 0.95V - 1.26V with +/-4% tolerance */
> > +			regulator-name = "vdd_mpu";
> > +			regulator-min-microvolt = <912500>;
> > +			regulator-max-microvolt = <1378000>;
> > +			regulator-boot-on;
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdd2_reg: regulator@3 {
> > +			/* VDD_CORE voltage limits 0.95V - 1.1V with +/-4% tolerance */
> > +			regulator-name = "vdd_core";
> > +			regulator-min-microvolt = <912500>;
> > +			regulator-max-microvolt = <1150000>;
> > +			regulator-boot-on;
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdd3_reg: regulator@4 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdig1_reg: regulator@5 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdig2_reg: regulator@6 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vpll_reg: regulator@7 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vdac_reg: regulator@8 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vaux1_reg: regulator@9 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vaux2_reg: regulator@10 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vaux33_reg: regulator@11 {
> > +			regulator-always-on;
> > +		};
> > +
> > +		vmmc_reg: regulator@12 {
> > +			compatible = "regulator-fixed";
> > +			regulator-name = "vmmc_reg";
> > +			regulator-min-microvolt = <3300000>;
> > +			regulator-max-microvolt = <3300000>;
> > +			regulator-always-on;
> > +		};
> > +	};
> > +};
> > +
> > +&mac {
> > +	pinctrl-names = "default";
> > +	pinctrl-0 = <&cpsw_default>;
> > +	dual_emac = <1>;
> > +	status = "okay";
> > +};
> > +
> > +&cpsw_emac0 {
> > +	status = "okay";
> > +	phy_id = <&davinci_mdio>, <0x4>;
> > +	phy-mode = "rmii";
> > +	dual_emac_res_vlan = <1>;
> > +};
> > +
> > +&cpsw_emac1 {
> > +	status = "okay";
> > +	phy_id = <&davinci_mdio>, <0x5>;
> > +	phy-mode = "rmii";
> > +	dual_emac_res_vlan = <2>;
> > +};
> > +
> > +&mmc1 {
> > +	pinctrl-names = "default";
> > +	vmmc-supply = <&vmmcsd_fixed>;
> > +	bus-width = <4>;
> > +	pinctrl-0 = <&mmc0_pins_default>;
> > +	cd-gpios = <&gpio3 14 GPIO_ACTIVE_HIGH>;
> > +	wp-gpios = <&gpio3 18 GPIO_ACTIVE_HIGH>;
> > +	status = "okay";
> > +};
> > --
> > 2.19.0.rc1
> >





[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux