Re: [PATCH v5 18/18] usb:hsotg:samsung: err_irq: support for probe function

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sergei,

>
> > -
> >  err_supplies:
> >   	s3c_hsotg_phy_disable(hsotg);
> > -
> >   	regulator_bulk_free(ARRAY_SIZE(hsotg->supplies),
> > hsotg->supplies); -
> > -	clk_disable_unprepare(hsotg->clk);
> > -	clk_put(hsotg->clk);
> > -
> > -err_regs:
> > + err_irq:
> > +	free_irq(hsotg->irq, hsotg);
> > + err_regs:
> 
>     Do not add spaces before labels. There's no spaces before
> existing labels. Have you run the patch thru scripts/checkpatch.pl?

Hmm, spaces have been added there.

Unfortunately the checkpatch.pl script hasn't complaint about this.

lukma@lmajewski:~/work/linux-usb(s3c-hsotg_devel_v5)$ cat
0001-usb-hsotg-samsung-err_irq-support-for-probe-function.patch
| ./scripts/checkpatch.pl - 

total: 0 errors, 0 warnings, 34 lines checked

Your patch has no obvious style problems and is ready for submission.

Ok, I will remove those paces and resend this patch.

Thanks for spotting :-)


-- 
Best regards,

Lukasz Majewski

Samsung Poland R&D Center
Platform Group
--
To unsubscribe from this list: send the line "unsubscribe linux-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux