On Tuesday 24 June 2008 09:04:47 pm Kenji Kaneshige wrote:
> Bjorn Helgaas wrote:
> > On Thursday 19 June 2008 09:06:24 pm Kenji Kaneshige wrote:
> >> Fix the typo in hpc_release_ctlr().
> >>
> >> Signed-off-by: Kenji Kaneshige <kaneshige.kenji@xxxxxxxxxxxxxx>
> >>
> >> ---
> >> drivers/pci/hotplug/pciehp_hpc.c | 2 +-
> >> 1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> Index: 20080617/drivers/pci/hotplug/pciehp_hpc.c
> >> ===================================================================
> >> --- 20080617.orig/drivers/pci/hotplug/pciehp_hpc.c
> >> +++ 20080617/drivers/pci/hotplug/pciehp_hpc.c
> >> @@ -613,7 +613,7 @@ static void hpc_release_ctlr(struct cont
> >> {
> >> /* Mask Hot-plug Interrupt Enable */
> >> if (pcie_write_cmd(ctrl, 0, HP_INTR_ENABLE | CMD_CMPL_INTR_ENABLE))
> >> - err("%s: Cannot mask hotplut interrupt enable\n", __func__);
> >> + err("%s: Cannot mask hotplug interrupt enable\n", __func__);
> >
> > Can you use dev_err() to associate the message with a device or
> > root port or something?
>
> Thank you very much for comment.
> There are many other messages that also need to be changed in pciehp
> driver. So I'll do that as another patch.
That'd be tremendous (IMHO). I posted patches to do that for some
of PCI, but I haven't done anything under drivers/pci/hotplug, because
it seemed like it'd be nice to do all the hotplug drivers at once. And
there are a lot.
Bjorn
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
