From: "Leif Lindholm" <leif.lindholm@linaro.org>
To: Masahisa Kojima <masahisa.kojima@linaro.org>
Cc: devel@edk2.groups.io, ard.biesheuvel@linaro.org,
okamoto.satoru@socionext.com
Subject: Re: [PATCH edk2-platforms v2 1/3] NetsecDxe: embed phy address into NETSEC SDK internal structure
Date: Tue, 6 Aug 2019 16:57:06 +0100 [thread overview]
Message-ID: <20190806155706.GI25813@bivouac.eciton.net> (raw)
In-Reply-To: <20190725011100.10176-2-masahisa.kojima@linaro.org>
On Thu, Jul 25, 2019 at 10:10:58AM +0900, Masahisa Kojima wrote:
> This is a refactoring of phy address handling in Netsec driver.
> NETSEC SDK, low level driver for NetsecDxe, did not store phy address.
> User should specify the phy address as an argument to
> the SDK public functions.
> It prevented NETSEC SDK from internally controlling phy,
> and it also bothers user application with phy address management.
>
> With that, we encapsulate the phy address into NETSEC SDK.
>
> Signed-off-by: Masahisa Kojima <masahisa.kojima@linaro.org>
> Signed-off-by: Satoru Okamoto <okamoto.satoru@socionext.com>
Oh, I just noticed this when I was about to push the patches.
(I am happy with the code and commit messages.)
There should only be your own Signed-off-by on patches you submit to
the mailing list. If Okamoto-san did was the majority author of the
patches, he should be listed as the author (indicated by a From:
statement in the patch, generated from the git authorship
information).
Best Regards,
Leif
> Reviewed-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.c | 10 ++--
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.h | 2 -
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/include/ogma_api.h | 6 +-
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_gmac_access.c | 61 +++++---------------
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_internal.h | 2 +
> Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_misc.c | 6 ++
> 6 files changed, 28 insertions(+), 59 deletions(-)
>
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.c b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.c
> index 160bb08a4632..0b91d4af44a3 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.c
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.c
> @@ -59,6 +59,8 @@ Probe (
> // phy-interface
> Param.gmac_config.phy_interface = OGMA_PHY_INTERFACE_RGMII;
>
> + Param.phy_addr = LanDriver->Dev->Resources[2].AddrRangeMin;
> +
> // Read and save the Permanent MAC Address
> EepromBase = LanDriver->Dev->Resources[1].AddrRangeMin;
> GetCurrentMacAddress (EepromBase, LanDriver->SnpMode.PermanentAddress.Addr);
> @@ -107,8 +109,6 @@ Probe (
> return EFI_DEVICE_ERROR;
> }
>
> - LanDriver->PhyAddress = LanDriver->Dev->Resources[2].AddrRangeMin;
> -
> ogma_enable_top_irq (LanDriver->Handle,
> OGMA_TOP_IRQ_REG_NRM_RX | OGMA_TOP_IRQ_REG_NRM_TX);
>
> @@ -280,7 +280,7 @@ SnpInitialize (
> ReturnUnlock (EFI_DEVICE_ERROR);
> }
>
> - ogma_err = ogma_get_phy_link_status (LanDriver->Handle, LanDriver->PhyAddress,
> + ogma_err = ogma_get_phy_link_status (LanDriver->Handle,
> &phy_link_status);
> if (ogma_err != OGMA_ERR_OK) {
> DEBUG ((DEBUG_ERROR,
> @@ -438,7 +438,7 @@ NetsecPollPhyStatus (
> LanDriver = INSTANCE_FROM_SNP_THIS (Snp);
>
> // Update the media status
> - ogma_err = ogma_get_phy_link_status (LanDriver->Handle, LanDriver->PhyAddress,
> + ogma_err = ogma_get_phy_link_status (LanDriver->Handle,
> &phy_link_status);
> if (ogma_err != OGMA_ERR_OK) {
> DEBUG ((DEBUG_ERROR,
> @@ -662,7 +662,7 @@ SnpGetStatus (
> LanDriver = INSTANCE_FROM_SNP_THIS (Snp);
>
> // Update the media status
> - ogma_err = ogma_get_phy_link_status (LanDriver->Handle, LanDriver->PhyAddress,
> + ogma_err = ogma_get_phy_link_status (LanDriver->Handle,
> &phy_link_status);
> if (ogma_err != OGMA_ERR_OK) {
> DEBUG ((DEBUG_ERROR,
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.h b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.h
> index 870833c8d31c..c95ff215199d 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.h
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/NetsecDxe.h
> @@ -70,8 +70,6 @@ typedef struct {
> NON_DISCOVERABLE_DEVICE *Dev;
>
> NETSEC_DEVICE_PATH DevicePath;
> -
> - UINTN PhyAddress;
> } NETSEC_DRIVER;
>
> #define NETSEC_SIGNATURE SIGNATURE_32('n', 't', 's', 'c')
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/include/ogma_api.h b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/include/ogma_api.h
> index 66f39150430b..be80dd9ae1fd 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/include/ogma_api.h
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/include/ogma_api.h
> @@ -318,6 +318,7 @@ struct ogma_param_s{
> ogma_desc_ring_param_t desc_ring_param[OGMA_DESC_RING_ID_MAX+1];
> ogma_gmac_config_t gmac_config;
> ogma_uint8 mac_addr[6];
> + ogma_uint8 phy_addr;
> };
>
> struct ogma_tx_pkt_ctrl_s{
> @@ -412,14 +413,12 @@ ogma_err_t ogma_set_gmac_mode (
>
> void ogma_set_phy_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr,
> ogma_uint16 value
> );
>
> ogma_uint16 ogma_get_phy_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr
> );
>
> @@ -660,7 +659,6 @@ ogma_err_t ogma_get_gmac_lpitimer_reg (
>
> void ogma_set_phy_mmd_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr,
> ogma_uint16 value
> @@ -668,14 +666,12 @@ void ogma_set_phy_mmd_reg (
>
> ogma_uint16 ogma_get_phy_mmd_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> );
>
> ogma_err_t ogma_get_phy_link_status (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_phy_link_status_t *phy_link_status_p
> );
>
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_gmac_access.c b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_gmac_access.c
> index 88c149c10466..150d25ac3fbf 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_gmac_access.c
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_gmac_access.c
> @@ -40,14 +40,12 @@
> **********************************************************************/
> static void ogma_set_phy_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr,
> ogma_uint16 value
> );
>
> static ogma_uint16 ogma_get_phy_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr
> );
>
> @@ -57,14 +55,12 @@ void ogma_dump_gmac_stat (ogma_ctrl_t *ctrl_p);
>
> static void ogma_set_phy_target_mmd_reg_addr (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> );
>
> static void ogma_set_phy_mmd_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr,
> ogma_uint16 value
> @@ -72,7 +68,6 @@ static void ogma_set_phy_mmd_reg_sub (
>
> static ogma_uint16 ogma_get_phy_mmd_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> );
> @@ -435,7 +430,6 @@ ogma_err_t ogma_set_gmac_mode (
>
> static void ogma_set_phy_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr,
> ogma_uint16 value
> )
> @@ -447,7 +441,7 @@ static void ogma_set_phy_reg_sub (
> OGMA_GMAC_REG_ADDR_GDR,
> value);
>
> - cmd = ( ( phy_addr << OGMA_GMAC_GAR_REG_SHIFT_PA) |
> + cmd = ( ( ctrl_p->param.phy_addr << OGMA_GMAC_GAR_REG_SHIFT_PA) |
> ( reg_addr << OGMA_GMAC_GAR_REG_SHIFT_GR) |
> ( OGMA_CLOCK_RANGE_IDX << OGMA_GMAC_GAR_REG_SHIFT_CR) |
> OGMA_GMAC_GAR_REG_GW |
> @@ -466,7 +460,6 @@ static void ogma_set_phy_reg_sub (
>
> void ogma_set_phy_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr,
> ogma_uint16 value
> )
> @@ -476,27 +469,25 @@ void ogma_set_phy_reg (
>
> if (( ctrl_p == NULL)
> || ( !ctrl_p->param.use_gmac_flag)
> - || ( phy_addr >= 32)
> || ( reg_addr >= 32) ) {
> pfdep_print( PFDEP_DEBUG_LEVEL_FATAL,
> "An error occurred at ogma_set_phy_reg.\nPlease set valid argument.\n");
> return;
> }
>
> - ogma_set_phy_reg_sub( ctrl_p, phy_addr, reg_addr, value);
> + ogma_set_phy_reg_sub( ctrl_p, reg_addr, value);
>
> }
>
> static ogma_uint16 ogma_get_phy_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr
> )
> {
>
> ogma_uint32 cmd;
>
> - cmd = ( ( phy_addr << OGMA_GMAC_GAR_REG_SHIFT_PA) |
> + cmd = ( ( ctrl_p->param.phy_addr << OGMA_GMAC_GAR_REG_SHIFT_PA) |
> ( reg_addr << OGMA_GMAC_GAR_REG_SHIFT_GR) |
> ( OGMA_CLOCK_RANGE_IDX << OGMA_GMAC_GAR_REG_SHIFT_CR) |
> OGMA_GMAC_GAR_REG_GB);
> @@ -516,7 +507,6 @@ static ogma_uint16 ogma_get_phy_reg_sub (
>
> ogma_uint16 ogma_get_phy_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 reg_addr
> )
> {
> @@ -525,14 +515,13 @@ ogma_uint16 ogma_get_phy_reg (
>
> if ( ( ctrl_p == NULL)
> || ( !ctrl_p->param.use_gmac_flag)
> - || ( phy_addr >= 32)
> || ( reg_addr >= 32) ) {
> pfdep_print( PFDEP_DEBUG_LEVEL_FATAL,
> "An error occurred at ogma_get_phy_reg.\nPlease set valid argument.\n");
> return 0;
> }
>
> - value = ogma_get_phy_reg_sub(ctrl_p, phy_addr, reg_addr);
> + value = ogma_get_phy_reg_sub(ctrl_p, reg_addr);
>
>
> return value;
> @@ -702,7 +691,6 @@ ogma_err_t ogma_get_gmac_status (
>
> static void ogma_set_phy_target_mmd_reg_addr (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> )
> @@ -713,21 +701,20 @@ static void ogma_set_phy_target_mmd_reg_addr (
> cmd = ( ogma_uint32)dev_addr;
>
> /*set command to MMD access control register */
> - ogma_set_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_MMD_AC, cmd);
> + ogma_set_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_MMD_AC, cmd);
>
> /* set MMD access address data register Write reg_addr */
> - ogma_set_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_MMD_AAD, reg_addr);
> + ogma_set_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_MMD_AAD, reg_addr);
>
> /* write value to MMD ADDR */
> cmd = ( (1U << 14) | dev_addr);
>
> /* set command to MMD access control register */
> - ogma_set_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_MMD_AC, cmd);
> + ogma_set_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_MMD_AC, cmd);
> }
>
> static void ogma_set_phy_mmd_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr,
> ogma_uint16 value
> @@ -735,30 +722,27 @@ static void ogma_set_phy_mmd_reg_sub (
> {
> /* set target mmd reg_addr */
> ogma_set_phy_target_mmd_reg_addr( ctrl_p,
> - phy_addr,
> dev_addr,
> reg_addr);
>
> /* Write value to MMD access address data register */
> - ogma_set_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_MMD_AAD, value);
> + ogma_set_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_MMD_AAD, value);
>
> }
>
> static ogma_uint16 ogma_get_phy_mmd_reg_sub (
> ogma_ctrl_t *ctrl_p,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> )
> {
> /* set target mmd reg_addr */
> ogma_set_phy_target_mmd_reg_addr( ctrl_p,
> - phy_addr,
> dev_addr,
> reg_addr);
>
> /* Read value for MMD access address data register */
> - return ogma_get_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_MMD_AAD);
> + return ogma_get_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_MMD_AAD);
> }
>
> ogma_err_t ogma_set_gmac_lpictrl_reg (
> @@ -878,7 +862,6 @@ ogma_err_t ogma_get_gmac_lpitimer_reg (
>
> void ogma_set_phy_mmd_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr,
> ogma_uint16 value
> @@ -890,8 +873,7 @@ void ogma_set_phy_mmd_reg (
> return;
> }
>
> - if ( ( phy_addr > 31U) ||
> - ( dev_addr > 31U) ) {
> + if ( dev_addr > 31U) {
> return;
> }
>
> @@ -900,7 +882,6 @@ void ogma_set_phy_mmd_reg (
> }
>
> ogma_set_phy_mmd_reg_sub ( ctrl_p,
> - phy_addr,
> dev_addr,
> reg_addr,
> value);
> @@ -911,7 +892,6 @@ void ogma_set_phy_mmd_reg (
>
> ogma_uint16 ogma_get_phy_mmd_reg (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_uint8 dev_addr,
> ogma_uint16 reg_addr
> )
> @@ -923,8 +903,7 @@ ogma_uint16 ogma_get_phy_mmd_reg (
> return 0;
> }
>
> - if ( ( phy_addr > 31U) ||
> - ( dev_addr > 31U) ) {
> + if ( dev_addr > 31U) {
> return 0;
> }
>
> @@ -933,7 +912,6 @@ ogma_uint16 ogma_get_phy_mmd_reg (
> }
>
> value = ogma_get_phy_mmd_reg_sub ( ctrl_p,
> - phy_addr,
> dev_addr,
> reg_addr);
>
> @@ -943,7 +921,6 @@ ogma_uint16 ogma_get_phy_mmd_reg (
>
> ogma_err_t ogma_get_phy_link_status (
> ogma_handle_t ogma_handle,
> - ogma_uint8 phy_addr,
> ogma_phy_link_status_t *phy_link_status_p
> )
> {
> @@ -955,10 +932,6 @@ ogma_err_t ogma_get_phy_link_status (
> return OGMA_ERR_PARAM;
> }
>
> - if ( phy_addr >= 32) {
> - return OGMA_ERR_RANGE;
> - }
> -
> if ( !ctrl_p->param.use_gmac_flag) {
> return OGMA_ERR_NOTAVAIL;
> }
> @@ -966,17 +939,17 @@ ogma_err_t ogma_get_phy_link_status (
> pfdep_memset( phy_link_status_p, 0, sizeof( ogma_phy_link_status_t) );
>
> /* Read PHY CONTROL Register */
> - tmp = ogma_get_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_CONTROL);
> + tmp = ogma_get_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_CONTROL);
>
> /* Read PHY STATUS Register */
> - value = ogma_get_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_STATUS);
> + value = ogma_get_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_STATUS);
>
> /* check latched_link_down_flag */
> if ( ( value & ( 1U << OGMA_PHY_STATUS_REG_LINK_STATUS) ) == 0) {
> phy_link_status_p->latched_link_down_flag = OGMA_TRUE;
>
> /* Read PHY STATUS Register */
> - value = ogma_get_phy_reg_sub( ctrl_p, phy_addr, OGMA_PHY_REG_ADDR_STATUS);
> + value = ogma_get_phy_reg_sub( ctrl_p, OGMA_PHY_REG_ADDR_STATUS);
>
> }
>
> @@ -1036,12 +1009,10 @@ ogma_err_t ogma_get_phy_link_status (
>
> /* Read MASTER-SLAVE Control Register */
> value = ogma_get_phy_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_REG_ADDR_MASTER_SLAVE_CONTROL);
>
> /* Read MASTER-SLAVE Status Register */
> tmp = ogma_get_phy_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_REG_ADDR_MASTER_SLAVE_STATUS);
>
> /* Check Current Link Speed */
> @@ -1061,12 +1032,10 @@ ogma_err_t ogma_get_phy_link_status (
>
> /* Read Auto-Negotiation Advertisement register */
> value = ogma_get_phy_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_REG_ADDR_AUTO_NEGO_ABILTY);
>
> /* Read Auto-Negotiation Link Partner Base Page Ability register */
> tmp = ogma_get_phy_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_REG_ADDR_AUTO_NEGO_LINK_PATNER_ABILTY);
>
> value = ( ( ( value & tmp) >> OGMA_PHY_ANA_REG_TAF) &
> @@ -1109,13 +1078,11 @@ ogma_err_t ogma_get_phy_link_status (
>
> /* Read EEE advertisement register */
> value = ogma_get_phy_mmd_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_DEV_ADDR_AUTO_NEGO,
> OGMA_PHY_AUTO_NEGO_REG_ADDR_EEE_ADVERTISE);
>
> /* Read EEE link partner ability register */
> tmp = ogma_get_phy_mmd_reg_sub( ctrl_p,
> - phy_addr,
> OGMA_PHY_DEV_ADDR_AUTO_NEGO,
> OGMA_PHY_AUTO_NEGO_REG_ADDR_EEE_LP_ABILITY);
>
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_internal.h b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_internal.h
> index ed09a7ada85d..a7bc69cf0777 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_internal.h
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_internal.h
> @@ -111,6 +111,8 @@ struct ogma_ctrl_s{
>
> pfdep_phys_addr_t dummy_desc_entry_phys_addr;
>
> + ogma_uint8 phy_addr;
> +
> #ifdef OGMA_CONFIG_REC_STAT
> /**
> * Statistics information.
> diff --git a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_misc.c b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_misc.c
> index 4dec66313aa1..7481d2da2d24 100644
> --- a/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_misc.c
> +++ b/Silicon/Socionext/SynQuacer/Drivers/Net/NetsecDxe/netsec_for_uefi/netsec_sdk/src/ogma_misc.c
> @@ -388,6 +388,12 @@ ogma_err_t ogma_init (
> return OGMA_ERR_DATA;
> }
>
> + if ( param_p->phy_addr >= 32) {
> + pfdep_print( PFDEP_DEBUG_LEVEL_FATAL,
> + "Error: phy_addr out of range\n");
> + return OGMA_ERR_DATA;
> + }
> +
> ogma_err = ogma_probe_hardware( base_addr);
>
> if ( ogma_err != OGMA_ERR_OK) {
> --
> 2.17.1
>
next prev parent reply other threads:[~2019-08-06 15:57 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-07-25 1:10 [PATCH edk2-platforms v2 0/3] Robust Netsec Initialiation Masahisa Kojima
2019-07-25 1:10 ` [PATCH edk2-platforms v2 1/3] NetsecDxe: embed phy address into NETSEC SDK internal structure Masahisa Kojima
2019-08-06 15:57 ` Leif Lindholm [this message]
2019-07-25 1:10 ` [PATCH edk2-platforms v2 2/3] NetsecDxe: put phy in loopback mode to guarantee stable RXCLK input Masahisa Kojima
2019-07-25 1:11 ` [PATCH edk2-platforms v2 3/3] NetsecDxe: SnpInitialize() waits for media linking up Masahisa Kojima
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20190806155706.GI25813@bivouac.eciton.net \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox