Re: [PATCH v3 7/8] mikrobus: Add mikrobus driver

From: Ayush Singh <ayushdevel1325@gmail.com>
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
	linux-kernel@vger.kernel.org
Cc: jkridner@beagleboard.org, robertcnelson@beagleboard.org,
	Vaishnav M A <vaishnav@beagleboard.org>,
	Rob Herring <robh@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>, Nishanth Menon <nm@ti.com>,
	Vignesh Raghavendra <vigneshr@ti.com>,
	Tero Kristo <kristo@kernel.org>,
	Derek Kiernan <derek.kiernan@amd.com>,
	Dragan Cvetic <dragan.cvetic@amd.com>,
	Arnd Bergmann <arnd@arndb.de>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Liam Girdwood <lgirdwood@gmail.com>,
	Mark Brown <broonie@kernel.org>,
	Jiri Slaby <jirislaby@kernel.org>,
	Johan Hovold <johan@kernel.org>, Alex Elder <elder@kernel.org>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-spi@vger.kernel.org, linux-serial@vger.kernel.org,
	greybus-dev@lists.linaro.org
Subject: Re: [PATCH v3 7/8] mikrobus: Add mikrobus driver
Date: Sat, 16 Mar 2024 18:36:38 +0530	[thread overview]
Message-ID: <402d1296-0a0c-4f85-a096-be7993869f94@gmail.com> (raw)
In-Reply-To: <8799b216-57a7-451b-80a3-3d4ae9693e0b@linaro.org>

 > Are you sure this fits in Linux coding style limit (not checkpatch 
limit, but the limit expressed by Linux coding style)?

Well, I am just using clang-format with column width of 100 instead of 
80. The docs now say 80 is prefered rather than mandatory, so well I was 
using 100 since I prefer that. If 80 is necessary or would make review 
easier than I can just switch to it.

I will remove serdev, pwm, clickID and send a new patch with the minimal 
driver and better commit messages as suggested with Vaishnav. It is 
important to have good support for mikroBUS boards without clickID as well.

Thanks for all your feedback and time.

Ayush Singh