Re: [PATCH] staging: fbtft: fb_st7789v: support setting offset

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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 :)





[Index of Archives]     [Video for Linux]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Tourism]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux