From mboxrd@z Thu Jan 1 00:00:00 1970 From: Romain Perier Date: Tue, 11 Jul 2017 16:30:30 +0200 Subject: [U-Boot] [PATCH] net: gmac_rockchip: Add support for pinctrl In-Reply-To: References: <20170703071312.19770-1-romain.perier@collabora.com> <55fd8342-a92c-f9a9-3ab3-36c34472c6ca@collabora.com> Message-ID: List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de Hello, Le 11/07/2017 à 15:43, Dr. Philipp Tomsich a écrit : >> On 10 Jul 2017, at 08:43, Romain Perier wrote: >> >> Le 07/07/2017 à 05:58, Simon Glass a écrit : >>> +Philipp >>> >>> Hi Romain, >>> >>> On 3 July 2017 at 01:13, Romain Perier wrote: >>>> Currently, selecting state simple is done by the device driver model, >>>> prior probing the driver. The problem is that it's done and called on >>>> the pinctrl node with "gmac" as the "periph" struct udevice *. So >>>> pinctrl-rk3288 is looking for an interrupt property that is not found, >>>> and then gmac_config is never called. >>>> >>>> This commits toggles the pinctrl on the right node from the probe >>>> function of the driver. >>> Is it possible to fix this while still using driver-model automatic pinctrl? >> This is what I have tried to do, without success. The purpose of this >> patch is also to discuss about the possible solutions we have. > From looking at pinctrl_rk3288.c, I wonder if there’s a problem with > pinctrl on the RK3288 outside of SPL: I see that the (full) set_state > is also implemented outside of SPL (#ifndef CONFIG_SPL_BUILD) > and not guarded by a CONFIG_IS_ENABLED(PINCTRL_FULL). So > something might go down the wrong path. > > Could start narrowing this down by enabling DEBUG in pinctrl_rk3288 > to see whether this is even the simple pinctrl that gets called? Yeah, the simple pinctrl is called at least for "hdmi" and the function for configuring hdmi is called internally. Romain