On 20/06/2022 17:35, Krzysztof Kozlowski wrote: > On 14/06/2022 03:30, Wangseok Lee wrote: >> Add support Axis, ARTPEC-8 SoC. >> ARTPEC-8 is the SoC platform of Axis Communications. >> > > (...) > >> + >> +enum artpec8_pcie_isolation { >> + PCIE_CLEAR_ISOLATION = 0, >> + PCIE_SET_ISOLATION = 1 >> +}; >> + >> +enum artpec8_pcie_reg_bit { >> + PCIE_REG_BIT_LOW = 0, >> + PCIE_REG_BIT_HIGH = 1 >> +}; > > Remove this enum. This is redefinition of true/false or 1/0. > > Best regards, > Krzysztof ok, i will change the above enum types to boolean. Thank you for kindness reivew. Best regards, Wangseok Lee