Re: [PATCH] Revert "tty: serial: imx.c: Reset UART before activating interrupts"

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

 



Dear Fabio,

On Tue, 28 Jul 2015 09:55:28 -0300
Fabio Estevam <festevam@xxxxxxxxx> wrote:

> On Tue, Jul 28, 2015 at 3:07 AM, David Jander <david@xxxxxxxxxxx> wrote:
> 
> > IMHO Spewing out a few corrupt characters during startup (which curiously
> > also appear sometimes due to other reasons), is a lot less of a problem
> > that a hanging system!
> 
> A few corrupted characters?

In my case 1 or 2....

> Look at this:
> 
> 0øgUª%à˜hyº”VÑáE%%ªEàÕþ0p~ï
> Ô¨ul¸j.#òm8žE7ǐ`í]òm‘n›0$h?>òàcÝ1Fàpýxð¸5D?â0$¸u}OGhý}ü¢0$8E¨='7Ô"óbíOðÁb¡·Ñ¨g5]0$¨êý˜h)Aý
> q4tè|hs¹ÑÕ<êp¿hs¹4776?hóY~kiÓ~˜Áâ£0ªuG%*WxùÍâþÐè5F'¼E_0x5g'¨ghC¬0$ºuT¸?¼U4wAˆw%•â𸖘Ô
> qèÁnyäp™0$èG?5U4(=8T‰˜h`m0 {b䩐hAx(zcA¨˜háŸjc_bu‰0$('ÑC`cA
> aQ"á[p¨[baM1Ð(ycbdo`r-hsÁük˜"cAZð8áéjba5ò©0(àcEdgÈC¼žJ
> 5LÑùð8W5ß0ú5¸gǘh~pÁ5]hs<xù6]áAb¹C7
> ]`s2ñ(ǐ`Ýxë³øW˜hY|¸(gW`]|Æà?Táb)Ì5% MwG˜`M|>
> 5º˜h~£bptb¯Ï<«'e'o÷è%
> 7õ**˜øÔÕu7E'U%-uwèrœ%/77*0d*xí]QU }]|š`ˆ<E0dºM6
> ñpˆWè}agð-+(aç0$ºmf?`ˆ' ø<Ð8…6%
> Á󔜸wT˜xnñU%8W`:Mí\Wºw}°=Ôâ`yòzç»p7]6B3¼Eœ_0D·ep…ÕxÕ˜4'%M¼x3Bhmà4Th}ArOyM4hAâñÁæ-Õc*c¶0d4U(*à0,£(cÕ>Tj¶0lø7ðÔifŽ*wÕ0lx(íx¨Ÿqx\u÷õ´Ò0?¸iI4ð47U¨5
> 'gt–Ž?9xÃ|`c~peu}=÷ègÐœôùx
> e'Eu%T‰˜hÃ|b~>èy`øð¦³'Ô?IãdhÜbïÙümQ|ÔÕ0lÝèkâüŸ:D–<ôyE¸'eu÷0d(e×0
> uWeÇ0l*=8ט˜-'%5=0,¨-'%5=à0,¨'gE5|ð!}ª'7ט<Eºg<xD¢5'7×0lªUtùðÀ/Qܘ¼\/eh6×0lè?=r}6`¾”ÔM
> 7=T?záù%7OwG0$8oÑ:}菴ww]0$YümOè›òcýä9oEQŸBîhvyÝ
> oI>òA1pf~ueuwôêUÕuEÅ0D"üwz|`/u¸ªE8w‚14G%Wu”èŘüÔ=u555(]üð¼¨=%”Шê.ilxiùTÕ˜ôÐô7ð0,h6Guo5אüè-/ÝÐÓ
> ?Ot5אüè-'7¨E-%?ÐS0òè '(?G==85÷ôPB;\p4w3`lhê®Nut¸? thc·0
> "üe%U¸0,]zûÏlîhL¨*üǘøèGD’xc]¼ê?ÕYñ0l¸e}
> w0,êWuuåT'e'|áWìy¨-ýh$±mºD%=ǐüTCcÓháeuew=oߘøeÔ/gE57ggՐôTCcCüå˜ô`þ}m1T
> 5èl¹Sý`hiäWwâiï(àml`í>`m¬WkIly1%]~ n+oü0%“ðI¨u%pÌê.+
> œe]=÷0M*gW=G''D%}ՐŠ%75-5uPð,]˜Äbmüû}lênþýñ0m]n=Ew=T”-]0e¨w4Éu•©jM÷0%]o=E¨}_hÇ|o‰=Eb7'üp¸7*Íh“
> ˜WüáÏ|É'}?øÆg¸o?57¨dT'dWhó;þ0-ªhn+oü˜9Lᘼt,î0%¨éã45¸¨èŸÔ‘W4õ*p[>5x|`Ærccrëyøìí|î㓐¹Tjãïp'7}°˜TÔø$e¸y>öýhfmí1Àjcbâj{}ôÕ‰¨u”8v©w=M˜<TѸ÷˜¼
> A3ù
> Ã<RC¦˜7=¡÷˜=Lº'm:w\u==…üàóê5Ý0j%5%טjúsÃz)Iæ0-¨D”GweU$ߘ=V•}8eŘ=VPœ³xiü˜=-%5ô×0EhòoÉü÷C}°`|¾S|¸;P[áY7¦púuU8-bEªb‹*ú˜Gmcrgˆ?%úguG%ՐÄVbã]Bíþyw=Gàé_þý0-Íà}Y>òýà&&l©u÷h.©u÷%Íp%'5'|ð½aG•’ÅÖšÉnîh1VB7tê'|áÅ»7=G7É0-êM%''ü0m]rsÍïðò›W_zç0%81Qßyk¼˜ˆ78}K„ùu
> 'EÝð(%']À~÷ø'·/ß7אWk>EwE%7}osòm‹uâsz¯8%QC¸˜Cýéjâkfâ|\טGkÝìc‹G5(Eg5ü˜SjýÍbPþ˜WjýÍb5÷ðküícõgVü˜ˆ4oO*ø55Ö˜Š>7*:U7ÜטŠuwðí}£?EF5ü0¨ohêmÁü}=àsKüíCõÅyŸ0%j+ý<Û*ø55אWýèjãKfã_|ÜאWküícuœ(Eg5üSküìcËԐWküìcY5'G˜GjýÍb7/eǘhl
> ã%jjÄVHvíg7PKh›=h‡³%g˜hÔ8‰}ôPá5x5'g=7Ý"˜hlâ+øAœ7M==ðp0-8M˜hnšòmO~ƒÁWlíÈgEuwð\ÕÆ~»pD5èא077T-7$:emq³4=èטhlcÃno“„Ì_0%]bãÏøŠœaû0—42c7å˜D'gGÔå-Si_ò0%"üwzç(7pF¯˜h:[h?¸0$x|³5
> ]˜h:[h?¸0,3ô—¨0Dxæüã]þ㉘xtG¸=è87ý0$*'7×0$¸LgUߘh*púg¨oüžA`cAü0$
> hí=ÐQ5Dà0DÀªg[    1.121820] EXT2-fs (mmcblk0p2): error: couldn't mount
> because of unsupported optional features (240) [    1.133338] EXT4-fs
> (mmcblk0p2): couldn't mount as ext3 due to feature incompatibilities
> [    1.171330] EXT4-fs (mmcblk0p2): mounted filesystem with ordered
> data mode. Opts: (null)
> [    1.181562] VFS: Mounted root (ext4 filesystem) readonly on device 179:2.
> [    1.206286] devtmpfs: mounted
> [    1.209851] Freeing unused kernel memory: 428K (80a3d000 - 80aa8000)
> [    1.262378] mmc2: host does not support reading read-only switch,
> assuming write-enable
> [    1.276443] mmc2: new high speed SDHC card at address 0001
> [    1.282878] mmcblk1: mmc2:0001 00000 7.60 GiB
> [    1.289501]  mmcblk1: p1 p2
> 
> Do you think this is acceptable?

Yuck! No, of course not. Why is this?
I don't have this issue, and I'd really like to understand why you have it?
Can you explain what happens to the UART on your board exactly after power-on
until this?

> You can't fix one problem by introducing other nasty one.

That is of course not the idea! I only don't really see how this can be
related. My patch does nothing more that what a simple power-on reset would
also do.

Best regards,

-- 
David Jander
Protonic Holland.
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux