Re: [PATCH 2.4] gcc-3.4 more fixes

From: Jeff Garzik
Date: Mon Aug 23 2004 - 17:27:33 EST


Marcelo Tosatti wrote:
On Mon, Aug 23, 2004 at 05:13:26PM +0300, O.Sezer wrote:

Marcelo Tosatti wrote:

On Mon, Aug 23, 2004 at 04:41:46PM +0300, O.Sezer wrote:


Ozkan,

This are just warning fixes right?

I dont like this patches, that is, I'm not confident about them.

Let the warnings be.

For gcc-3.4 they're warnings. For gcc-3.5 they'll cause compiler
failures (that's what mikpe says on cset-1.1490, too)

As a side note, almost all of them are in 2.6 anyway (can't
honestly remember which aren't)


Have you nocited the deadly mistake you made I showed with the grep?


Oopss :/ Than 2.6 has the same deadly thing. I'm too trusting I
guess.. The correct thing should be to change "if (!(PRIV(dev) ="
into "if (!(dev->phy_data =", right?


I think so yes. A network driver expert can confirm this for us.


Not enough context is quoted for me to decipher what this refers to :(

URL?

Jeff


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/