Paul Fulghum <[email protected]> writes:
> We were in a perpetual state of:
>
> 1. supply patch
> 2. get criticism from new person just joining thread
> 3. change patch to address criticism
> 4. goto #1
Right, the description fits. OTOH I recall the criticism had a fair
amount of merit. That's how the things work here. Been there many
times BTW.
>> Nevertheless a fix outlined above would be acceptable, wouldn't it?
>
> It breaks things by forcing the customer to include code
> that is not required. That is an issue for embedded systems.
Nope, the fix which didn't change Kconfig and just assumed HDLC=m and
SYNCLINK*=y => HDLC is not available. I.e., the one with
#if defined(CONFIG_HDLC_MODULE) &&/|| defined(CONFIG_SYNCLINK*something)
#define CONFIG_HDLC something
changed into
#if defined(CONFIG_HDLC_MODULE) &&/|| defined(CONFIG_SYNCLINK*something)
#define USE_HDLC something
^^^^^^^^
> But since we seem stuck in a state where real fixes
> are not allowed, and this breakage is constantly reintroduced,
It may look like that sometimes but it's not real. Anyway I think
everyone would benefit from the correct fix and the issue wouldn't
come again. Having looked at it I'd fix it myself but I'm pretty
sure you still have the old patch (which changes CONFIG_ macros
outside Kconfig, I mean in .c files) and it could be trivially
modified then applied (and perhaps tested with real hardware if
needed).
--
Krzysztof Halasa
-
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]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
