On Fri, May 3, 2013 at 3:37 AM, Grant Likely <grant.lik...@secretlab.ca> wrote:
>
> Here are the GPIO changes staged for v3.10. Please pull.
Grrr. These clearly have not been tested at all, and they break the
allmodconfig build.
> Jean Delvare (2):
> gpio: ucb1400: Can be built as a module
The above one is pure and utter garbage. That driver cannot be built
as a module, and clearly nobody ever even tested it.
You have:
void __init ucb1400_gpio_set_data(struct ucb1400_gpio_data *data)
(which, btw, seems entirely unused), but then in the header file
(remember: this is a totally unused function as far as I can tell) we
have
#ifdef CONFIG_GPIO_UCB1400
void __init ucb1400_gpio_set_data(struct ucb1400_gpio_data *data);
#else
static inline void ucb1400_gpio_set_data(struct ucb1400_gpio_data *data) {}
#endif
which means that in the modular case, you get
drivers/gpio/gpio-ucb1400.c:106:13: error: redefinition of
‘ucb1400_gpio_set_data’
again, I see absolutely no way that (a) this piece of crap has ever
been tested and (b) how /why that useless piece of shit even gets
used.
I'm upset. How the f*ck did this get into your tree in the first
place, and after it got into the tree, WHY THE HELL DID YOU SEND THIS
CRAP TO ME?
The one-liner commit that enables this piece of shit is signed off by
two people, has another person with a "Reviewed-by:" and there is no
way in hell it was ever tested or anybody has ever looked at the
driver in question. WTF?
I'm going out on a limb here, and guessing that it wasn't in linux-next either.
Piece of unadulterated shit.
Quite frankly, I'm not taking any GPIO changes from you this merge
window. I only noticed this after having done two other merges on top
of it, so now I'll have to go back and undo all of this, because quite
frankly, I'm upset enough that I don't want to have any remains of
this whole experience in my tree.
Linus
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
