]> git.proxmox.com Git - mirror_edk2.git/commitdiff
EmbeddedPkg/Lan9118Dxe: minor DEBUG tidyup
authorRyan Harkin <ryan.harkin@linaro.org>
Tue, 9 Feb 2016 10:07:13 +0000 (10:07 +0000)
committerArd Biesheuvel <ard.biesheuvel@linaro.org>
Wed, 10 Feb 2016 16:56:47 +0000 (17:56 +0100)
This patch makes a few minor DEBUG output changes:

- Fix typo in DEBUG output: Negociation->Negotiation

- Change DEBUG occurrences of "Lan9118" to "LAN9118" to make grepping
  the log output easier.

- Change the warning that auto-negotiation is not supported when
  AutoNegotiate() returns an error.
  The function already reports if the feature is supported or not and
  can also return an error for other reasons.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Ryan Harkin <ryan.harkin@linaro.org>
Reviewed-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
EmbeddedPkg/Drivers/Lan9118Dxe/Lan9118Dxe.c

index 79bee3f47cacde98ecf5e0c557ffef496ac3f222..d0bf7beefe2afd2eded73a7c2fbec62b2c91b262 100644 (file)
@@ -142,7 +142,7 @@ Lan9118DxeEntry (
   // Power up the device so we can find the MAC address\r
   Status = Lan9118Initialize (Snp);\r
   if (EFI_ERROR (Status)) {\r
-    DEBUG ((EFI_D_ERROR, "Lan9118: Error initialising hardware\n"));\r
+    DEBUG ((EFI_D_ERROR, "LAN9118: Error initialising hardware\n"));\r
     return EFI_DEVICE_ERROR;\r
   }\r
 \r
@@ -342,7 +342,7 @@ SnpInitialize (
   // Do auto-negotiation if supported\r
   Status = AutoNegotiate (AUTO_NEGOTIATE_ADVERTISE_ALL, Snp);\r
   if (EFI_ERROR(Status)) {\r
-    DEBUG ((EFI_D_WARN, "Lan9118: Auto Negociation not supported.\n"));\r
+    DEBUG ((EFI_D_WARN, "LAN9118: Auto Negotiation failed.\n"));\r
   }\r
 \r
   // Configure flow control depending on speed capabilities\r
@@ -767,7 +767,7 @@ SnpStationAddress (
       New = (EFI_MAC_ADDRESS *) PermAddr;\r
       Lan9118SetMacAddress ((EFI_MAC_ADDRESS *) PermAddr, Snp);\r
     } else {\r
-      DEBUG ((EFI_D_ERROR, "Lan9118: Warning: No valid MAC address in EEPROM, using fallback\n"));\r
+      DEBUG ((EFI_D_ERROR, "LAN9118: Warning: No valid MAC address in EEPROM, using fallback\n"));\r
       New = (EFI_MAC_ADDRESS*) (FixedPcdGet64 (PcdLan9118DefaultMacAddress));\r
     }\r
   } else {\r