Subject: Re: port-evbmips/31992: alchemy ICU is too Au1000 specific
To: Garrett D'Amore <garrett_damore@tadpole.com>
From: Simon Burge <simonb@wasabisystems.com>
List: netbsd-bugs
Date: 11/10/2005 15:34:39
"Garrett D'Amore" wrote:

> I've taken GNATs off the CC list, since this discussion doesn't need to 
> be in the bug, I think.  More below...

There's still a few gnats-style addresses above - maybe move this all to
port-mips?

> Simon Burge wrote:
> 
> >>>>The interrupt strings for the au_icu.c (and some of the identifiers,
> >>>>etc.) are really indicative of the specific Au1000 port, and do not
> >>>>reflect the general Alchemy family.
> >>>>
> >>>>This can actually lead to confusing output, e.g. from vmstat -e,
> >>>>because the interrupts are not wired the same on all parts.
> >>>>
> >>>>A better solution, IMO, is to generalize the strings to "irq XX".
> >>>>This provides less information than the current port does, but at
> >>>>least the information is not incorrect!  (And someone familiar with a
> >>>>given board can easily lookup up the IRQ mappings if needed.  And the
> >>>>actual drivers print out the IRQ they are on at attach time, so it
> >>>>should all be good. :-)
> >>>>
> >>>The way I'd previously worked around this was to have a table per
> >>>supported CPU type, similar to the way that aubus.c does things.  While
> >>>you can refer back to dmesg for interrupt number assignments, I think
> >>>I prefer to see the irq names in vmstat -e/-i without having to do the
> >>>cross-referencing elsewhere.
> >>>
> >>I've not looked at this too closely, but can any of the interrupt lines 
> >>other than those associated with specific busses (PCI, PCMCIA) be pulled 
> >>off the processor?  (In other words, is this mapping specific to a 
> >>board, or to a processor?  If it is processor specific, then this is 
> >>good enough, but if it can vary from board to board, then I'd prefer to 
> >>just leave it general.)
> >
> >The interrupt definitions there are CPU specific, not board specific.
> >For example, all the "gpio" interrupts are specific to hard-wired GPIO
> >pins, but what those pins are connected to on a specific board can vary.
> >  
> >
> Okay, and I see now that there are PSC mappings, etc. which can go to 
> different kinds of parts (PCMCIA, different memorys, I2C, AC'97, etc.)
> 
> So I'd request that the strings in this file only reflect the details of 
> the processor -- i.e. report an interrupt as being for "psc3" rather 
> than "AC97", because the mapping of AC97 to psc3 is a board specific detail.

Yah, that's it exactly.

> >>My assumption as far as vmstat -e related stuff was that (for this port 
> >>at least) most embedded developers will more or less "know" the 
> >>interrupt routing for their board, and frankly might not be too bothered 
> >>by having to go back and forth.  (Granted, for someone like you with a 
> >>lot of different ports that you maintain, this isn't such a big deal.)
> >>
> >>My other consideration is that with more and more parts coming out, it 
> >>becomes somewhat onerous to write up a new table for each new part, and 
> >>that folks will be tempted not to make it correct.  The current evidence 
> >>with the table being horribly wrong sort of serves as evidence of this.
> >>    
> >>
> >
> >We already need a table for the devices on the bus.  This is what I
> >meant but putting all processor-specific information in a single file,
> >then it's easy to update all info for a new processor.
> >  
> >
> Hmm.. there are a couple of ways to do this:
> 
> 1) detection at run time (what is currently done)
> 2) selection at compile time with separate files
> 3) selection at compile time via config option and #ifdef's in a single file
> 4) allow support for either #1 or #3 based on a config directive
> 
> I personally like option #4, which gives the best of all worlds, I 
> think.  Is there some reason, other than the size of the kernel, to 
> separate out the processor support for different alchemy parts?  I can't 
> think of any.  So option number 4 lets you still build a generic kernel, 
> but it also lets you build a leaner proc-specific config.

One reason is size - you probably don't want info about Au1000 devices
in your product if it's only using a Au1550.  Also, there's the
maintainability side.  Right now, you need to change a few files to add
support for a new CPU.  I'd like to centralise this.  Also, lots of
#ifdef's start to get ugly.  So I'm proposing:

