Re: [PATCH] staging: fbtft: remove unnecessary braces in if statements and space before close parenthesis in function call

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

 



On Tue, Jul 23, 2024 at 06:28:19PM +0530, Riyan Dhiman wrote:
> Hi,
> 
> Thank you for your feedback on my patch submission.
> I have reviewed the mailing list traffic and noted that the last patch
> addressing the same coding style issue was submitted in April. Given this
> timeframe, I believed it was appropriate to submit my patch to contribute
> to the resolution of the ongoing issue.
> As this is my first contribution to the Linux kernel, I want to ensure that
> my submission aligns with the requirements. I would greatly appreciate your
> guidance on how to improve my patch and avoid duplicating previous efforts.

The issue is that the checkpatch warning for too many () characters is
just not right, and shouldn't be applied for staging fixes, sorry.  The
code is fine as-is.

thanks,

greg k-h



[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux