Read OTP field in DA14585

I am trying to migrate an old DA14580 project to the DA14585:

The old code:

int cnt = 100000;
SetBits16(CLK_AMBA_REG, OTP_ENABLE, 1); // enable OTP clock
while ((GetWord16(ANA_STATUS_REG) & LDO_OTP_OK) != LDO_OTP_OK && cnt--)
// set OTP in read mode
SetWord32(OTPC_MODE_REG, 0x1);
uint8_t *otp_major = (uint8_t *)0x47F54;
uint8_t *otp_minor = (uint8_t *)0x47F56;
memcpy(&user_beacon_config.major_ALT_val1, otp_major, 2);
memcpy(&user_beacon_config.minor_ALT_val2, otp_minor, 2);
SetBits16(CLK_AMBA_REG, OTP_ENABLE, 0); //disable OTP clock

The new code:

int cnt = 100000;
SetBits16(CLK_AMBA_REG, OTP_ENABLE, 1); // enable OTP clock
while ((GetWord16(ANA_STATUS_REG) & LDO_VDD_OK) != LDO_VDD_OK && cnt--)
// set OTP in read mode
SetWord32(OTPC_MODE_REG, 0x1);
uint8_t *otp_major = (uint8_t *)0x47F54;
uint8_t *otp_minor = (uint8_t *)0x47F56;
memcpy (&beacon_data[主要],otp_major, 2);
memcpy(&beacon_data[MINOR], otp_minor, 2);
SetBits16(CLK_AMBA_REG, OTP_ENABLE, 0); //disable OTP clock

Is it correct to use LDO_VDD_OK instead of LDO_OTP_OK?

I think the DA14585 has a larger OTP size, does the memory location still works ok?

Thank you