On Wed, Mar 06, 2024 at 01:33:28AM -0800, Changhuang Liang wrote: > current_fmt only can store one pad format, when setting other pad it > will be overwrote. Replaced it with get from sd_state directly. > These commit descriptions are kind of hard to understand so I have proposed new commit messages. Subject: staging: media: starfive: Get rid of current_fmt We want to support mutiple formats so saving one "current_fmt" doesn't work. This was only used to set the ISP_REG_STRIDE so use the sd_state directly for that and delete the ->current_fmt pointer. No functional change. regards, dan carpenter