On Wed, Apr 10, 2024 at 01:28:06AM +0800, Yuguo Pei wrote: > On Tue, 9 Apr 2024 17:49:25 +0200, Greg KH wrote: > > So you are now hard-coding the size? > > Yes, the change just helps developers who want to adapt to different screen sizes. So there is no change? I don't understand. > > Is this always going to be 0, if so, why need it at all? > > Not sure. For example, when HEIGHT and WIDTH are 240 and 280, LEFT_OFFSET is 20 and TOP_OFFSET is 0. Different screens may have different offsets. Then why is this needed? Don't make changes that are not needed by anyone, especially for code in drivers/staging/ > > In addition, do I need to resend the patch to modify the Signed-off-by? Of course you do :)