On 18-04-2025 07:59, Longbin Li wrote: > +#define SG2044_REG_POLARITY 0x40 > +#define SG2044_REG_PWMSTART 0x44 > +#define SG2044_REG_PWM_OE 0xD0 > + please use lowercase 0xd0 > #define SG2042_PWM_HLPERIOD(chan) ((chan) * 8 + 0) > #define SG2042_PWM_PERIOD(chan) ((chan) * 8 + 4) Thanks, Alok