mirror of
https://mirrors.bfsu.edu.cn/git/linux.git
synced 2024-12-15 15:04:27 +08:00
399426cadf
Usually, I don't like fixing coding style issues on non-staging drivers, as it could be a mess pretty easy, and could become like a snow ball. That's the case of recent changes on two changesets: they disalign some statements. Yet, a care a lot with cx88 driver, as it was the first driver I touched at the Kernel, and I've been maintaining it since 2005. So, several of the coding style issues were due to my code. Per Andrey's suggestion, I ran checkpatch.pl in strict mode, with fixed several other issues, did some function alinments, but broke other alinments. So, I had to manually apply another round of manual fixes to make sure that everything is ok, and to make checkpatch happy with this patch. With this patch, checkpatch.pl is now happy when called with: ./scripts/checkpatch.pl -f --max-line-length=998 --ignore PREFER_PR_LEVEL Also, the 80-cols violations that made sense were fixed. Checkpatch would be happier if we convert it to use dev_foo(), but this is a more complex change. NOTE: there are some places with msleep(1). As this driver was written at the time that the default was to sleep at least 10ms on such calls (e. g. CONFIG_HZ=100), I replaced those calls by usleep_range(10000, 20000), with should be safe to avoid breakages. Fixes: |
||
|---|---|---|
| .. | ||
| b2c2 | ||
| bt8xx | ||
| cobalt | ||
| cx18 | ||
| cx88 | ||
| cx23885 | ||
| cx25821 | ||
| ddbridge | ||
| dm1105 | ||
| dt3155 | ||
| ivtv | ||
| mantis | ||
| meye | ||
| netup_unidvb | ||
| ngene | ||
| pluto2 | ||
| pt1 | ||
| pt3 | ||
| saa7134 | ||
| saa7146 | ||
| saa7164 | ||
| smipcie | ||
| solo6x10 | ||
| sta2x11 | ||
| ttpci | ||
| tw68 | ||
| tw686x | ||
| tw5864 | ||
| zoran | ||
| Kconfig | ||
| Makefile | ||