From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.8 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8FD73C43603 for ; Thu, 12 Dec 2019 07:33:11 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 6227924658 for ; Thu, 12 Dec 2019 07:33:11 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1576135991; bh=p12HIynD9FdwHF+0wBqafDSonAXyb1SgrBSwbjgba2k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=Le+73ozfQsozhYUmEZJ9AG05ZOoV4e90yllsFYtWvfQ8BzjnPUr4pXiIKaFxOTa8R nmfmBLTb52MCHUSuYmb+AVovEyeuVE6WqHG2hwEOz5EbfAXjE3PY8aubgDWF/n3V9V NRoXnjbi6ctS4MfUodZPDMD/sH6C8OR95nefQYYs= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728118AbfLLHdK (ORCPT ); Thu, 12 Dec 2019 02:33:10 -0500 Received: from mail.kernel.org ([198.145.29.99]:57830 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728072AbfLLHdK (ORCPT ); Thu, 12 Dec 2019 02:33:10 -0500 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 5FBB524656; Thu, 12 Dec 2019 07:33:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1576135989; bh=p12HIynD9FdwHF+0wBqafDSonAXyb1SgrBSwbjgba2k=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WeeJAVaKEcQ79OCKF1uE2gMatc9ftJQESVzr7N51pSimXsoOUrl+5gATZmvX59JKi e14oruK9gl4gVqy/j2zvltEVVeU/XBJbTzg5wOGRtmQMAUbSymS0KQNYmcFDydfEAm TbL+eYw0/M27oW6kYE5GQ0TenwSb9MxC6751v7UE= Date: Thu, 12 Dec 2019 08:33:06 +0100 From: Greg Kroah-Hartman To: Kieran Bingham Cc: Laurent Pinchart , Sasha Levin , Kuninori Morimoto , Mauro Carvalho Chehab , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org, koji.matsuoka.xm@renesas.com, takeshi.kihara.df@renesas.com, harunobu.kurokawa.dn@renesas.com, khiem.nguyen.xt@renesas.com, hien.dang.eb@renesas.com Subject: Re: [PATCH] media: vsp1: tidyup VI6_HGT_LBn_H() macro Message-ID: <20191212073306.GB1364286@kroah.com> References: <87k173bp76.wl-kuninori.morimoto.gx@renesas.com> <20191211175811.GC4863@pendragon.ideasonboard.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-renesas-soc@vger.kernel.org On Wed, Dec 11, 2019 at 09:58:11PM +0000, Kieran Bingham wrote: > Hi Laurent, > > +Greg, +Sasha to opine on the merit of whether this should go to stable > trees (for my future learning and understanding more so than this > specific case) > > On 11/12/2019 17:58, Laurent Pinchart wrote: > > Hello, > > > > On Wed, Dec 11, 2019 at 12:59:57PM +0000, Kieran Bingham wrote: > >> Hi Morimoto-san, > >> > >> Thank you for the patch, > > > > Likewise :-) > > > >> On 11/12/2019 01:55, Kuninori Morimoto wrote: > >>> > >>> From: Kuninori Morimoto > >>> > >>> The address of VSP2_VI6_HGT_LBx_H are > >>> VSP2_VI6_HGT_LB0_H : 0x3428 > >>> VSP2_VI6_HGT_LB1_H : 0x3430 > >>> VSP2_VI6_HGT_LB2_H : 0x3438 > >>> VSP2_VI6_HGT_LB3_H : 0x3440 > >>> > >>> Thus, VI6_HGT_LBn_H() macro should start from 0x3420 instead of 0x3430. > >>> This patch fixup it. > > s/fixup/fixes/ > > > >> I think this deserves a fixes tag: > >> > >> Fixes: 26e0ca22c3b8 ("[media] v4l: Renesas R-Car VSP1 driver") > > > > Given that this macro is not used, we could argue that it doesn't fix > > anything yet :-) I'd rather avoid having this backported to stable > > kernels as it's not useful to have it there, and thus not add a Fixes > > I'm sorry - I'm not sure I can agree here, Do you know that no one will > use this macro when they back port the HGT functionality to an LTSI kernel? > > We know the Renesas BSP uses LTSI kernels, and the very nature of the > fact that this typo has been spotted by the Renesas BSP team suggests > that they are indeed looking at/using this functionality ... > > (Ok, so maybe they will thus apply the fix themselves, but that's not my > point, and if they 'have' to apply the fix - it should be in stable?) > > It feels a bit presumptuous to state that we shouldn't fix this because > /we/ don't utilise it yet, when this issue is in mainline regardless ... Nothing should be in the kernel tree that is not already used by something in that specific kernel tree. We don't care about out-of-tree code, and especially for stable kernel patches, it does not matter in the least. If you have out-of-tree code, you are on your own here, sorry. So no, no backporting of stuff that no one actually uses in the codebase itself. thanks, greg k-h