On Wed, 2013-12-11 at 17:36 +0100, Tomas Henzl wrote: > On 12/10/2013 06:01 AM, Viswas G wrote: > > pm80xx_get_gsm_dump() was returning "1" in error case > > instead of negative error value. > > > > Signed-off-by: Viswas G <Viswas.G@xxxxxxxx> > > Reviewed-by: Jack Wang <jinpu.wang@xxxxxxxxxxxxxxxx> > > Acked-by: TomasHenzl <thenzl@xxxxxxxxxx> > > Hi Viswas, > > I've ack-ed a different patch before, so I don't think you should add that acked-by line. > The V2 patch is fine for me too, so again - > Acked-by: Tomas Henzl <thenzl@xxxxxxxxxx> Actually, it can't be acked-by; that's for maintainers agreeing to patches not via them ... I assume it should be reviewed by? James -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html