on the first post i think i made a mistake with Defines.

Is this the correct way to assign a constant value to the name?

Code:
;-------------------------------------------------------------|
;------------------------ max30102 registers------------------|
;-------------------------------------------------------------|
I2C_WRITE_ADDR con  $AE     ; b10101110
I2C_READ_ADDR con   $AF     ; b10101111
I2C_ID_ADDR  con    $57     ; 7-bit version of the above
;I2C_ID_ADDR con  57h ; 7-bit version of the above

;//register addresses
;---------------------- STATUS REGISTER -----------------------
REG_INTR_STATUS_1 con   $00
REG_INTR_STATUS_2 con   $01

REG_INTR_ENABLE_1 con   $02
REG_INTR_ENABLE_2 con   $03

;----------------------- FIFO REGISTERS -----------------------
REG_FIFO_WR_PTR con     $04   ; FIFO WRITE POINTER REG ADDR IS 0X04
REG_OVF_COUNTER con     $05
REG_FIFO_RD_PTR con     $06
REG_FIFO_DATA con       $07

;---------------------- CONFIG REGISTER -----------------------
REG_FIFO_CONFIG con     $08
REG_MODE_CONFIG con     $09
REG_SPO2_CONFIG con     $0A
REG_LED1_PA con         $0C
REG_LED2_PA con         $0D


REG_PILOT_PA con        $10
REG_MULTI_LED_CTRL1 con $11
REG_MULTI_LED_CTRL2 con $12

;----------------------- TEMP REGISTER -------------------------
REG_TEMP_INTR con       $1F
REG_TEMP_FRAC con       $20
REG_TEMP_CONFIG con     $21
REG_PROX_INT_THRESH con $30

;----------------------- PART ID REGISTER ----------------------
REG_REV_ID  con         $FE
REG_PART_ID  con        $FF