On Wed, Aug 31, 2005 at 03:19:37PM -0600, Bjorn Helgaas wrote:
> On Wednesday 31 August 2005 2:10 pm, Tom Rini wrote:
> > On Wed, Aug 31, 2005 at 01:38:52PM -0600, Bjorn Helgaas wrote:
> > > On Monday 29 August 2005 10:09 am, Tom Rini wrote:
> > I've tried intentionally to not mention 'ttyS' anywhere (exposed to the
> > user) because it's really not 'ttySN' but it is the port registered to
> > us.
>
> So kgdb's port N is different from ttySN? That sounds really
> confusing. And KGDB_SIMPLE_SERIAL does mention "ttyS".
It's not intentionally different, and really only might be different in
the we have ttySX case, but ttySX isn't registered to KGDB case.
> > There's really two cases we have to deal with. The first case is a
> > known at compile time or can be registered at boot-time easily port (ie
> > dumb old PC or ARM boards). The second case is "serial port over
> > there". Perhaps we should change the kgdb8250 arg to be an override of
> > the default port, so:
> > kgdb8250={io,mmio},<irq>,<token>,<baud rate>
>
> That makes sense. But I'd make it {io,mmio},<token>,<baud>,<irq>
> so it's more like the existing "console=uart" argument.
ok.
> > > > + printk(KERN_ERR "kgdb8250: argument error, usage: "
> > > > + "kgdb8250=<port number>,<baud rate>");
> > > > +#ifdef CONFIG_IA64
> > > > + printk(",<irq>,<iomem base>");
> > > > +#endif
> > >
> > > This isn't ia64-specific.
> >
> > It is and it isn't. Since no one's tried a PCI card uart for KGDB nor
> > had a case where we have to pass in the mmio addr except on ia64, it is
> > ia64-specific.
>
> Maybe it's only been *tested* on ia64, but I don't think that's a
> reason to make it compiled only on ia64.
It's only been needed on ia64. But it's moot since I've reworked things
for kgdb8250= is always a complete override.
> Actually, I think KGDB_SIMPLE_SERIAL, KGDB_*BAUD, KGDB_PORT_*,
> KGDB_PORT, and KGDB_IRQ are overkill. Could they all be nuked
> in favor of a KGDB_8250_DEVICE that could be set to things like
> "ttyS0,115200" or "io,0x3f8,115200,49"?
Hmm. I'll give that a shot momentarily... That sounds like a good idea
'tho..
--
Tom Rini
http://gate.crashing.org/~trini/
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
| |
 |