* Add UPF_FIXED_TYPE flag - eedacbf02df462d6c915d8d642b23fb450d485c7
@ 2009-11-23 16:53 Michal Simek
2009-11-24 9:07 ` Michal Simek
0 siblings, 1 reply; 2+ messages in thread
From: Michal Simek @ 2009-11-23 16:53 UTC (permalink / raw)
To: benh, dmitchell; +Cc: Grant Likely, John Williams, LKML
Hi Ben and David,
I have found one problem with input overrun. My problem is of course on
Microblaze with uart16550.
Here is the behavior - I just pressed left arrow.
~ # fdasfdakjsdhfkajsdhfkjadklasdjfhajksdhfkahsdfjkahsdkjfhjkattyS0: 3
input overrun(s)
[[DttyS0: 3 input overrun(s)
[[[[[DDDahsdk[[[[D[DDjfhjka[D [[[DttyS0: 6 input
overrun(s)jk[[D[D[[D[[
[[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[[[D[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[
D[[[[DDDahsdk[[[[D[DDjfhjka[D
-sh:
fdasfdakjsdhfkajsdhfkjadklasdjfhajk[[D[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[D[[[[DDDahsdk[[[[D[DDjfhjka[D:
not found
I bisected it and yours Subject patch caused my problems. It is really
old patch and I was surprised how old it is.
Here is your diff
diff --git a/drivers/serial/of_serial.c b/drivers/serial/of_serial.c
index 02406ba..54483cd 100644
--- a/drivers/serial/of_serial.c
+++ b/drivers/serial/of_serial.c
@@ -67,7 +67,7 @@ static int __devinit of_platform_serial_setup(struct
of_device *ofdev,
port->type = type;
port->uartclk = *clk;
port->flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF | UPF_IOREMAP
- | UPF_FIXED_PORT | UPF_FIXED_TYPE;
+ | UPF_FIXED_PORT;
port->dev = &ofdev->dev;
/* If current-speed was set, then try not to change it. */
if (spd)
If I trace port->type I am getting 3. (drivers/serial/8250.c:3089
serial8250_register_port func 3118 - serial8250_register_port)
Grant: Could you please print that value on ppc?
Can you give me any hints what could be wrong? I will continue on it
tomorrow.
Thanks,
Michal
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: Add UPF_FIXED_TYPE flag - eedacbf02df462d6c915d8d642b23fb450d485c7
2009-11-23 16:53 Add UPF_FIXED_TYPE flag - eedacbf02df462d6c915d8d642b23fb450d485c7 Michal Simek
@ 2009-11-24 9:07 ` Michal Simek
0 siblings, 0 replies; 2+ messages in thread
From: Michal Simek @ 2009-11-24 9:07 UTC (permalink / raw)
Cc: benh, dmitchell, Grant Likely, John Williams, LKML
Hi,
I found where the problem was. We have uart16550a compatible device not
only uart16550.
I will send patch for extension of compatible list.
Regards,
Michal
Michal Simek wrote:
> Hi Ben and David,
>
> I have found one problem with input overrun. My problem is of course on
> Microblaze with uart16550.
>
> Here is the behavior - I just pressed left arrow.
>
> ~ # fdasfdakjsdhfkajsdhfkjadklasdjfhajksdhfkahsdfjkahsdkjfhjkattyS0: 3
> input overrun(s)
> [[DttyS0: 3 input overrun(s)
> [[[[[DDDahsdk[[[[D[DDjfhjka[D [[[DttyS0: 6 input
> overrun(s)jk[[D[D[[D[[
> [[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[[[D[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[
>
> D[[[[DDDahsdk[[[[D[DDjfhjka[D
> -sh:
> fdasfdakjsdhfkajsdhfkjadklasdjfhajk[[D[[[D[Dsd[[Dhfkah[[[[DD[[DDsdfjk[[D[D[[D[[[[DDDahsdk[[[[D[DDjfhjka[D:
> not found
>
>
> I bisected it and yours Subject patch caused my problems. It is really
> old patch and I was surprised how old it is.
>
>
> Here is your diff
> diff --git a/drivers/serial/of_serial.c b/drivers/serial/of_serial.c
> index 02406ba..54483cd 100644
> --- a/drivers/serial/of_serial.c
> +++ b/drivers/serial/of_serial.c
> @@ -67,7 +67,7 @@ static int __devinit of_platform_serial_setup(struct
> of_device *ofdev,
> port->type = type;
> port->uartclk = *clk;
> port->flags = UPF_SHARE_IRQ | UPF_BOOT_AUTOCONF | UPF_IOREMAP
> - | UPF_FIXED_PORT | UPF_FIXED_TYPE;
> + | UPF_FIXED_PORT;
> port->dev = &ofdev->dev;
> /* If current-speed was set, then try not to change it. */
> if (spd)
>
>
> If I trace port->type I am getting 3. (drivers/serial/8250.c:3089
> serial8250_register_port func 3118 - serial8250_register_port)
> Grant: Could you please print that value on ppc?
>
> Can you give me any hints what could be wrong? I will continue on it
> tomorrow.
>
> Thanks,
> Michal
>
>
--
Michal Simek, Ing. (M.Eng)
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel 2.6 Microblaze Linux - http://www.monstr.eu/fdt/
Microblaze U-BOOT custodian
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-11-24 9:07 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-11-23 16:53 Add UPF_FIXED_TYPE flag - eedacbf02df462d6c915d8d642b23fb450d485c7 Michal Simek
2009-11-24 9:07 ` Michal Simek
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.