From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f52.google.com (mail-wm1-f52.google.com [209.85.128.52]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 93C422FAF for ; Wed, 1 Sep 2021 21:00:00 +0000 (UTC) Received: by mail-wm1-f52.google.com with SMTP id l7-20020a1c2507000000b002e6be5d86b3so565052wml.3 for ; Wed, 01 Sep 2021 14:00:00 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=philpotter-co-uk.20150623.gappssmtp.com; s=20150623; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=FjeZ4dlBjB70RlRBxV6Srwhf3xdbtXfhDi5m412yaQw=; b=q1biW8AeA7Ooox76RYpSXEBDIRelue7d6rT0ovjCORVu+WZvbmUGIyhKYb/bf61Mwv naCX4uIrZIGWUsb9GjoedI3QYLGIhYEtKRhmrC4Pr0BnhTxcf+zTYceOwcic6a/cmgie 67/dwh0OECt4TT+cH8obFPFSlylryxK24wGSuhKAnxHJsg7CIeVWXVNxFwRoy8HI9OUb w/C/IECjihHd7kZJKNeQQxc+sJna7CS0shfw+DE8Ch5Ob7wB5XVHwMn3Lb3mC4z3QXUa 7/TNHMhBH6U/DrlkSBF+SnACE+BiXWJX16pQklG0sHLLtu0OcDuPFYze/r1vXQB+Ud1h DsTg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=FjeZ4dlBjB70RlRBxV6Srwhf3xdbtXfhDi5m412yaQw=; b=YKuGsTI032l0Pj/hmcIkg/okFriLcUiRKTO7DclfaxrSsDWT1H0VVHTsWLijuGpBrc cTIfBsV/M05zLwZNnlNGG8TlJCGpjJn3QIX3d/4/EMqU8NF6inJ6z9jSN4LkKEEKhGII wSo36VmVuMKHzIFsYeZta/jHnlQDazX0OCucx6dH8QxZlbrZapmK9/kVGTkASbpv1MRy ZGhZnnOzN5ET7x/vr/W6jpmjRVWtQh20Xf8Z2ny7ZWmp3kL0ShVSdBL4esOPrBSes4Wn 23Mu+y84DOzbbfiDGaVWOonwKpbEV4tyQo2YWbZDHWQScBFaMT8uWRjmhrTPYX7F2yCV 0V5w== X-Gm-Message-State: AOAM530qbv8+2nyIrA21oVunfWI7vrWE3ELS3Hb21fz9GteK65Slm77e AN68cw6O+4xl6gpcypTTpvN6/g== X-Google-Smtp-Source: ABdhPJysn9IAfw6kPJJTaw1OmEQXr3M0uQxEHOUHBYTGapBJAsFQVmPx3QfXXoQPLTf3DZhKmasFpw== X-Received: by 2002:a05:600c:210a:: with SMTP id u10mr1267894wml.127.1630529998821; Wed, 01 Sep 2021 13:59:58 -0700 (PDT) Received: from equinox (2.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.0.a.1.e.e.d.f.d.0.b.8.0.1.0.0.2.ip6.arpa. [2001:8b0:dfde:e1a0::2]) by smtp.gmail.com with ESMTPSA id o26sm553804wmc.17.2021.09.01.13.59.57 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 01 Sep 2021 13:59:58 -0700 (PDT) Date: Wed, 1 Sep 2021 21:59:56 +0100 From: Phillip Potter To: Linus Torvalds Cc: gregkh@linuxfoundation.org, arnd@arndb.de, Larry.Finger@lwfinger.net, akpm@linux-foundation.org, sfr@canb.auug.org.au, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev Subject: Re: [GIT PULL] Staging / IIO driver changes for 5.15-rc1 Message-ID: References: Precedence: bulk X-Mailing-List: linux-staging@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, Sep 01, 2021 at 09:49:47AM -0700, Linus Torvalds wrote: > On Wed, Sep 1, 2021 at 7:17 AM Greg KH wrote: > > > > Lots of churn in some staging drivers, we dropped the "old" rtl8188eu > > driver and replaced it with a newer version of the driver that had been > > maintained out-of-tree by Larry with the end goal of actually being able > > to get this driver out of staging eventually. Despite that driver being > > "newer" the line count of this pull request is going up. > > Ugh. > > So this had a conflict with the networking tree, and commit > 89939e890605 ("staging: rtlwifi: use siocdevprivate") in particular. > > Ok, so that conflict looked annoying but harmless - git saw the new > driver as a rename of the old one, and tried to actually apply the > changes from that commit to the new one. > > And git actually did a reasonable job, everything considered. There > were enough similarities that it wasn't entirely crazy, and enough > differences that it caused conflicts. > > HOWEVER. > > Actually then looking at the root causes of the conflicts, as part of > just trying to finish what git had started, I notice that a lot of the > code in the new driver was just completely dead. > > As of commit ae7471cae00a ("staging: r8188eu: remove rtw_ioctl > function") the only caller of rtw_android_priv_cmd() is entirely gone. > > But rtw_android_priv_cmd() was kept around, and was in fact the only > reason that the files > > drivers/staging/r8188eu/include/rtw_android.h > drivers/staging/r8188eu/os_dep/rtw_android.c > > existed at all. > > End result: instead of trying to fix up the conflict in dead code, I > just ripped out the code completely. > > So my merge actually looks simple and clean: when you look at the > conflict diff of my merge, all you see is that I removed > 'os_dep/rtw_android.o' from drivers/staging/r8188eu/Makefile. > > What you don't actually see as a conflict, is that I removed those > files entirely. That removal doesn't show up as "conflicts", because > that filename didn't exist in my HEAD commit before the merge at all > (because my tree had that old "rtl8188eu" driver). > > So this email is just a long explanation for what I did, to make > people aware that maybe I screwed up. It builds for me, and honestly, > it makes sense to me in ways that your git tree did not, but I can't > test any of it. > > Added Arnd (for the siocdevprivate conversion) and Phillip/Larry (for > the r8188eu driver side) - can you please double-check what I did. > > Linus Dear Linus, Thank you for merging the code and resolving the merge conflict firstly, much appreciated. You have not screwed up - the driver still works, I've just built it from your tree and tested this kernel with my USB-N10 Nano, and it works as expected. Indeed, other than the removal of the unused Android code, the driver is identical to the one in staging according to my checks with diff etc. I left this code in when I wrote ae7471cae00a ("staging: r8188eu: remove rtw_ioctl function") with the intention of modifying and reconnecting, once the changes to struct net_device_ops made their way into the staging tree. Removal of it is not an issue though, it can always be reintroduced as and when myself or someone else gets a chance to do it, assuming this is worthwhile - others may have an opinion on that also :-) Regards, Phil