From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Osipenko Subject: Re: [PATCH v1] usb: chipidea: tegra: Fix missed ci_hdrc_remove_device() Date: Tue, 26 Feb 2019 18:08:17 +0300 Message-ID: References: <20190224153622.8877-1-digetx@gmail.com> <20190225080715.4965810a@dimatab> <20190226105615.GA12397@kroah.com> <20190226145857.GB26080@kroah.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: In-Reply-To: <20190226145857.GB26080@kroah.com> Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org To: Greg Kroah-Hartman Cc: Joe Perches , Andy Whitcroft , Peter Chen , Thierry Reding , Jonathan Hunter , "linux-usb@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-Id: linux-tegra@vger.kernel.org 26.02.2019 17:58, Greg Kroah-Hartman пишет: > On Tue, Feb 26, 2019 at 05:33:05PM +0300, Dmitry Osipenko wrote: >> 26.02.2019 13:56, Greg Kroah-Hartman пишет: >>> On Mon, Feb 25, 2019 at 08:07:15AM +0300, Dmitry Osipenko wrote: >>>> В Mon, 25 Feb 2019 02:27:19 +0000 >>>> Peter Chen пишет: >>>> >>>>> >>>>>> >>>>>> Fixes: dfebb5f43a78827a ("usb: chipidea: Add support for >>>>>> Tegra20/30/114/124") >>>>> >>>>> I suppose you need to apply at stable tree too, right? >>>>> >>>> >>>> It is enough to have the "Fixes" tag to get patch backported into all >>>> relevant kernel versions. >>> >>> No it is not. My scripts do NOT trigger off of the fixes: tag, please >>> read: >>> https://www.kernel.org/doc/html/latest/process/stable-kernel-rules.html >>> for how to do this properly. >> >> Okay, my bad then. Maybe this is something that could warned by checkpatch.. adding Joe and Andy to the thread. > > Why? It's allowed to put fixes: tags for a patch that does not belong > in a stable tree. That happens all the time, and is encouraged. Look > at some of the stuff in linux-next now, we have Fixes: for commits that > are still in linux-next as well, because we do not rebase our trees. > When they all merge into Linus's tree, all is good. > > So this is not something that checkpatch needs to do anything about. At least that might help in cases like this if maintainer is also oblivious. I guess it wouldn't hurt at all to add a "green" warning if there is a "Fixes" tag and no "stable", something like this: WARNING: There is a "Fixes" tag and no "Cc: stable ", please add the stable tag if patch is intended to be backported to stable kernels. From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: [v1] usb: chipidea: tegra: Fix missed ci_hdrc_remove_device() From: Dmitry Osipenko Message-Id: Date: Tue, 26 Feb 2019 18:08:17 +0300 To: Greg Kroah-Hartman Cc: Joe Perches , Andy Whitcroft , Peter Chen , Thierry Reding , Jonathan Hunter , "linux-usb@vger.kernel.org" , "linux-tegra@vger.kernel.org" , "linux-kernel@vger.kernel.org" List-ID: MjYuMDIuMjAxOSAxNzo1OCwgR3JlZyBLcm9haC1IYXJ0bWFuINC/0LjRiNC10YI6Cj4gT24gVHVl LCBGZWIgMjYsIDIwMTkgYXQgMDU6MzM6MDVQTSArMDMwMCwgRG1pdHJ5IE9zaXBlbmtvIHdyb3Rl Ogo+PiAyNi4wMi4yMDE5IDEzOjU2LCBHcmVnIEtyb2FoLUhhcnRtYW4g0L/QuNGI0LXRgjoKPj4+ IE9uIE1vbiwgRmViIDI1LCAyMDE5IGF0IDA4OjA3OjE1QU0gKzAzMDAsIERtaXRyeSBPc2lwZW5r byB3cm90ZToKPj4+PiDQkiBNb24sIDI1IEZlYiAyMDE5IDAyOjI3OjE5ICswMDAwCj4+Pj4gUGV0 ZXIgQ2hlbiA8cGV0ZXIuY2hlbkBueHAuY29tPiDQv9C40YjQtdGCOgo+Pj4+Cj4+Pj4+ICAKPj4+ Pj4+Cj4+Pj4+PiBGaXhlczogZGZlYmI1ZjQzYTc4ODI3YSAoInVzYjogY2hpcGlkZWE6IEFkZCBz dXBwb3J0IGZvcgo+Pj4+Pj4gVGVncmEyMC8zMC8xMTQvMTI0IikgIAo+Pj4+Pgo+Pj4+PiBJIHN1 cHBvc2UgeW91IG5lZWQgdG8gYXBwbHkgYXQgc3RhYmxlIHRyZWUgdG9vLCByaWdodD8KPj4+Pj4K Pj4+Pgo+Pj4+IEl0IGlzIGVub3VnaCB0byBoYXZlIHRoZSAiRml4ZXMiIHRhZyB0byBnZXQgcGF0 Y2ggYmFja3BvcnRlZCBpbnRvIGFsbAo+Pj4+IHJlbGV2YW50IGtlcm5lbCB2ZXJzaW9ucy4KPj4+ Cj4+PiBObyBpdCBpcyBub3QuICBNeSBzY3JpcHRzIGRvIE5PVCB0cmlnZ2VyIG9mZiBvZiB0aGUg Zml4ZXM6IHRhZywgcGxlYXNlCj4+PiByZWFkOgo+Pj4gICAgIGh0dHBzOi8vd3d3Lmtlcm5lbC5v cmcvZG9jL2h0bWwvbGF0ZXN0L3Byb2Nlc3Mvc3RhYmxlLWtlcm5lbC1ydWxlcy5odG1sCj4+PiBm b3IgaG93IHRvIGRvIHRoaXMgcHJvcGVybHkuCj4+Cj4+IE9rYXksIG15IGJhZCB0aGVuLiBNYXli ZSB0aGlzIGlzIHNvbWV0aGluZyB0aGF0IGNvdWxkIHdhcm5lZCBieSBjaGVja3BhdGNoLi4gYWRk aW5nIEpvZSBhbmQgQW5keSB0byB0aGUgdGhyZWFkLgo+IAo+IFdoeT8gIEl0J3MgYWxsb3dlZCB0 byBwdXQgZml4ZXM6IHRhZ3MgZm9yIGEgcGF0Y2ggdGhhdCBkb2VzIG5vdCBiZWxvbmcKPiBpbiBh IHN0YWJsZSB0cmVlLiAgVGhhdCBoYXBwZW5zIGFsbCB0aGUgdGltZSwgYW5kIGlzIGVuY291cmFn ZWQuICBMb29rCj4gYXQgc29tZSBvZiB0aGUgc3R1ZmYgaW4gbGludXgtbmV4dCBub3csIHdlIGhh dmUgRml4ZXM6IGZvciBjb21taXRzIHRoYXQKPiBhcmUgc3RpbGwgaW4gbGludXgtbmV4dCBhcyB3 ZWxsLCBiZWNhdXNlIHdlIGRvIG5vdCByZWJhc2Ugb3VyIHRyZWVzLgo+IFdoZW4gdGhleSBhbGwg bWVyZ2UgaW50byBMaW51cydzIHRyZWUsIGFsbCBpcyBnb29kLgo+IAo+IFNvIHRoaXMgaXMgbm90 IHNvbWV0aGluZyB0aGF0IGNoZWNrcGF0Y2ggbmVlZHMgdG8gZG8gYW55dGhpbmcgYWJvdXQuCgpB dCBsZWFzdCB0aGF0IG1pZ2h0IGhlbHAgaW4gY2FzZXMgbGlrZSB0aGlzIGlmIG1haW50YWluZXIg aXMgYWxzbyBvYmxpdmlvdXMuCgpJIGd1ZXNzIGl0IHdvdWxkbid0IGh1cnQgYXQgYWxsIHRvIGFk ZCBhICJncmVlbiIgd2FybmluZyBpZiB0aGVyZSBpcyBhICJGaXhlcyIgdGFnIGFuZCBubyAic3Rh YmxlIiwgc29tZXRoaW5nIGxpa2UgdGhpczoKCldBUk5JTkc6IFRoZXJlIGlzIGEgIkZpeGVzIiB0 YWcgYW5kIG5vICJDYzogc3RhYmxlIDxzdGFibGVAdmdlci5rZXJuZWwub3JnPiIsIHBsZWFzZSBh ZGQgdGhlIHN0YWJsZSB0YWcgaWYgcGF0Y2ggaXMgaW50ZW5kZWQgdG8gYmUgYmFja3BvcnRlZCB0 byBzdGFibGUga2VybmVscy4K