On Thu, 14 Jan 2010 17:55:55 +0100 Roel Kluin <roel.kluin@xxxxxxxxx> wrote: > These statements appear redundant. > > Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx> > --- > Is there any reason to assign this to itself? > > diff --git a/arch/sparc/kernel/central.c b/arch/sparc/kernel/central.c > index f3b5466..773caf4 100644 > --- a/arch/sparc/kernel/central.c > +++ b/arch/sparc/kernel/central.c > @@ -99,7 +99,6 @@ static int __devinit clock_board_probe(struct > of_device *op, > p->leds_resource.start = (unsigned long) > (p->clock_regs + CLOCK_CTRL); > - p->leds_resource.end = p->leds_resource.end; I have the feeling that the right fix is to have: p->leds_resource.end = p->leds_resource.start; hence describing a 1 byte region. M. -- I'm the slime oozin' out from your TV set... -- 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
