Skip to contentSkip to conversation

v4.1-rc1 & N900: annoying Nokia modem spam

PATCH New Aaro Koskinen 2 participants 4 messages May 2, 2015 – May 4, 2015 HSI

lore
Share & tools
lore
#0 lore raw

Hi,

With v4.1-rc1, after loading nokia_modem I get the following endless log spam until I load cmt_speech (for whatever reason it does not get modprobed automatically with busybox modprobe):

	[   41.626983] nokia-modem n900-modem: Missing cmt-speech driver
	[   41.804565] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.804626] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.804687] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[0]' - status (0)
	[   41.804718] no flags found for (null)
	[   41.805023] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805023] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805145] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[1]' - status (0)
	[   41.805145] no flags found for (null)
	[   41.805389] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805419] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805541] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[2]' - status (0)
	[   41.805572] no flags found for (null)
	[   41.805816] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805816] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805969] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[3]' - status (0)
	[   41.805999] no flags found for (null)

Not nice.

Also, I now realize there is a forced nokia_modem Kconfig dependency to cmt_speech. Before I was able to use data connection only, so why now force this dependency to cmt_speech?

A.

#1 lore raw

Hi,

Quoted 3 lines · Aaro Koskinen

With v4.1-rc1, after loading nokia_modem I get the following endless log spam until I load cmt_speech (for whatever reason it does not get modprobed automatically with busybox modprobe):

So ssi-protocol is loaded automatically and cmt-speech is not? That's strange, since they are requested the same way.

Quoted 18 lines
	[   41.626983] nokia-modem n900-modem: Missing cmt-speech driver
	[   41.804565] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.804626] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.804687] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[0]' - status (0)
	[   41.804718] no flags found for (null)
	[   41.805023] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805023] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805145] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[1]' - status (0)
	[   41.805145] no flags found for (null)
	[   41.805389] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805419] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805541] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[2]' - status (0)
	[   41.805572] no flags found for (null)
	[   41.805816] nokia-modem n900-modem: GPIO lookup for consumer (null)
	[   41.805816] nokia-modem n900-modem: using device tree for GPIO lookup
	[   41.805969] of_get_named_gpiod_flags: parsed 'gpios' property of node '/ocp/ssi-controller@48058000/ssi-port@4805a000/hsi-client[3]' - status (0)
	[   41.805999] no flags found for (null)
Quoted 1 line

Not nice.

Everything (except of the first line) is because you have CONFIG_DEBUG_GPIO enabled and nokia-modem returns -EPROBE_DEFER until all drivers are available, so GPIOs will be acquired again the next time the drivers tries to probe.

Quoted 3 lines

Also, I now realize there is a forced nokia_modem Kconfig dependency to cmt_speech. Before I was able to use data connection only, so why now force this dependency to cmt_speech?

I assumed, that there is no real usecase for not building cmt-speech together with nokia-modem. I think CMT_SPEECH and SSI_PROTOCOL should actually be hidden and autoselected by NOKIA_MODEM.

-- Sebastian

#2 lore raw

Hi,

Quoted 7 lines · Sebastian Reichel
Quoted 3 lines · Aaro Koskinen

With v4.1-rc1, after loading nokia_modem I get the following endless log spam until I load cmt_speech (for whatever reason it does not get modprobed automatically with busybox modprobe):

So ssi-protocol is loaded automatically and cmt-speech is not? That's strange, since they are requested the same way.

Hmm, right. This seems to be issue in my initramfs setup.

Quoted 6 lines
Quoted 1 line
	[   41.626983] nokia-modem n900-modem: Missing cmt-speech driver

Everything (except of the first line) is because you have CONFIG_DEBUG_GPIO enabled and nokia-modem returns -EPROBE_DEFER until all drivers are available, so GPIOs will be acquired again the next time the drivers tries to probe.

Actually only the first line (Missing cmt-speech driver) was the annoying part - because it's an error log it reached the frame buffer and there was so many of them that everything else scrolled off...

A.

#3 lore raw

PATCH 1/1 HSI: nokia-modem: Reduce missing driver message to debug level

In reply to Aaro Koskinen

Reduce message priority from dev_err to dev_dbg for missing cmt-speech or ssi-protocol drivers, since they will be probed again and it may result in spamming the boot log.

---
 drivers/hsi/clients/nokia-modem.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
drivers/hsi/clients/nokia-modem.c +2 -2
index bbb1923..f50d110 100644
@@ -208,7 +208,7 @@ static int nokia_modem_probe(struct device *dev)
 
 	err = device_attach(&modem->ssi_protocol->device);
 	if (err == 0) {
-		dev_err(dev, "Missing ssi-protocol driver\n");
+		dev_dbg(dev, "Missing ssi-protocol driver\n");
 		err = -EPROBE_DEFER;
 		goto error3;
 	} else if (err < 0) {
@@ -231,7 +231,7 @@ static int nokia_modem_probe(struct device *dev)
 
 	err = device_attach(&modem->cmt_speech->device);
 	if (err == 0) {
-		dev_err(dev, "Missing cmt-speech driver\n");
+		dev_dbg(dev, "Missing cmt-speech driver\n");
 		err = -EPROBE_DEFER;
 		goto error4;
 	} else if (err < 0) {
Signature · 1 line
2.1.4