Hi Sowmini, On Wed, Oct 22, 2014 at 9:39 AM, Sowmini Varadhan <sowmini.varadhan@xxxxxxxxxx> wrote: > On (10/22/14 09:35), Julian Calaby wrote: >> > void vio_port_up(struct vio_driver_state *vio) >> > { >> > - unsigned long flags; >> > + unsigned long flags = 0; >> >> Is gcc not smart enough to know that this variable isn't used before >> it's set? (I assume it isn't used elsewhere in this function) > > No, it's not used elsewhere in the function, and yes, I too was > surprised by the build warning, which is why I initialized it > as above. Ok, fair enough then. Thanks, -- Julian Calaby Email: julian.calaby@xxxxxxxxx Profile: http://www.google.com/profiles/julian.calaby/ -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html