5) allow support for either #1 or #2 based on a config directive

> >>I also don't have details on IRQ mappings for each of the 
> >>boards/processors.  (Granted, I could probably go search the CD-ROM I 
> >>got for specs for the other parts...)
> >>
> >
> >See above - we don't need to know board details at this level.
> >
> This is good news.  See my comments above about making sure that we only 
> report mappings that are known for the processor, and don't have some 
> board-specific mapping.
>
> >>>I've also been thinking of stripping out tables for each CPU in to their
> >>>own files, and having a way of just configuring support for a single CPU
> >>>type, instead of always being able to support CPUs (and devices) for all
> >>>Alchemy CPU types that exist.  While it's sometimes convenient (eg, I
> >>>don't have to use a different kernel when I turn a different board on
> >>>here and boot with tftp kernel), I think it makes more sense from an
> >>>embededed POV to just support the CPU and features you need.
> >>> 
> >>Yes, I am strongly in favor of this, and actually *need* it not just 
> >>per-processor but per *board*.  This is because there are some details, 
> >>like PCI interrupt routing, that are *board* specific and cannot be 
> >>determined programmatically.
> >
> >As long as you can identify which board you are running on at run-time,
> >you can do this.  Unfortunately Alchemy didn't have a board-id FPGA
> >register on the PB1000, but only boards after that.  For PCI, all
> >Alchemy eval boards have the ability to identify themselves.  As I
> >mention further down, you'll probably have your own kernel config
> >anyways.
>
> Hmmm... I've not groked around for the board-id FPGA.  This could be 
> very useful.  I'd be willing to use this to build a "generic" kernel for 
> the alchemy boards.  Do you have any more info about this FPGA, or 
> should I go RTFM?

Look on one of the CDs - for the DBAu1500 for example look for
DbAu1500/Docs/Zinfandel_Registers.pdf and look at the WHO_AM_I register.

> >>I already have some of this separated out in my Au1550 workspace, and 
> >>I'll be sending you diffs.
> >>
> >>Therefore, when the PCI diffs come to you, they will include different 
> >>kernel configs (and board-specific .c files) for each of the supported 
> >>platforms (which as of right now are DbAu1500 and DbAu1550, because 
> >>that's all I have to test with.)
> >
> >Certainly the PB1000 kernel config just supported the PB1500 and
> >DBAu1500 without any problems, although of course there's no PCI support
> >for the latter two.  However, for a simple "boots to multiuser with
> >ethernet and serial support", a single kernel config file is handy.
> >
> Yes, and indeed I've been using that with changes even on the Au1550.  
> And if I can glean the board ids, then I can even keep this around while 
> continuing to support PCI and other board-specific features (granted at 
> the cost of additional kernel bloat.)
> 
> >For PCI support, etc, we can still do some run-time detection, but
> >having separate kernel config files per board for space reasons also
> >makes sense.
> >  
> >
> I like having both options.

Yes.

> >Also, my original plan was to have PB1000 (which probably should be
> >renamed to something more generic) only support the Alchemy eval boards.
> >A full port to your board if it were contributed back to NetBSD would
> >almost certainly have a separate kernel config file to the Alchemy eval
> >boards (unless it was very close to one or more of these boards in
> >implementation).
> >  
> >
> Yes, the PB1000 config should be renamed to something like ALCHEMY or 
> DB1XXX or somesuch.

I was thinking something like ALCHEMY myself.  Not DB1XXX since the
kernel also works on the PB1xxx boards too.

> And my port will have a separate config file (and probably *not* be 
> contributed back to NetBSD, though the kernel bits of source code apart 
> from some low level hacks to the boot rom support will probably be 
> contributed back).  But right now I'm working with DBAu1500 and DBAu1550 
> boards until I get our own hardware.  (So I'm working on improving 
> support for those boards.)

Cool that you're working on it!  I know of another NetBSD developer who
was planning on a port to a small Au1500 box (meshcube?), and they'll be
able to leverage off this too.

Cheers,
Simon.
--
Simon Burge                            <simonb@wasabisystems.com>
NetBSD Support and Service:         http://www.wasabisystems.com/