From mboxrd@z Thu Jan 1 00:00:00 1970 From: nsekhar@ti.com (Sekhar Nori) Date: Thu, 25 Aug 2016 19:12:16 +0530 Subject: [PATCH] ARM: dts: da850: Add missing pin muxing for the UARTs In-Reply-To: <20160824082934.GA20246@gobelin> References: <20160804110656.25318-1-kbeldan@baylibre.com> <20160804210045.GD15013@gobelin> <20160824082934.GA20246@gobelin> Message-ID: <354081bd-653d-ec98-f18e-eb0b1b1ee7e3@ti.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wednesday 24 August 2016 02:08 PM, Karl Beldan wrote: > On Tue, Aug 23, 2016 at 04:46:03PM +0530, Sekhar Nori wrote: >> On Tuesday 23 August 2016 04:39 PM, Sekhar Nori wrote: >>> On Friday 05 August 2016 04:30 AM, Kevin Hilman wrote: >>>> Karl Beldan writes: >>>> >>>>> On Thu, Aug 04, 2016 at 12:20:27PM -0700, Kevin Hilman wrote: >>>>>> Karl Beldan writes: >>>>>> >>>>>>> This adds 2 pinctrl groups (rtscts, rxtx) for each of the 3 UARTs. >>>>>>> >>>>>>> Signed-off-by: Karl Beldan >>>>>> >>>>>> Should da850-evm be updated to use the serial2_rxtx_pins also? >>>>>> >>>>> I could not find the EVM schematics on the net and I only have an LCDK, >>>>> but according to the code it should, however I can't tell whether flow >>>>> control pins are used. >>>> >>>> Ok, let's just leave it for now, since it's working fine. Sekhar can >>>> fix that up if he can dig up the schematics. >>> >>> Looks like the flow control pins are being used for McASP also on the >>> EVM. So lets leave the EVM as-is. >> >> Rather, the EVM dts file should be updated to use serial2_rxtx_pins like >> the LCDK. Right now it seems to be relying on bootloader to serial2 >> setup pimux correctly. I can make a patch to fix that. Or if you can do >> it, that will be great too. >> > > Indeed ATM the EVM relies on the bootloader to setup the pin muxing. > > I just checked the uart pins routing of the EVM, the dts: > - should reclaim serial2_rxtx_pins and serial2_rtscts_pins Can you please clarify what you mean by dts should "reclaim". You mean move the pins from da850.dtsi to da850-evm.dts or something else? Thanks, Sekhar