On Tue, 21 Sep 2004, Wolfgang Denk wrote:
In message [ref] you wrote:
quoted
quoted
In message [ref] you wrote:
quoted
rather than get into more detailed discussion on microcode patches,
here's a (partial) patch that represents what i'd really, really,
^^^^^^^^^^^^^^^^^^^^^^^^
...
quoted
what i was showing was just an example, it didn't need to represent
*exactly* the set of choices that would be in the final menu. if the
Ummm. To me "partial patch" means that this is an excerpt of exactly
the patch you want to see applied. Please write "example" if this is
what you mean.
ok, sorry, what i meant was that it was a patch that would simply
demonstrate what the config menu would look like. anyone who was
interested could apply it, see how the config menu changed, then just
un-apply the patch after they got the idea. it wasn't submitted as an
official patch, so i apologize for the confusion.
so, regarding terminology, i should have called that an "example
patch"? gotcha. profuse apologies here.
It looked like a ready-to-sumbit patch, and you announced it as such.
yup, my mistake. i'll be more careful about my choice of words in the
future.
I have to admit that I don't understand what _exactly_ you suggest.
from list discussions some time back, i already gave up on the idea of
the configuration being able to detect the exact processor and taking
that into account. we've already established that, and i'm happy with
that.
all i'm suggesting now is a drop down choice menu of known patches
relevant to that architecture, that's all. in some cases, there would
be two choices, as in:
[ ] I2C/SPI for 850
[ ] I2C/SPI for 8xx (non-850)
with underlying help menus to clarify the situation. AFAICT, this
would just mirror the available choices in the denx 2.4 kernel tree.
i can't imagine that this list would get overly long. from what i can
see, the denx tree supports the following list of patches:
USB SOF
I2C/SPI for 850
I2C/SPI for 8xx (non-850)
I2C/SPI/SMC1 for 850
I2C/SPI/SMC1 for 8xx (non-850)
and, unless i'm reading this badly, that's the entire list. all of
five available patches. i don't think that's overly long, and it
would be fairly simple to implement.
so ... why is this such an objectionable thing?
rday