Thread (12 messages) 12 messages, 4 authors, 2016-11-21

Re: serial: imx: regression triggered by newly introduced DSR irq handling

From: Fabio Estevam <festevam@gmail.com>
Date: 2016-08-10 20:54:32
Also in: linux-arm-kernel

Hi Christoph,

On Fri, Aug 5, 2016 at 9:03 AM, Christoph Fritz
[off-list ref] wrote:
SION is necessary for pinconfig MX6SX_PAD_ENET1_TX_CLK__ENET1_REF_CLK1
if bit 17 in IOMUXC_GPR_GPR1 is set and bit 13 not.

So that ref_enetpll1 provides a clock not only for the external PHY but
also for the internal controller.

Using SION is a quirk here, because the silicon doesn't feed the clock
back to the internal controller automatically.

On the other hand, NXP could argue: You need to add a wire on your PCB
between ENET1_REF_CLK1 and ENET1_TX_CLK -- but referring to the
datasheet ENET1_TX_CLK isn't used in RMII config...

So if Fabio or Shawn agrees with my assumption above, I'll add something
like 27e16501052e5341934d3 "serial: imx: implement DSR irq
handling for DTE mode".

Okay?
Could you please post your suggestion as a patch or RFC so that we can
understand what your proposal is?

Thanks
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help