* [PATCH] [MEGARAID_SAS]: Use pr_info
@ 2007-08-08 23:14 Arnaldo Carvalho de Melo
2007-08-10 18:18 ` Patro, Sumant
0 siblings, 1 reply; 3+ messages in thread
From: Arnaldo Carvalho de Melo @ 2007-08-08 23:14 UTC (permalink / raw)
To: megaraidlinux; +Cc: linux-scsi, James Bottomley
Signed-off-by: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
---
drivers/scsi/megaraid/megaraid_sas.c | 14 ++++++--------
1 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/drivers/scsi/megaraid/megaraid_sas.c b/drivers/scsi/megaraid/megaraid_sas.c
index 59099da..4a8cf06 100644
--- a/drivers/scsi/megaraid/megaraid_sas.c
+++ b/drivers/scsi/megaraid/megaraid_sas.c
@@ -1341,8 +1341,7 @@ megasas_transition_to_ready(struct megasas_instance* instance)
fw_state = instance->instancet->read_fw_status_reg(instance->reg_set) & MFI_STATE_MASK;
if (fw_state != MFI_STATE_READY)
- printk(KERN_INFO "megasas: Waiting for FW to come to ready"
- " state\n");
+ pr_info("megasas: Waiting for FW to come to ready state\n");
while (fw_state != MFI_STATE_READY) {
@@ -1444,7 +1443,7 @@ megasas_transition_to_ready(struct megasas_instance* instance)
return -ENODEV;
}
};
- printk(KERN_INFO "megasas: FW now in Ready state\n");
+ pr_info("megasas: FW now in Ready state\n");
return 0;
}
@@ -2277,9 +2276,9 @@ megasas_probe_one(struct pci_dev *pdev, const struct pci_device_id *id)
/*
* Announce PCI information
*/
- printk(KERN_INFO "megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
- pdev->vendor, pdev->device, pdev->subsystem_vendor,
- pdev->subsystem_device);
+ pr_info("megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
+ pdev->vendor, pdev->device, pdev->subsystem_vendor,
+ pdev->subsystem_device);
printk("bus %d:slot %d:func %d\n",
pdev->bus->number, PCI_SLOT(pdev->devfn), PCI_FUNC(pdev->devfn));
@@ -3030,8 +3029,7 @@ static int __init megasas_init(void)
/*
* Announce driver version and other information
*/
- printk(KERN_INFO "megasas: %s %s\n", MEGASAS_VERSION,
- MEGASAS_EXT_VERSION);
+ pr_info("megasas: %s %s\n", MEGASAS_VERSION, MEGASAS_EXT_VERSION);
/*
* Register character device node
--
1.5.2.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* RE: [PATCH] [MEGARAID_SAS]: Use pr_info
2007-08-08 23:14 [PATCH] [MEGARAID_SAS]: Use pr_info Arnaldo Carvalho de Melo
@ 2007-08-10 18:18 ` Patro, Sumant
2007-08-10 18:26 ` Brian King
0 siblings, 1 reply; 3+ messages in thread
From: Patro, Sumant @ 2007-08-10 18:18 UTC (permalink / raw)
To: Arnaldo Carvalho de Melo; +Cc: linux-scsi, Kolli, Neela, Yang, Bo
Hello Arnaldo,
Throughout the megaraid_sas driver code there are printk's with
appropriate flags (that includes KERN_INFO). What is the incentive to
change printk with KERN_INFO to pr_info?
Regards,
Sumant
-----Original Message-----
From: Arnaldo Carvalho de Melo [mailto:arnaldo.melo@gmail.com] On Behalf
Of Arnaldo Carvalho de Melo
Sent: Wednesday, August 08, 2007 4:15 PM
To: DL-MegaRAID Linux
Cc: linux-scsi@vger.kernel.org; James Bottomley
Subject: [PATCH] [MEGARAID_SAS]: Use pr_info
Signed-off-by: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
---
drivers/scsi/megaraid/megaraid_sas.c | 14 ++++++--------
1 files changed, 6 insertions(+), 8 deletions(-)
diff --git a/drivers/scsi/megaraid/megaraid_sas.c
b/drivers/scsi/megaraid/megaraid_sas.c
index 59099da..4a8cf06 100644
--- a/drivers/scsi/megaraid/megaraid_sas.c
+++ b/drivers/scsi/megaraid/megaraid_sas.c
@@ -1341,8 +1341,7 @@ megasas_transition_to_ready(struct
megasas_instance* instance)
fw_state =
instance->instancet->read_fw_status_reg(instance->reg_set) &
MFI_STATE_MASK;
if (fw_state != MFI_STATE_READY)
- printk(KERN_INFO "megasas: Waiting for FW to come to
ready"
- " state\n");
+ pr_info("megasas: Waiting for FW to come to ready
state\n");
while (fw_state != MFI_STATE_READY) {
@@ -1444,7 +1443,7 @@ megasas_transition_to_ready(struct
megasas_instance* instance)
return -ENODEV;
}
};
- printk(KERN_INFO "megasas: FW now in Ready state\n");
+ pr_info("megasas: FW now in Ready state\n");
return 0;
}
@@ -2277,9 +2276,9 @@ megasas_probe_one(struct pci_dev *pdev, const
struct pci_device_id *id)
/*
* Announce PCI information
*/
- printk(KERN_INFO "megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
- pdev->vendor, pdev->device, pdev->subsystem_vendor,
- pdev->subsystem_device);
+ pr_info("megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
+ pdev->vendor, pdev->device, pdev->subsystem_vendor,
+ pdev->subsystem_device);
printk("bus %d:slot %d:func %d\n",
pdev->bus->number, PCI_SLOT(pdev->devfn),
PCI_FUNC(pdev->devfn)); @@ -3030,8 +3029,7 @@ static int __init
megasas_init(void)
/*
* Announce driver version and other information
*/
- printk(KERN_INFO "megasas: %s %s\n", MEGASAS_VERSION,
- MEGASAS_EXT_VERSION);
+ pr_info("megasas: %s %s\n", MEGASAS_VERSION,
MEGASAS_EXT_VERSION);
/*
* Register character device node
--
1.5.2.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] [MEGARAID_SAS]: Use pr_info
2007-08-10 18:18 ` Patro, Sumant
@ 2007-08-10 18:26 ` Brian King
0 siblings, 0 replies; 3+ messages in thread
From: Brian King @ 2007-08-10 18:26 UTC (permalink / raw)
To: Patro, Sumant
Cc: Arnaldo Carvalho de Melo, linux-scsi, Kolli, Neela, Yang, Bo
Patro, Sumant wrote:
> Hello Arnaldo,
>
> Throughout the megaraid_sas driver code there are printk's with
> appropriate flags (that includes KERN_INFO). What is the incentive to
> change printk with KERN_INFO to pr_info?
And why not change most of these to dev_printk and friends?
-Brian
>
> Regards,
>
> Sumant
>
> -----Original Message-----
> From: Arnaldo Carvalho de Melo [mailto:arnaldo.melo@gmail.com] On Behalf
> Of Arnaldo Carvalho de Melo
> Sent: Wednesday, August 08, 2007 4:15 PM
> To: DL-MegaRAID Linux
> Cc: linux-scsi@vger.kernel.org; James Bottomley
> Subject: [PATCH] [MEGARAID_SAS]: Use pr_info
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@ghostprotocols.net>
> ---
> drivers/scsi/megaraid/megaraid_sas.c | 14 ++++++--------
> 1 files changed, 6 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/scsi/megaraid/megaraid_sas.c
> b/drivers/scsi/megaraid/megaraid_sas.c
> index 59099da..4a8cf06 100644
> --- a/drivers/scsi/megaraid/megaraid_sas.c
> +++ b/drivers/scsi/megaraid/megaraid_sas.c
> @@ -1341,8 +1341,7 @@ megasas_transition_to_ready(struct
> megasas_instance* instance)
> fw_state =
> instance->instancet->read_fw_status_reg(instance->reg_set) &
> MFI_STATE_MASK;
>
> if (fw_state != MFI_STATE_READY)
> - printk(KERN_INFO "megasas: Waiting for FW to come to
> ready"
> - " state\n");
> + pr_info("megasas: Waiting for FW to come to ready
> state\n");
>
> while (fw_state != MFI_STATE_READY) {
>
> @@ -1444,7 +1443,7 @@ megasas_transition_to_ready(struct
> megasas_instance* instance)
> return -ENODEV;
> }
> };
> - printk(KERN_INFO "megasas: FW now in Ready state\n");
> + pr_info("megasas: FW now in Ready state\n");
>
> return 0;
> }
> @@ -2277,9 +2276,9 @@ megasas_probe_one(struct pci_dev *pdev, const
> struct pci_device_id *id)
> /*
> * Announce PCI information
> */
> - printk(KERN_INFO "megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
> - pdev->vendor, pdev->device, pdev->subsystem_vendor,
> - pdev->subsystem_device);
> + pr_info("megasas: %#4.04x:%#4.04x:%#4.04x:%#4.04x: ",
> + pdev->vendor, pdev->device, pdev->subsystem_vendor,
> + pdev->subsystem_device);
>
> printk("bus %d:slot %d:func %d\n",
> pdev->bus->number, PCI_SLOT(pdev->devfn),
> PCI_FUNC(pdev->devfn)); @@ -3030,8 +3029,7 @@ static int __init
> megasas_init(void)
> /*
> * Announce driver version and other information
> */
> - printk(KERN_INFO "megasas: %s %s\n", MEGASAS_VERSION,
> - MEGASAS_EXT_VERSION);
> + pr_info("megasas: %s %s\n", MEGASAS_VERSION,
> MEGASAS_EXT_VERSION);
>
> /*
> * Register character device node
> --
> 1.5.2.2
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
--
Brian King
Linux on Power Virtualization
IBM Linux Technology Center
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-08-10 18:26 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-08 23:14 [PATCH] [MEGARAID_SAS]: Use pr_info Arnaldo Carvalho de Melo
2007-08-10 18:18 ` Patro, Sumant
2007-08-10 18:26 ` Brian King
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.