[PATCH 3.2 139/140] ACPI: sbshc: remove raw pointer from printk() message
3.2.100-rc1 review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Ben Hutchings --- drivers/acpi/sbshc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -288,8 +288,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 3.2 139/140] ACPI: sbshc: remove raw pointer from printk() message
3.2.100-rc1 review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Ben Hutchings --- drivers/acpi/sbshc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -288,8 +288,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 3.16 253/254] ACPI: sbshc: remove raw pointer from printk() message
3.16.55-rc1 review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Ben Hutchings --- drivers/acpi/sbshc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -287,8 +287,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 3.16 253/254] ACPI: sbshc: remove raw pointer from printk() message
3.16.55-rc1 review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Ben Hutchings --- drivers/acpi/sbshc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -287,8 +287,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 3.18 44/45] ACPI: sbshc: remove raw pointer from printk() message
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -309,8 +309,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 3.18 44/45] ACPI: sbshc: remove raw pointer from printk() message
3.18-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -309,8 +309,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.4 105/108] ACPI: sbshc: remove raw pointer from printk() message
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.4 105/108] ACPI: sbshc: remove raw pointer from printk() message
4.4-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.15 190/202] ACPI: sbshc: remove raw pointer from printk() message
4.15-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.15 190/202] ACPI: sbshc: remove raw pointer from printk() message
4.15-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.14 186/195] ACPI: sbshc: remove raw pointer from printk() message
4.14-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.14 186/195] ACPI: sbshc: remove raw pointer from printk() message
4.14-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.9 84/88] ACPI: sbshc: remove raw pointer from printk() message
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartmancommit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
[PATCH 4.9 84/88] ACPI: sbshc: remove raw pointer from printk() message
4.9-stable review patch. If anyone has any objections, please let me know. -- From: Greg Kroah-Hartman commit 43cdd1b716b26f6af16da4e145b6578f98798bf6 upstream. There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Signed-off-by: Greg Kroah-Hartman Signed-off-by: Rafael J. Wysocki Signed-off-by: Greg Kroah-Hartman --- drivers/acpi/sbshc.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
Re: Re: ACPI: sbshc: remove raw pointer from printk message
On Mon, Jan 22, 2018 at 11:19:36AM +0800, wang_q...@venustech.com.cn wrote: > > I has recvied a cve-id from mitre.org for this security bug . > should this cve-id( CVE-2018-5750) be mentioned in kernel change log? Only if Rafael wants to hand-edit the patch, it's not really needed for a kernel log commit message to have CVE items in it, especially as the id was received after the patch was created and submitted. thanks, greg k-h
Re: Re: ACPI: sbshc: remove raw pointer from printk message
On Mon, Jan 22, 2018 at 11:19:36AM +0800, wang_q...@venustech.com.cn wrote: > > I has recvied a cve-id from mitre.org for this security bug . > should this cve-id( CVE-2018-5750) be mentioned in kernel change log? Only if Rafael wants to hand-edit the patch, it's not really needed for a kernel log commit message to have CVE items in it, especially as the id was received after the patch was created and submitted. thanks, greg k-h
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 09:03:33AM +0100, Rafael J. Wysocki wrote: > On Sat, Jan 20, 2018 at 8:36 AM, Greg Kroah-Hartman >wrote: > > On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: > >> On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman > >> wrote: > >> > There's no need to be printing a raw kernel pointer to the kernel log at > >> > every boot. So just remove it, and change the whole message to use the > >> > correct dev_info() call at the same time. > >> > > >> > Reported-by: Wang Qize > >> > Cc: stable > >> > >> I will remember this one. ;-) > >> > >> > Signed-off-by: Greg Kroah-Hartman > >> > >> Acked-by: Rafael J. Wysocki > > > > Does that mean you want me to take this through my tree? Or is there an > > acpi tree it will go through? > > I thought you'd push it, but I can take it, of course. For 4.16, though. 4.16 is fine, the issue has been around for a very long time, no rush :)
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 09:03:33AM +0100, Rafael J. Wysocki wrote: > On Sat, Jan 20, 2018 at 8:36 AM, Greg Kroah-Hartman > wrote: > > On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: > >> On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman > >> wrote: > >> > There's no need to be printing a raw kernel pointer to the kernel log at > >> > every boot. So just remove it, and change the whole message to use the > >> > correct dev_info() call at the same time. > >> > > >> > Reported-by: Wang Qize > >> > Cc: stable > >> > >> I will remember this one. ;-) > >> > >> > Signed-off-by: Greg Kroah-Hartman > >> > >> Acked-by: Rafael J. Wysocki > > > > Does that mean you want me to take this through my tree? Or is there an > > acpi tree it will go through? > > I thought you'd push it, but I can take it, of course. For 4.16, though. 4.16 is fine, the issue has been around for a very long time, no rush :)
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 8:36 AM, Greg Kroah-Hartmanwrote: > On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: >> On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman >> wrote: >> > There's no need to be printing a raw kernel pointer to the kernel log at >> > every boot. So just remove it, and change the whole message to use the >> > correct dev_info() call at the same time. >> > >> > Reported-by: Wang Qize >> > Cc: stable >> >> I will remember this one. ;-) >> >> > Signed-off-by: Greg Kroah-Hartman >> >> Acked-by: Rafael J. Wysocki > > Does that mean you want me to take this through my tree? Or is there an > acpi tree it will go through? I thought you'd push it, but I can take it, of course. For 4.16, though.
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 8:36 AM, Greg Kroah-Hartman wrote: > On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: >> On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman >> wrote: >> > There's no need to be printing a raw kernel pointer to the kernel log at >> > every boot. So just remove it, and change the whole message to use the >> > correct dev_info() call at the same time. >> > >> > Reported-by: Wang Qize >> > Cc: stable >> >> I will remember this one. ;-) >> >> > Signed-off-by: Greg Kroah-Hartman >> >> Acked-by: Rafael J. Wysocki > > Does that mean you want me to take this through my tree? Or is there an > acpi tree it will go through? I thought you'd push it, but I can take it, of course. For 4.16, though.
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: > On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman >wrote: > > There's no need to be printing a raw kernel pointer to the kernel log at > > every boot. So just remove it, and change the whole message to use the > > correct dev_info() call at the same time. > > > > Reported-by: Wang Qize > > Cc: stable > > I will remember this one. ;-) > > > Signed-off-by: Greg Kroah-Hartman > > Acked-by: Rafael J. Wysocki Does that mean you want me to take this through my tree? Or is there an acpi tree it will go through? thanks, greg k-h
Re: ACPI: sbshc: remove raw pointer from printk message
On Sat, Jan 20, 2018 at 12:53:39AM +0100, Rafael J. Wysocki wrote: > On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman > wrote: > > There's no need to be printing a raw kernel pointer to the kernel log at > > every boot. So just remove it, and change the whole message to use the > > correct dev_info() call at the same time. > > > > Reported-by: Wang Qize > > Cc: stable > > I will remember this one. ;-) > > > Signed-off-by: Greg Kroah-Hartman > > Acked-by: Rafael J. Wysocki Does that mean you want me to take this through my tree? Or is there an acpi tree it will go through? thanks, greg k-h
Re: ACPI: sbshc: remove raw pointer from printk message
On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartmanwrote: > There's no need to be printing a raw kernel pointer to the kernel log at > every boot. So just remove it, and change the whole message to use the > correct dev_info() call at the same time. > > Reported-by: Wang Qize > Cc: stable I will remember this one. ;-) > Signed-off-by: Greg Kroah-Hartman Acked-by: Rafael J. Wysocki > diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c > index 2fa8304171e0..217e1caf58d6 100644 > --- a/drivers/acpi/sbshc.c > +++ b/drivers/acpi/sbshc.c > @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi_device *device) > device->driver_data = hc; > > acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, > hc); > - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit > = 0x%0x\n", > - hc->ec, hc->offset, hc->query_bit); > + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", > +hc->offset, hc->query_bit); > > return 0; > } > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: ACPI: sbshc: remove raw pointer from printk message
On Fri, Jan 19, 2018 at 10:06 AM, Greg Kroah-Hartman wrote: > There's no need to be printing a raw kernel pointer to the kernel log at > every boot. So just remove it, and change the whole message to use the > correct dev_info() call at the same time. > > Reported-by: Wang Qize > Cc: stable I will remember this one. ;-) > Signed-off-by: Greg Kroah-Hartman Acked-by: Rafael J. Wysocki > diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c > index 2fa8304171e0..217e1caf58d6 100644 > --- a/drivers/acpi/sbshc.c > +++ b/drivers/acpi/sbshc.c > @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi_device *device) > device->driver_data = hc; > > acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, > hc); > - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit > = 0x%0x\n", > - hc->ec, hc->offset, hc->query_bit); > + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", > +hc->offset, hc->query_bit); > > return 0; > } > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majord...@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
ACPI: sbshc: remove raw pointer from printk message
There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang QizeCc: stable Signed-off-by: Greg Kroah-Hartman diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c index 2fa8304171e0..217e1caf58d6 100644 --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi_device *device) device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }
ACPI: sbshc: remove raw pointer from printk message
There's no need to be printing a raw kernel pointer to the kernel log at every boot. So just remove it, and change the whole message to use the correct dev_info() call at the same time. Reported-by: Wang Qize Cc: stable Signed-off-by: Greg Kroah-Hartman diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c index 2fa8304171e0..217e1caf58d6 100644 --- a/drivers/acpi/sbshc.c +++ b/drivers/acpi/sbshc.c @@ -275,8 +275,8 @@ static int acpi_smbus_hc_add(struct acpi_device *device) device->driver_data = hc; acpi_ec_add_query_handler(hc->ec, hc->query_bit, NULL, smbus_alarm, hc); - printk(KERN_INFO PREFIX "SBS HC: EC = 0x%p, offset = 0x%0x, query_bit = 0x%0x\n", - hc->ec, hc->offset, hc->query_bit); + dev_info(>dev, "SBS HC: offset = 0x%0x, query_bit = 0x%0x\n", +hc->offset, hc->query_bit); return 0; }