From c194ccca2613355f0f4506e7692bc55e2eaf6ff4 Mon Sep 17 00:00:00 2001 From: Antoine Coeur Date: Fri, 7 Feb 2020 02:07:44 +0100 Subject: [PATCH] NetworkPkg/Dhcp4Dxe: Fix various typos Fix various typos in comments and documentation. Cc: Jiaxin Wu Cc: Siyuan Fu Cc: Maciej Rabeda Signed-off-by: Antoine Coeur Reviewed-by: Philippe Mathieu-Daude Reviewed-by: Maciej Rabeda Signed-off-by: Philippe Mathieu-Daude Message-Id: <20200207010831.9046-32-philmd@redhat.com> --- NetworkPkg/Dhcp4Dxe/Dhcp4Driver.c | 8 ++++---- NetworkPkg/Dhcp4Dxe/Dhcp4Driver.h | 4 ++-- NetworkPkg/Dhcp4Dxe/Dhcp4Impl.c | 6 +++--- NetworkPkg/Dhcp4Dxe/Dhcp4Io.c | 12 ++++++------ NetworkPkg/Dhcp4Dxe/Dhcp4Io.h | 2 +- NetworkPkg/Dhcp4Dxe/Dhcp4Option.c | 26 +++++++++++++------------- NetworkPkg/Dhcp4Dxe/Dhcp4Option.h | 18 +++++++++--------- 7 files changed, 38 insertions(+), 38 deletions(-) diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.c b/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.c index e891b68883..970141abfe 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.c +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.c @@ -326,7 +326,7 @@ Dhcp4DriverBindingStart ( } // - // Install the Dhcp4ServiceBinding Protocol onto ControlerHandle + // Install the Dhcp4ServiceBinding Protocol onto ControllerHandle // Status = gBS->InstallMultipleProtocolInterfaces ( &ControllerHandle, @@ -438,7 +438,7 @@ Dhcp4DriverBindingStop ( DhcpSb = DHCP_SERVICE_FROM_THIS (ServiceBinding); if (!IsListEmpty (&DhcpSb->Children)) { // - // Destroy all the children instances before destory the service. + // Destroy all the children instances before destroy the service. // List = &DhcpSb->Children; Status = NetDestroyLinkList ( @@ -523,7 +523,7 @@ DhcpInitProtocol ( then a new handle is created. If it is a pointer to an existing UEFI handle, then the protocol is added to the existing UEFI handle. - @retval EFI_SUCCES The protocol was added to ChildHandle. + @retval EFI_SUCCESS The protocol was added to ChildHandle. @retval EFI_INVALID_PARAMETER ChildHandle is NULL. @retval EFI_OUT_OF_RESOURCES There are not enough resources available to create the child @@ -617,7 +617,7 @@ Dhcp4ServiceBindingCreateChild ( @param This Pointer to the EFI_SERVICE_BINDING_PROTOCOL instance. @param ChildHandle Handle of the child to destroy - @retval EFI_SUCCES The protocol was removed from ChildHandle. + @retval EFI_SUCCESS The protocol was removed from ChildHandle. @retval EFI_UNSUPPORTED ChildHandle does not support the protocol that is being removed. @retval EFI_INVALID_PARAMETER Child handle is NULL. @retval EFI_ACCESS_DENIED The protocol could not be removed from the ChildHandle diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.h b/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.h index cebde20dca..f6370582cb 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.h +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Driver.h @@ -104,7 +104,7 @@ Dhcp4DriverBindingStop ( then a new handle is created. If it is a pointer to an existing UEFI handle, then the protocol is added to the existing UEFI handle. - @retval EFI_SUCCES The protocol was added to ChildHandle. + @retval EFI_SUCCESS The protocol was added to ChildHandle. @retval EFI_INVALID_PARAMETER ChildHandle is NULL. @retval EFI_OUT_OF_RESOURCES There are not enough resources available to create the child @@ -128,7 +128,7 @@ Dhcp4ServiceBindingCreateChild ( @param This Pointer to the EFI_SERVICE_BINDING_PROTOCOL instance. @param ChildHandle Handle of the child to destroy - @retval EFI_SUCCES The protocol was removed from ChildHandle. + @retval EFI_SUCCESS The protocol was removed from ChildHandle. @retval EFI_UNSUPPORTED ChildHandle does not support the protocol that is being removed. @retval EFI_INVALID_PARAMETER Child handle is NULL. @retval EFI_ACCESS_DENIED The protocol could not be removed from the ChildHandle diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Impl.c b/NetworkPkg/Dhcp4Dxe/Dhcp4Impl.c index 0b35bdf4df..8290533790 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Impl.c +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Impl.c @@ -64,7 +64,7 @@ EfiDhcp4GetModeData ( not NULL. Otherwise, the state was left unchanged. @retval EFI_ACCESS_DENIED This instance of the EFI DHCPv4 Protocol driver was not in the Dhcp4Stopped, Dhcp4Init, Dhcp4InitReboot, or Dhcp4Bound state; - Or onother instance of this EFI DHCPv4 Protocol driver is already + Or another instance of this EFI DHCPv4 Protocol driver is already in a valid configured state. @retval EFI_INVALID_PARAMETER Some parameter is NULL. @retval EFI_OUT_OF_RESOURCES Required system resources could not be allocated. @@ -611,7 +611,7 @@ DhcpYieldControl ( not NULL. Otherwise, the state was left unchanged. @retval EFI_ACCESS_DENIED This instance of the EFI DHCPv4 Protocol driver was not in the Dhcp4Stopped, Dhcp4Init, Dhcp4InitReboot, or Dhcp4Bound state; - Or onother instance of this EFI DHCPv4 Protocol driver is already + Or another instance of this EFI DHCPv4 Protocol driver is already in a valid configured state. @retval EFI_INVALID_PARAMETER Some parameter is NULL. @retval EFI_OUT_OF_RESOURCES Required system resources could not be allocated. @@ -805,7 +805,7 @@ EfiDhcp4Start ( } // - // Check Media Satus. + // Check Media Status. // MediaStatus = EFI_SUCCESS; NetLibDetectMediaWaitTimeout (DhcpSb->Controller, DHCP_CHECK_MEDIA_WAITING_TIME, &MediaStatus); diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Io.c b/NetworkPkg/Dhcp4Dxe/Dhcp4Io.c index 4728b94c58..34c72d690a 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Io.c +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Io.c @@ -604,7 +604,7 @@ DhcpEndSession ( is, all the option value that we care. @retval EFI_SUCCESS The packet is successfully processed. - @retval Others Some error occured. + @retval Others Some error occurred. **/ EFI_STATUS @@ -632,7 +632,7 @@ DhcpHandleSelect ( // // Call the user's callback. The action according to the return is as: - // 1. EFI_SUCESS: stop waiting for more offers, select the offer now + // 1. EFI_SUCCESS: stop waiting for more offers, select the offer now // 2. EFI_NOT_READY: wait for more offers // 3. EFI_ABORTED: abort the address acquiring. // @@ -679,7 +679,7 @@ ON_EXIT: is, all the option value that we care. @retval EFI_SUCCESS The packet is successfully processed. - @retval Others Some error occured. + @retval Others Some error occurred. **/ EFI_STATUS @@ -772,7 +772,7 @@ ON_EXIT: is, all the option value that we care. @retval EFI_SUCCESS The packet is successfully processed. - @retval Others Some error occured. + @retval Others Some error occurred. **/ EFI_STATUS @@ -856,7 +856,7 @@ ON_EXIT: is, all the option value that we care. @retval EFI_SUCCESS The packet is successfully processed. - @retval Others Some error occured. + @retval Others Some error occurred. **/ EFI_STATUS @@ -1473,7 +1473,7 @@ DhcpRetransmit ( /** Each DHCP service has three timer. Two of them are count down timer. One for the packet retransmission. The other is to collect the offers. - The third timer increaments the lease life which is compared to T1, T2, + The third timer increments the lease life which is compared to T1, T2, and lease to determine the time to renew and rebind the lease. DhcpOnTimerTick will be called once every second. diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Io.h b/NetworkPkg/Dhcp4Dxe/Dhcp4Io.h index 01283e5e17..0dd6b46b87 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Io.h +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Io.h @@ -108,7 +108,7 @@ DhcpSendMessage ( /** Each DHCP service has three timer. Two of them are count down timer. One for the packet retransmission. The other is to collect the offers. - The third timer increaments the lease life which is compared to T1, T2, + The third timer increments the lease life which is compared to T1, T2, and lease to determine the time to renew and rebind the lease. DhcpOnTimerTick will be called once every second. diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Option.c b/NetworkPkg/Dhcp4Dxe/Dhcp4Option.c index 3b077de111..37a63022c5 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Option.c +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Option.c @@ -195,7 +195,7 @@ DhcpOptionIsValid ( } // - // Validate the occurance of the option unit is with in [MinOccur, MaxOccur] + // Validate the occurrence of the option unit is with in [MinOccur, MaxOccur] // Occur = Len / Unit; @@ -230,7 +230,7 @@ DhcpOptionIsValid ( @param[out] Para The variable to save the interested parameter @retval EFI_SUCCESS The DHCP option is successfully extracted. - @retval EFI_INVALID_PARAMETER The DHCP option is malformatted + @retval EFI_INVALID_PARAMETER The DHCP option is mal-formatted **/ EFI_STATUS @@ -303,7 +303,7 @@ DhcpGetParameter ( option. @retval EFI_SUCCESS All the options are valid - @retval EFI_INVALID_PARAMETER The options are malformatted. + @retval EFI_INVALID_PARAMETER The options are mal-formatted. **/ EFI_STATUS @@ -376,8 +376,8 @@ DhcpIterateBufferOptions ( found @param[in] Context The opaque parameter for Check - @retval EFI_SUCCESS The DHCP packet's options are well formated - @retval EFI_INVALID_PARAMETER The DHCP packet's options are not well formated + @retval EFI_SUCCESS The DHCP packet's options are well formatted + @retval EFI_INVALID_PARAMETER The DHCP packet's options are not well formatted **/ EFI_STATUS @@ -438,7 +438,7 @@ DhcpIterateOptions ( /** Call back function to DhcpIterateOptions to compute each option's - length. It just adds the data length of all the occurances of this + length. It just adds the data length of all the occurrences of this Tag. Context is an array of 256 DHCP_OPTION_COUNT. @param[in] Tag The current option to check @@ -473,7 +473,7 @@ DhcpGetOptionLen ( @param[in] Tag The option to consolidate its data @param[in] Len The length of option data - @param[in] Data The data of the option's current occurance + @param[in] Data The data of the option's current occurrence @param[in] Context The context, which is DHCP_OPTION_CONTEXT. This array is just a wrap to pass THREE parameters. @@ -517,8 +517,8 @@ DhcpFillOption ( /** Parse the options of a DHCP packet. It supports RFC 3396: Encoding Long Options in DHCP. That is, it will combine all the option value - of all the occurances of each option. - A little bit of implemenation: + of all the occurrences of each option. + A little bit of implementation: It adopts the "Key indexed counting" algorithm. First, it allocates an array of 256 DHCP_OPTION_COUNTs because DHCP option tag is encoded as a UINT8. It then iterates the DHCP packet to get data length of @@ -538,7 +538,7 @@ DhcpFillOption ( @retval EFI_NOT_FOUND Cannot find any option. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory to parse the packet. - @retval EFI_INVALID_PARAMETER The options are malformatted + @retval EFI_INVALID_PARAMETER The options are mal-formatted @retval EFI_SUCCESS The options are parsed into OptionPoint **/ @@ -639,7 +639,7 @@ ON_EXIT: @param[out] Para The variable to save the DHCP parameters. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory to validate the packet. - @retval EFI_INVALID_PARAMETER The options are malformatted + @retval EFI_INVALID_PARAMETER The options are mal-formatted @retval EFI_SUCCESS The options are parsed into OptionPoint **/ @@ -770,7 +770,7 @@ DhcpAppendOption ( function. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory - @retval EFI_INVALID_PARAMETER The options in SeekPacket are malformatted + @retval EFI_INVALID_PARAMETER The options in SeekPacket are mal-formatted @retval EFI_SUCCESS The packet is build. **/ @@ -794,7 +794,7 @@ DhcpBuild ( UINT8 *Buf; // - // Use an array of DHCP_OPTION to mark the existance + // Use an array of DHCP_OPTION to mark the existence // and position of each valid options. // Mark = AllocatePool (sizeof (DHCP_OPTION) * DHCP_MAX_OPTIONS); diff --git a/NetworkPkg/Dhcp4Dxe/Dhcp4Option.h b/NetworkPkg/Dhcp4Dxe/Dhcp4Option.h index 393176c899..a58bb1b0ba 100644 --- a/NetworkPkg/Dhcp4Dxe/Dhcp4Option.h +++ b/NetworkPkg/Dhcp4Dxe/Dhcp4Option.h @@ -81,9 +81,9 @@ typedef struct { /// /// Structure used to describe and validate the format of DHCP options. /// Type is the options' data type, such as DHCP_OPTION_INT8. MinOccur -/// is the minium occurance of this data type. MaxOccur is defined +/// is the minimum occurrence of this data type. MaxOccur is defined /// similarly. If MaxOccur is -1, it means that there is no limit on the -/// maximum occurance. Alert tells whether DHCP client should further +/// maximum occurrence. Alert tells whether DHCP client should further /// inspect the option to parse DHCP_PARAMETER. /// typedef struct { @@ -114,8 +114,8 @@ EFI_STATUS found @param[in] Context The opaque parameter for Check - @retval EFI_SUCCESS The DHCP packet's options are well formated - @retval EFI_INVALID_PARAMETER The DHCP packet's options are not well formated + @retval EFI_SUCCESS The DHCP packet's options are well formatted + @retval EFI_INVALID_PARAMETER The DHCP packet's options are not well formatted **/ EFI_STATUS @@ -133,7 +133,7 @@ DhcpIterateOptions ( @param[out] Para The variable to save the DHCP parameters. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory to validate the packet. - @retval EFI_INVALID_PARAMETER The options are malformatted + @retval EFI_INVALID_PARAMETER The options are mal-formatted @retval EFI_SUCCESS The options are parsed into OptionPoint **/ @@ -146,8 +146,8 @@ DhcpValidateOptions ( /** Parse the options of a DHCP packet. It supports RFC 3396: Encoding Long Options in DHCP. That is, it will combine all the option value - of all the occurances of each option. - A little bit of implemenation: + of all the occurrences of each option. + A little bit of implementation: It adopts the "Key indexed counting" algorithm. First, it allocates an array of 256 DHCP_OPTION_COUNTs because DHCP option tag is encoded as a UINT8. It then iterates the DHCP packet to get data length of @@ -167,7 +167,7 @@ DhcpValidateOptions ( @retval EFI_NOT_FOUND Cannot find any option. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory to parse the packet. - @retval EFI_INVALID_PARAMETER The options are malformatted + @retval EFI_INVALID_PARAMETER The options are mal-formatted @retval EFI_SUCCESS The options are parsed into OptionPoint **/ @@ -211,7 +211,7 @@ DhcpAppendOption ( function. @retval EFI_OUT_OF_RESOURCES Failed to allocate memory - @retval EFI_INVALID_PARAMETER The options in SeekPacket are malformatted + @retval EFI_INVALID_PARAMETER The options in SeekPacket are mal-formatted @retval EFI_SUCCESS The packet is build. **/ -- 2.39.2