Message ID | 1511281705-30849-1-git-send-email-narmstrong@baylibre.com |
---|---|
Headers | show |
Series | ARM: Add Variscite DART-MX6 SoM and Carrier-board support | expand |
On Tue, Nov 21, 2017 at 2:28 PM, Neil Armstrong <narmstrong@baylibre.com> wrote: > + gpio-keys { > + compatible = "gpio-keys"; > + #address-cells = <1>; > + #size-cells = <0>; > + autorepeat; > + > + button@0 { If you build this with W=1 it will give you warnings about using a unit address without its corresponding reg field. You could just drop the @x. > + gpios = <&gpio4 26 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_BACK>; > + label = "Key Back"; > + linux,input-type = <1>; > + debounce-interval = <100>; > + gpio-key,wakeup; > + }; > + > + button@1 { > + gpios = <&gpio5 11 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_HOME>; > + label = "Key Home"; > + linux,input-type = <1>; > + debounce-interval = <100>; > + gpio-key,wakeup; > + }; > + > + button@2 { > + gpios = <&gpio4 25 GPIO_ACTIVE_LOW>; > + linux,code = <KEY_MENU>; > + label = "Key Menu"; > + linux,input-type = <1>; > + debounce-interval = <100>; > + gpio-key,wakeup; > + }; > + }; > + panel1: lvds-panel { > + compatible = "sgd,gktw70sdae4se", "panel-lvds"; sgd,gktw70sdae4se is not defined anywhere. Documentation/devicetree/bindings/display/panel/panel-lvds.txt says" "- compatible: Shall contain "panel-lvds" in addition to a mandatory panel-specific compatible string defined in individual panel bindings. The "panel-lvds" value shall never be used on its own." > + backlight = <&backlight_lvds>; > + > + width-mm = <153>; > + height-mm = <86>; > + > + label = "gktw70sdae4se"; > + > + data-mapping = "jeida-18"; > + > + panel-timing { > + clock-frequency = <32000000>; > + hactive = <800>; > + vactive = <480>; > + hback-porch = <39>; > + hfront-porch = <39>; > + vback-porch = <29>; > + vfront-porch = <13>; > + hsync-len = <47>; > + vsync-len = <2>; > + }; > + > + port { > + panel_in: endpoint { > + remote-endpoint = <&lvds1_out>; > + }; > + }; > + }; > + > + reg_usb_h1_vbus: regulator-usbh1vbus { > + compatible = "regulator-fixed"; > + regulator-name = "usb_h1_vbus"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + gpio = <&gpio1 28 0>; It would be better to use: gpio = <&gpio1 28 GPIO_ACTIVE_HIGH>; > + enable-active-high; > + }; > + > + reg_usb_otg_vbus: regulator-usbotgvbus { > + compatible = "regulator-fixed"; > + regulator-name = "usb_otg_vbus"; > + regulator-min-microvolt = <5000000>; > + regulator-max-microvolt = <5000000>; > + gpio = <&gpio4 15 0>; Same here. > + enable-active-high; > + }; > + > + > +&fec { > + status = "okay"; > + phy-mode = "rgmii"; > + phy-reset-gpios = <&gpio1 25 0>; GPIO_ACTIVE_LOW please. > +}; > + > +&hdmi { > + ddc-i2c-bus = <&i2c1>; > + status = "okay"; > +}; > + > +&pcie { > + reset-gpio = <&gpio4 11 0>; GPIO_ACTIVE_LOW > + wake-up-gpio = <&gpio4 31 1>; This is not a valid property. > + disable-gpio = <&gpio5 5 0>; This is not a valid property.
On Tue, Nov 21, 2017 at 2:28 PM, Neil Armstrong <narmstrong@baylibre.com> wrote: > + reg_wl18xx_vmmc: regulator-wl18xx { > + compatible = "regulator-fixed"; > + regulator-name = "vwl1807"; > + regulator-min-microvolt = <1800000>; > + regulator-max-microvolt = <1800000>; > + gpio = <&gpio7 8 0>; GPIO_ACTIVE_HIGH please. > +&can2 { > + pinctrl-names = "default"; > + pinctrl-0 = <&pinctrl_flexcan2>; > + status = "disabled"; > +}; > + > +&cpu0 { > + arm-supply = <&sw1a_reg>; > + soc-supply = <&sw1c_reg>; > + operating-points = < > + /* kHz uV */ > + 1200000 1350000 > + 996000 1250000 > + 852000 1250000 > + 792000 1175000 > + 396000 1175000 > + >; > + fsl,soc-operating-points = < > + /* ARM kHz SOC-PU uV */ > + 1200000 1275000 > + 996000 1250000 > + 852000 1250000 > + 792000 1175000 > + 396000 1175000 > + >; Don't the standard operating points described at imx6q.dtsi work on this board? If not, please add a comment explaining why you need custom operating points here. > + pinctrl_flexcan1: flexcan1grp { > + fsl,pins = < > + MX6QDL_PAD_GPIO_7__FLEXCAN1_TX 0x80000000 > + MX6QDL_PAD_GPIO_8__FLEXCAN1_RX 0x80000000 Please avoid 0x80000000 and use the real IOMUX values instead. > + >; > + }; > + > + pinctrl_flexcan2: flexcan2grp { > + fsl,pins = < > + MX6QDL_PAD_KEY_COL4__FLEXCAN2_TX 0x80000000 > + MX6QDL_PAD_KEY_ROW4__FLEXCAN2_RX 0x80000000 Please avoid 0x80000000 and use the real IOMUX values instead.
On Tue, Nov 21, 2017 at 2:28 PM, Neil Armstrong <narmstrong@baylibre.com> wrote: > Add missing pinmux entry for OSC32K_32K_OUT on pad ENET_RXD0 used by the > Variscite DART-MX6 SoM. > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com> Reviewed-by: Fabio Estevam <fabio.estevam@nxp.com>
Hi Fabio, On 21/11/2017 17:54, Fabio Estevam wrote: > On Tue, Nov 21, 2017 at 2:28 PM, Neil Armstrong <narmstrong@baylibre.com> wrote: > >> + reg_wl18xx_vmmc: regulator-wl18xx { >> + compatible = "regulator-fixed"; >> + regulator-name = "vwl1807"; >> + regulator-min-microvolt = <1800000>; >> + regulator-max-microvolt = <1800000>; >> + gpio = <&gpio7 8 0>; > > GPIO_ACTIVE_HIGH please. OK > >> +&can2 { >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_flexcan2>; >> + status = "disabled"; >> +}; >> + >> +&cpu0 { >> + arm-supply = <&sw1a_reg>; >> + soc-supply = <&sw1c_reg>; >> + operating-points = < >> + /* kHz uV */ >> + 1200000 1350000 >> + 996000 1250000 >> + 852000 1250000 >> + 792000 1175000 >> + 396000 1175000 >> + >; >> + fsl,soc-operating-points = < >> + /* ARM kHz SOC-PU uV */ >> + 1200000 1275000 >> + 996000 1250000 >> + 852000 1250000 >> + 792000 1175000 >> + 396000 1175000 >> + >; > > > Don't the standard operating points described at imx6q.dtsi work on this board? > > If not, please add a comment explaining why you need custom operating > points here. It should, I will remove them, anyway it will conflict with the ones in imx6dl.dtsi. > >> + pinctrl_flexcan1: flexcan1grp { >> + fsl,pins = < >> + MX6QDL_PAD_GPIO_7__FLEXCAN1_TX 0x80000000 >> + MX6QDL_PAD_GPIO_8__FLEXCAN1_RX 0x80000000 > > Please avoid 0x80000000 and use the real IOMUX values instead. Ok, will replace with 0x1b0b0 > >> + >; >> + }; >> + >> + pinctrl_flexcan2: flexcan2grp { >> + fsl,pins = < >> + MX6QDL_PAD_KEY_COL4__FLEXCAN2_TX 0x80000000 >> + MX6QDL_PAD_KEY_ROW4__FLEXCAN2_RX 0x80000000 > > Please avoid 0x80000000 and use the real IOMUX values instead. > Same here. Thanks, Neil
Hi Fabio, On 21/11/2017 17:45, Fabio Estevam wrote: > On Tue, Nov 21, 2017 at 2:28 PM, Neil Armstrong <narmstrong@baylibre.com> wrote: > >> + gpio-keys { >> + compatible = "gpio-keys"; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + autorepeat; >> + >> + button@0 { > > If you build this with W=1 it will give you warnings about using a > unit address without its corresponding reg field. > > You could just drop the @x. Ok > >> + gpios = <&gpio4 26 GPIO_ACTIVE_LOW>; >> + linux,code = <KEY_BACK>; >> + label = "Key Back"; >> + linux,input-type = <1>; >> + debounce-interval = <100>; >> + gpio-key,wakeup; >> + }; >> + >> + button@1 { >> + gpios = <&gpio5 11 GPIO_ACTIVE_LOW>; >> + linux,code = <KEY_HOME>; >> + label = "Key Home"; >> + linux,input-type = <1>; >> + debounce-interval = <100>; >> + gpio-key,wakeup; >> + }; >> + >> + button@2 { >> + gpios = <&gpio4 25 GPIO_ACTIVE_LOW>; >> + linux,code = <KEY_MENU>; >> + label = "Key Menu"; >> + linux,input-type = <1>; >> + debounce-interval = <100>; >> + gpio-key,wakeup; >> + }; >> + }; > >> + panel1: lvds-panel { >> + compatible = "sgd,gktw70sdae4se", "panel-lvds"; > > sgd,gktw70sdae4se is not defined anywhere. > > Documentation/devicetree/bindings/display/panel/panel-lvds.txt says" > > "- compatible: Shall contain "panel-lvds" in addition to a mandatory > panel-specific compatible string defined in individual panel bindings. The > "panel-lvds" value shall never be used on its own." > Ok add vendor prefix and bindings. >> + backlight = <&backlight_lvds>; >> + >> + width-mm = <153>; >> + height-mm = <86>; >> + >> + label = "gktw70sdae4se"; >> + >> + data-mapping = "jeida-18"; >> + >> + panel-timing { >> + clock-frequency = <32000000>; >> + hactive = <800>; >> + vactive = <480>; >> + hback-porch = <39>; >> + hfront-porch = <39>; >> + vback-porch = <29>; >> + vfront-porch = <13>; >> + hsync-len = <47>; >> + vsync-len = <2>; >> + }; >> + >> + port { >> + panel_in: endpoint { >> + remote-endpoint = <&lvds1_out>; >> + }; >> + }; >> + }; >> + >> + reg_usb_h1_vbus: regulator-usbh1vbus { >> + compatible = "regulator-fixed"; >> + regulator-name = "usb_h1_vbus"; >> + regulator-min-microvolt = <5000000>; >> + regulator-max-microvolt = <5000000>; >> + gpio = <&gpio1 28 0>; > > It would be better to use: > gpio = <&gpio1 28 GPIO_ACTIVE_HIGH>; Ok > > >> + enable-active-high; >> + }; >> + >> + reg_usb_otg_vbus: regulator-usbotgvbus { >> + compatible = "regulator-fixed"; >> + regulator-name = "usb_otg_vbus"; >> + regulator-min-microvolt = <5000000>; >> + regulator-max-microvolt = <5000000>; >> + gpio = <&gpio4 15 0>; > > Same here. Ok > >> + enable-active-high; >> + }; >> + > >> + >> +&fec { >> + status = "okay"; >> + phy-mode = "rgmii"; >> + phy-reset-gpios = <&gpio1 25 0>; > > GPIO_ACTIVE_LOW please. Ok > > >> +}; >> + >> +&hdmi { >> + ddc-i2c-bus = <&i2c1>; >> + status = "okay"; >> +}; >> + >> +&pcie { >> + reset-gpio = <&gpio4 11 0>; > > GPIO_ACTIVE_LOW > >> + wake-up-gpio = <&gpio4 31 1>; > > This is not a valid property. > >> + disable-gpio = <&gpio5 5 0>; > > This is not a valid property. > I will drop pcie for now until I figure out the requirements for these 2 gpios. Thanks, Neil