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 Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 494A3E743C4 for ; Thu, 28 Sep 2023 21:17:35 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232166AbjI1VRe (ORCPT ); Thu, 28 Sep 2023 17:17:34 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34804 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230325AbjI1VRb (ORCPT ); Thu, 28 Sep 2023 17:17:31 -0400 Received: from mail-ej1-x62f.google.com (mail-ej1-x62f.google.com [IPv6:2a00:1450:4864:20::62f]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id B067AF3 for ; Thu, 28 Sep 2023 14:17:27 -0700 (PDT) Received: by mail-ej1-x62f.google.com with SMTP id a640c23a62f3a-9ad8a822508so1779523666b.0 for ; Thu, 28 Sep 2023 14:17:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1695935846; x=1696540646; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:from:to:cc:subject:date:message-id :reply-to; bh=YOLKD+dZ4eKUfIGgNxL5xyo6MXfu46Z0PMVOhkalApk=; b=HWMcDsBhoC8OcpJzthYDv+JDLRXkUlAGuXWR3YIYy1c5pMm/pbmoeOMCQA8KnOv/rZ pRDbrPFwUj8vQx22cPN5nbfSBcaYdt1Yg9IFm4YL26gqVHLezZ5grCyqLI8AZMnGwG2d c58hSl4HiNCxF+JnaWcNY++cVmEXjQI/xbBPYN6GZPbzFXqKybOjv46ko/DTF5rFDGP5 3A38ETRrZAmYqq1JVoBomIuVRjtpaI+kV6DcGR6SsPFGwmdAyNv/o73CIiqvjTmhbwqg Nc/3Ufjte3nh1i9YprM4ZkHW3iz9X0rC6LLnIa7OE8YiKxcj2sqkiBysxD41AMsHSXpN phXw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1695935846; x=1696540646; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:sender:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=YOLKD+dZ4eKUfIGgNxL5xyo6MXfu46Z0PMVOhkalApk=; b=FcsAEZZkB1tecAorzh+DiLZDYi5L409qYzbtaJgt8m9jgVqW6Y04P4HylVT2JsADwf 1jSAuXdyQvlRoTqpf7koeHKpTABk+2+UR11ZuCpJsxYUeM+U1HquFkNQH3ZjY+oVam9n p8v0OcDqs2kU3r4Kvu/e3SdLLrquocvkR75yyuVGdJK7SCtZYpFzlrxK85mtnJs60/9k pYpgdCEMznbInPXxMbwBGAkiyqrpPYIsvYst3RSzPfJOP9mkh7FRjNvsZVMxjCCnH/Lp V09Q8Gob/Z4Zws8QFE6kW7G9jpsOineHno3PZ71YRLQW9Wbn2Uuk59Klfz2ZVsOwFLCL tlzQ== X-Gm-Message-State: AOJu0Yx0IU1nm2i7+MYhdWHlFUuqi2Qi1ELjzFLDDg8QgMWGPi13nd1u 6cFt70KH3EAmH+g5FWFOUZI= X-Google-Smtp-Source: AGHT+IFiqFTKmdhbxuHnd0UkrdV7E62jBybvnImlXxobRDF3xRWcomLQKKackBEZSRbT9T+ehKcVlQ== X-Received: by 2002:a17:906:530c:b0:9a1:e941:6f49 with SMTP id h12-20020a170906530c00b009a1e9416f49mr2551299ejo.44.1695935845835; Thu, 28 Sep 2023 14:17:25 -0700 (PDT) Received: from gmail.com (1F2EF49C.nat.pool.telekom.hu. [31.46.244.156]) by smtp.gmail.com with ESMTPSA id bu10-20020a170906a14a00b0099bd7b26639sm11455976ejb.6.2023.09.28.14.17.24 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 28 Sep 2023 14:17:25 -0700 (PDT) Sender: Ingo Molnar Date: Thu, 28 Sep 2023 23:17:23 +0200 From: Ingo Molnar To: Marc Zyngier Cc: Thomas Gleixner , Biju Das , Claudiu Beznea , Dmitry Baryshkov , Geert Uytterhoeven , Konrad Dybcio , Lad Prabhakar , Maulik Shah , Neil Armstrong , linux-kernel@vger.kernel.org Subject: Re: [GIT PULL] irqchip fixes for 6.6, take #1 Message-ID: References: <20230924094105.2361754-1-maz@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20230924094105.2361754-1-maz@kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org * Marc Zyngier wrote: > Dmitry Baryshkov (1): > arm64: dts: qcom: sm8150: extend the size of the PDC resource This SOB chain, while valid, is a hodgepodge with no rhyme and reason: Fixes: 397ad94668c1 ("arm64: dts: qcom: sm8150: Add pdc interrupt controller node") Reviewed-by: Konrad Dybcio Signed-off-by: Dmitry Baryshkov Reviewed-by: Neil Armstrong Signed-off-by: Neil Armstrong Signed-off-by: Marc Zyngier Link: https://lore.kernel.org/r/20230905-topic-sm8x50-upstream-pdc-ver-v4-2-fc633c7df84b@linaro.org Besides the ordering, people in the SOB chain should not have a separate Reviewed-by tag: *of course* anyone who forwards a patch with a SOB has reviewed it, it's basically implicit. So the tags should be something like: Fixes: 397ad94668c1 ("arm64: dts: qcom: sm8150: Add pdc interrupt controller node") Signed-off-by: Dmitry Baryshkov Signed-off-by: Neil Armstrong Signed-off-by: Marc Zyngier Reviewed-by: Konrad Dybcio Link: https://lore.kernel.org/r/20230905-topic-sm8x50-upstream-pdc-ver-v4-2-fc633c7df84b@linaro.org Ie. we generally sort the tags not chronologically (or whatever the original idea was), but group them topically, sort the groups by importance, and only within the group is there chronological order. Thanks, Ingo