aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Dooks <ben.dooks@codethink.co.uk>2017-07-26 05:51:10 -0400
committerRichard Weinberger <richard@nod.at>2017-09-13 16:05:30 -0400
commita51b7ccf3bf57956c64076987a9ad1152e896e79 (patch)
tree406b3222b6915f211d946137cb4f85e4de9c99bd
parent278f31a70d2e40aa31bf465795bfce2ca8bb01b6 (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.c8
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 }