diff options
| author | Ben Dooks <ben.dooks@codethink.co.uk> | 2017-07-26 05:51:10 -0400 |
|---|---|---|
| committer | Richard Weinberger <richard@nod.at> | 2017-09-13 16:05:30 -0400 |
| commit | a51b7ccf3bf57956c64076987a9ad1152e896e79 (patch) | |
| tree | 406b3222b6915f211d946137cb4f85e4de9c99bd | |
| parent | 278f31a70d2e40aa31bf465795bfce2ca8bb01b6 (diff) | |
ubi: pr_err() strings should end with newlines
In ubi_attach_mtd_dev() the pr_err() calls should have their
messgaes terminated with a new-line to avoid other messages
being concatenated onto the end.
Signed-off-by: Ben Dooks <ben.dooks@codethink.co.uk>
Signed-off-by: Richard Weinberger <richard@nod.at>
| -rw-r--r-- | drivers/mtd/ubi/build.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/mtd/ubi/build.c b/drivers/mtd/ubi/build.c index f9c576b8463e..842550b5712a 100644 --- a/drivers/mtd/ubi/build.c +++ b/drivers/mtd/ubi/build.c | |||
| @@ -825,7 +825,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num, | |||
| 825 | for (i = 0; i < UBI_MAX_DEVICES; i++) { | 825 | for (i = 0; i < UBI_MAX_DEVICES; i++) { |
| 826 | ubi = ubi_devices[i]; | 826 | ubi = ubi_devices[i]; |
| 827 | if (ubi && mtd->index == ubi->mtd->index) { | 827 | if (ubi && mtd->index == ubi->mtd->index) { |
| 828 | pr_err("ubi: mtd%d is already attached to ubi%d", | 828 | pr_err("ubi: mtd%d is already attached to ubi%d\n", |
| 829 | mtd->index, i); | 829 | mtd->index, i); |
| 830 | return -EEXIST; | 830 | return -EEXIST; |
| 831 | } | 831 | } |
| @@ -840,7 +840,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num, | |||
| 840 | * no sense to attach emulated MTD devices, so we prohibit this. | 840 | * no sense to attach emulated MTD devices, so we prohibit this. |
| 841 | */ | 841 | */ |
| 842 | if (mtd->type == MTD_UBIVOLUME) { | 842 | if (mtd->type == MTD_UBIVOLUME) { |
| 843 | pr_err("ubi: refuse attaching mtd%d - it is already emulated on top of UBI", | 843 | pr_err("ubi: refuse attaching mtd%d - it is already emulated on top of UBI\n", |
| 844 | mtd->index); | 844 | mtd->index); |
| 845 | return -EINVAL; | 845 | return -EINVAL; |
| 846 | } | 846 | } |
| @@ -851,7 +851,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num, | |||
| 851 | if (!ubi_devices[ubi_num]) | 851 | if (!ubi_devices[ubi_num]) |
| 852 | break; | 852 | break; |
| 853 | if (ubi_num == UBI_MAX_DEVICES) { | 853 | if (ubi_num == UBI_MAX_DEVICES) { |
| 854 | pr_err("ubi: only %d UBI devices may be created", | 854 | pr_err("ubi: only %d UBI devices may be created\n", |
| 855 | UBI_MAX_DEVICES); | 855 | UBI_MAX_DEVICES); |
| 856 | return -ENFILE; | 856 | return -ENFILE; |
| 857 | } | 857 | } |
| @@ -861,7 +861,7 @@ int ubi_attach_mtd_dev(struct mtd_info *mtd, int ubi_num, | |||
| 861 | 861 | ||
| 862 | /* Make sure ubi_num is not busy */ | 862 | /* Make sure ubi_num is not busy */ |
| 863 | if (ubi_devices[ubi_num]) { | 863 | if (ubi_devices[ubi_num]) { |
| 864 | pr_err("ubi: ubi%i already exists", ubi_num); | 864 | pr_err("ubi: ubi%i already exists\n", ubi_num); |
| 865 | return -EEXIST; | 865 | return -EEXIST; |
| 866 | } | 866 | } |
| 867 | } | 867 | } |
