From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH TRIVIAL v2] gpu: Fix Kconfig indentation Date: Tue, 08 Oct 2019 09:05:49 +0300 Message-ID: <87v9szdb5u.fsf@intel.com> References: <20191004144549.3567-1-krzk@kernel.org> <87sgo4hjii.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain Return-path: In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Krzysztof Kozlowski , Alex Deucher Cc: Maarten Lankhorst , Maxime Ripard , Sean Paul , David Airlie , Daniel Vetter , Jiri Kosina , Maling list - DRI developers , LKML , amd-gfx list , Intel Graphics Development , lima@lists.freedesktop.org, nouveau , linux-arm-kernel , linux-rockchip@lists.infradead.org List-Id: linux-rockchip.vger.kernel.org On Mon, 07 Oct 2019, Krzysztof Kozlowski wrote: > On Mon, 7 Oct 2019 at 18:09, Alex Deucher wrote: >> >> On Mon, Oct 7, 2019 at 7:39 AM Jani Nikula wrote: >> > >> > On Fri, 04 Oct 2019, Krzysztof Kozlowski wrote: >> > > drivers/gpu/drm/i915/Kconfig | 12 +- >> > > drivers/gpu/drm/i915/Kconfig.debug | 144 +++++++++++------------ >> > >> > Please split these out to a separate patch. Can't speak for others, but >> > the patch looks like it'll be conflicts galore and a problem to manage >> > if merged in one big lump. >> >> Yes, it would be nice to have the amd patch separate as well. > > I'll split the AMD and i915 although I am not sure if it is sense to > split such trivial patch per each driver. Thanks. See MAINTAINERS, many of the drivers are maintained in the same drm-misc repo, and it makes no difference to split those. In general it's, well, trivial to split up patches like this per driver or repo, but not splitting it up generates extra busywork in managing conflicts until some common merge/backmerge happens. We just want to apply the patch and forget about it, instead of dealing with a trivial whitespace cleanup many times over. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center