diff options
| author | Andreas Schwab <schwab@linux-m68k.org> | 2016-11-28 21:29:07 +0100 | 
|---|---|---|
| committer | Michael Ellerman <mpe@ellerman.id.au> | 2018-01-27 21:10:22 +1100 | 
| commit | f2be6295684b0fe879a824ac97479799e760b7e9 (patch) | |
| tree | c0bbf5fc61f7aa330ff4162db0c32836c90da258 /drivers/macintosh | |
| parent | e67e02a544e9a0d24b99eb383e808bb3433b048d (diff) | |
| download | linux-f2be6295684b0fe879a824ac97479799e760b7e9.tar.bz2 | |
macintosh/adb: Properly mark continued kernel messages
Use pr_cont where appropriate, and switch to pr_foo throughout.
Additionally, lower messages in adb_probe_task to debug level.
Signed-off-by: Andreas Schwab <schwab@linux-m68k.org>
[mpe: Clean up whitespace slightly]
Signed-off-by: Michael Ellerman <mpe@ellerman.id.au>
Diffstat (limited to 'drivers/macintosh')
| -rw-r--r-- | drivers/macintosh/adb.c | 24 | ||||
| -rw-r--r-- | drivers/macintosh/adbhid.c | 55 | 
2 files changed, 38 insertions, 41 deletions
diff --git a/drivers/macintosh/adb.c b/drivers/macintosh/adb.c index 289800b5235d..4c8097e0e6fe 100644 --- a/drivers/macintosh/adb.c +++ b/drivers/macintosh/adb.c @@ -203,18 +203,17 @@ static int adb_scan_bus(void)  	}  	/* Now fill in the handler_id field of the adb_handler entries. */ -	printk(KERN_DEBUG "adb devices:"); +	pr_debug("adb devices:\n");  	for (i = 1; i < 16; i++) {  		if (adb_handler[i].original_address == 0)  			continue;  		adb_request(&req, NULL, ADBREQ_SYNC | ADBREQ_REPLY, 1,  			    (i << 4) | 0xf);  		adb_handler[i].handler_id = req.reply[2]; -		printk(" [%d]: %d %x", i, adb_handler[i].original_address, -		       adb_handler[i].handler_id); +		pr_debug(" [%d]: %d %x\n", i, adb_handler[i].original_address, +			 adb_handler[i].handler_id);  		devmask |= 1 << i;  	} -	printk("\n");  	return devmask;  } @@ -225,9 +224,9 @@ static int adb_scan_bus(void)  static int  adb_probe_task(void *x)  { -	printk(KERN_INFO "adb: starting probe task...\n"); +	pr_debug("adb: starting probe task...\n");  	do_adb_reset_bus(); -	printk(KERN_INFO "adb: finished probe task...\n"); +	pr_debug("adb: finished probe task...\n");  	up(&adb_probe_mutex); @@ -337,7 +336,7 @@ static int __init adb_init(void)  	    adb_controller->init())  		adb_controller = NULL;  	if (adb_controller == NULL) { -		printk(KERN_WARNING "Warning: no ADB interface detected\n"); +		pr_warn("Warning: no ADB interface detected\n");  	} else {  #ifdef CONFIG_PPC  		if (of_machine_is_compatible("AAPL,PowerBook1998") || @@ -480,8 +479,7 @@ adb_register(int default_id, int handler_id, struct adb_ids *ids,  		    (!handler_id || (handler_id == adb_handler[i].handler_id) ||   		    try_handler_change(i, handler_id))) {  			if (adb_handler[i].handler != 0) { -				printk(KERN_ERR -				       "Two handlers for ADB device %d\n", +				pr_err("Two handlers for ADB device %d\n",  				       default_id);  				continue;  			} @@ -535,10 +533,10 @@ adb_input(unsigned char *buf, int nb, int autopoll)  	id = buf[0] >> 4;  	if (dump_adb_input) { -		printk(KERN_INFO "adb packet: "); +		pr_info("adb packet: ");  		for (i = 0; i < nb; ++i) -			printk(" %x", buf[i]); -		printk(", id = %d\n", id); +			pr_cont(" %x", buf[i]); +		pr_cont(", id = %d\n", id);  	}  	write_lock_irqsave(&adb_handler_lock, flags);  	handler = adb_handler[id].handler; @@ -884,7 +882,7 @@ static void __init  adbdev_init(void)  {  	if (register_chrdev(ADB_MAJOR, "adb", &adb_fops)) { -		printk(KERN_ERR "adb: unable to get major %d\n", ADB_MAJOR); +		pr_err("adb: unable to get major %d\n", ADB_MAJOR);  		return;  	} diff --git a/drivers/macintosh/adbhid.c b/drivers/macintosh/adbhid.c index e091193104f7..a261892c03b3 100644 --- a/drivers/macintosh/adbhid.c +++ b/drivers/macintosh/adbhid.c @@ -268,7 +268,7 @@ adbhid_keyboard_input(unsigned char *data, int nb, int apoll)  	int id = (data[0] >> 4) & 0x0f;  	if (!adbhid[id]) { -		printk(KERN_ERR "ADB HID on ID %d not yet registered, packet %#02x, %#02x, %#02x, %#02x\n", +		pr_err("ADB HID on ID %d not yet registered, packet %#02x, %#02x, %#02x, %#02x\n",  		       id, data[0], data[1], data[2], data[3]);  		return;  	} @@ -320,8 +320,7 @@ adbhid_input_keycode(int id, int scancode, int repeat)  					ahid->flags &= ~FLAG_CAPSLOCK_TRANSLATE;  				}  			} else { -				printk(KERN_INFO "Spurious caps lock event " -						 "(scancode 0xff).\n"); +				pr_info("Spurious caps lock event (scancode 0xff).\n");  			}  		}  	} @@ -397,8 +396,8 @@ adbhid_input_keycode(int id, int scancode, int repeat)  		input_report_key(adbhid[id]->input, key, !up_flag);  		input_sync(adbhid[id]->input);  	} else -		printk(KERN_INFO "Unhandled ADB key (scancode %#02x) %s.\n", keycode, -		       up_flag ? "released" : "pressed"); +		pr_info("Unhandled ADB key (scancode %#02x) %s.\n", keycode, +			up_flag ? "released" : "pressed");  } @@ -408,7 +407,7 @@ adbhid_mouse_input(unsigned char *data, int nb, int autopoll)  	int id = (data[0] >> 4) & 0x0f;  	if (!adbhid[id]) { -		printk(KERN_ERR "ADB HID on ID %d not yet registered\n", id); +		pr_err("ADB HID on ID %d not yet registered\n", id);  		return;  	} @@ -506,7 +505,7 @@ adbhid_buttons_input(unsigned char *data, int nb, int autopoll)  	int id = (data[0] >> 4) & 0x0f;  	if (!adbhid[id]) { -		printk(KERN_ERR "ADB HID on ID %d not yet registered\n", id); +		pr_err("ADB HID on ID %d not yet registered\n", id);  		return;  	} @@ -534,8 +533,8 @@ adbhid_buttons_input(unsigned char *data, int nb, int autopoll)  			break;  		default: -			printk(KERN_INFO "Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", -			       data[0], data[1], data[2], data[3]); +			pr_info("Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", +				data[0], data[1], data[2], data[3]);  			break;  		}  	  } @@ -609,14 +608,14 @@ adbhid_buttons_input(unsigned char *data, int nb, int autopoll)  				break;  			default: -				printk(KERN_INFO "Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", -				       data[0], data[1], data[2], data[3]); +				pr_info("Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", +					data[0], data[1], data[2], data[3]);  				break;  			}  			break;  		default: -			printk(KERN_INFO "Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", -			       data[0], data[1], data[2], data[3]); +			pr_info("Unhandled ADB_MISC event %02x, %02x, %02x, %02x\n", +				data[0], data[1], data[2], data[3]);  			break;  		}  	  } @@ -760,7 +759,7 @@ adbhid_input_register(int id, int default_id, int original_handler_id,  	int i;  	if (adbhid[id]) { -		printk(KERN_ERR "Trying to reregister ADB HID on ID %d\n", id); +		pr_err("Trying to reregister ADB HID on ID %d\n", id);  		return -EEXIST;  	} @@ -799,24 +798,24 @@ adbhid_input_register(int id, int default_id, int original_handler_id,  		memcpy(hid->keycode, adb_to_linux_keycodes, sizeof(adb_to_linux_keycodes)); -		printk(KERN_INFO "Detected ADB keyboard, type "); +		pr_info("Detected ADB keyboard, type ");  		switch (original_handler_id) {  		default: -			printk("<unknown>.\n"); +			pr_cont("<unknown>.\n");  			input_dev->id.version = ADB_KEYBOARD_UNKNOWN;  			break;  		case 0x01: case 0x02: case 0x03: case 0x06: case 0x08:  		case 0x0C: case 0x10: case 0x18: case 0x1B: case 0x1C:  		case 0xC0: case 0xC3: case 0xC6: -			printk("ANSI.\n"); +			pr_cont("ANSI.\n");  			input_dev->id.version = ADB_KEYBOARD_ANSI;  			break;  		case 0x04: case 0x05: case 0x07: case 0x09: case 0x0D:  		case 0x11: case 0x14: case 0x19: case 0x1D: case 0xC1:  		case 0xC4: case 0xC7: -			printk("ISO, swapping keys.\n"); +			pr_cont("ISO, swapping keys.\n");  			input_dev->id.version = ADB_KEYBOARD_ISO;  			i = hid->keycode[10];  			hid->keycode[10] = hid->keycode[50]; @@ -825,7 +824,7 @@ adbhid_input_register(int id, int default_id, int original_handler_id,  		case 0x12: case 0x15: case 0x16: case 0x17: case 0x1A:  		case 0x1E: case 0xC2: case 0xC5: case 0xC8: case 0xC9: -			printk("JIS.\n"); +			pr_cont("JIS.\n");  			input_dev->id.version = ADB_KEYBOARD_JIS;  			break;  		} @@ -884,7 +883,7 @@ adbhid_input_register(int id, int default_id, int original_handler_id,  		/* else fall through */  	default: -		printk(KERN_INFO "Trying to register unknown ADB device to input layer.\n"); +		pr_info("Trying to register unknown ADB device to input layer.\n");  		err = -ENODEV;  		goto fail;  	} @@ -1073,12 +1072,12 @@ adbhid_probe(void)  			    (req.reply[1] == 0x4b) && (req.reply[2] == 0x4f) &&  			    (req.reply[3] == 0x49) && (req.reply[4] == 0x54)) {  				if (adb_try_handler_change(id, 0x42)) { -					printk("\nADB MacAlly 2-button mouse at %d, handler set to 0x42", id); +					pr_cont("\nADB MacAlly 2-button mouse at %d, handler set to 0x42", id);  					mouse_kind = ADBMOUSE_MACALLY2;  				}  			}  		} -		printk("\n"); +		pr_cont("\n");  		adb_get_infos(id, &default_id, &cur_handler_id);  		reg |= adbhid_input_reregister(id, default_id, org_handler_id, @@ -1093,12 +1092,12 @@ init_trackpad(int id)  	struct adb_request req;  	unsigned char r1_buffer[8]; -	printk(" (trackpad)"); +	pr_cont(" (trackpad)");  	adb_request(&req, NULL, ADBREQ_SYNC | ADBREQ_REPLY, 1,  		    ADB_READREG(id,1));  	if (req.reply_len < 8) -	    printk("bad length for reg. 1\n"); +	    pr_cont("bad length for reg. 1\n");  	else  	{  	    memcpy(r1_buffer, &req.reply[1], 8); @@ -1146,7 +1145,7 @@ init_trackball(int id)  {  	struct adb_request req; -	printk(" (trackman/mouseman)"); +	pr_cont(" (trackman/mouseman)");  	adb_request(&req, NULL, ADBREQ_SYNC, 3,  	ADB_WRITEREG(id,1), 00,0x81); @@ -1178,7 +1177,7 @@ init_turbomouse(int id)  {  	struct adb_request req; -        printk(" (TurboMouse 5)"); +	pr_cont(" (TurboMouse 5)");  	adb_request(&req, NULL, ADBREQ_SYNC, 1, ADB_FLUSH(id)); @@ -1214,7 +1213,7 @@ init_microspeed(int id)  {  	struct adb_request req; -        printk(" (Microspeed/MacPoint or compatible)"); +	pr_cont(" (Microspeed/MacPoint or compatible)");  	adb_request(&req, NULL, ADBREQ_SYNC, 1, ADB_FLUSH(id)); @@ -1254,7 +1253,7 @@ init_ms_a3(int id)  {  	struct adb_request req; -	printk(" (Mouse Systems A3 Mouse, or compatible)"); +	pr_cont(" (Mouse Systems A3 Mouse, or compatible)");  	adb_request(&req, NULL, ADBREQ_SYNC, 3,  	ADB_WRITEREG(id, 0x2),  	    0x00,  |