From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f173.google.com (mail-pl1-f173.google.com [209.85.214.173]) (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 83E161F8912 for ; Tue, 21 Jan 2025 22:50:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.173 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737499845; cv=none; b=BctUpriWR2LZifhNP9H9vARIFDJZUJf0kVUp2S7x/jkSXy3dk7+zZmIXjimZIS12cOPgHDkutA2h8k2x+EwL5j3Z4PKDZKb4OWW1L8XTaDBdVWwmviMtbTQPi5+UYlKOnLOklKec3K4jwgNqeZLIVu4U8iE91ujmzNMY6GsRnDw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1737499845; c=relaxed/simple; bh=J/09iJn/f8OIFoYmfEGJR1plKN5XM4G4egV7j6tGXdU=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=BcaBVExKB9bMBBkUnOfPb5ZAgCNlOkE5t+oxxvA2b8a+TMPdVRZCyJoXfczWLvrHvjp0sJ2UQ6/qSEERd1pBfLm5O1isgFY8HRlPdiApwJXwlVJZBj3Y3xvwOGo2QPdx27JeN6Ro3Pm63ngvhHTc5LP8hvh+/a7DSdG7p6Uh36w= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=kernel.org; spf=pass smtp.mailfrom=baylibre.com; arc=none smtp.client-ip=209.85.214.173 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=quarantine dis=none) header.from=kernel.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=baylibre.com Received: by mail-pl1-f173.google.com with SMTP id d9443c01a7336-2164b662090so114449585ad.1 for ; Tue, 21 Jan 2025 14:50:43 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1737499843; x=1738104643; h=mime-version:message-id:date:references:in-reply-to:subject:cc:to :from:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=eY/bocEY4/ixO9G6Mvr/Z5L8XQkKrCMbNaB8Z7NbGGs=; b=Yl85h+toEy31ojCNz+Tw2pnW715hXdcWDrx8qRz5bD1F6mLREWj281/APuOqmIX3V7 1mADs60/6SqWBq/cW46cJY8UklAEJcBMzdosh319y5JOCWYdyzbx5xklwlPxlGF4Xphe JsEGFrped3U/Ijq8v/JgLE9kPWNQgEBZIfwsGwGYfje6EnXfKnkEwkMoIj8lvLMcrHAa fWOoPq2dL3eZ6t8lUVlRLDK2TAZnCQHwvOHT8hiSoY4TSFJoKS83a6vSkPLfEHKtBcfj AIvK5nNimMuCi4jxOEbElKEdqdMnkL6h5aZwpCr/sBoFZ/r0DWKe40z1L3eEyLfYdkcZ 9zgA== X-Forwarded-Encrypted: i=1; AJvYcCXjn1aLG9HGWnHgufZydO38z/GeSyQgbk6ZDluX8qi6bfdXWbExFaa7OhYnVqa7VXoNHDo60fWcVmShBMc=@vger.kernel.org X-Gm-Message-State: AOJu0Yw/EOtPOUs8M+zmmL9VTGA63peK0AwjmjrKbTv8s5W5sFkfDocW 25me/DlzzipeyP046zk5EsNxRROhQUw+2ZktOCvDtJLFnonngSxi8gdJQnNsVI0= X-Gm-Gg: ASbGncsd1wIjvEP2Jml2RSAFjecfyZeJPgruXfqFR7GC8o4IX2kdSi/TxP3+QBTUmzz +0vIi+9+DbCkRJcPrfBNzZkyyjNQ7L2puq2DVk8kip45mRtg/xcDRGQjS1x2fgEYZnzZ+g+Q3Pu GD6sklEiWCZMTwo1GXpxKZWKYYPORHYEj8sBQOvNbexsR6Ifidg2gkG+0MtePCb5o35bxybmf7E f5G4G+LKaGNJqdihzzsb93ayP2ik7fnu9k74nwQk9KHwnmNN7tRt/7N1arVq+hpHKQ3 X-Google-Smtp-Source: AGHT+IHaaAXCvryBAXeNGBTJxWdGI8mRRoTbDGvpoGAQHpPdsSxC9BsUktepgvmvmzRhBvsfjx8DQQ== X-Received: by 2002:a05:6a21:998d:b0:1e1:b28e:a148 with SMTP id adf61e73a8af0-1eb2144d513mr25264291637.5.1737499842764; Tue, 21 Jan 2025 14:50:42 -0800 (PST) Received: from localhost ([97.126.182.119]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-72dabaa6407sm9649528b3a.163.2025.01.21.14.50.42 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 21 Jan 2025 14:50:42 -0800 (PST) From: Kevin Hilman To: Tom Rini , linux-kernel@vger.kernel.org Cc: Nishanth Menon , Tony Lindgren , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-omap@vger.kernel.org, devicetree@vger.kernel.org Subject: Re: [PATCH] ARM: dts: omap4-panda-a4: Add missing model and compatible properties In-Reply-To: <20250121200749.4131923-1-trini@konsulko.com> References: <20250121200749.4131923-1-trini@konsulko.com> Date: Tue, 21 Jan 2025 14:50:41 -0800 Message-ID: <7hmsfjn5mm.fsf@baylibre.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain Hi Tom, Tom Rini writes: > When moving the model and compatible properties out of the common > Pandaboard files and in to the specific boards, the omap4-panda-a4 > file wasn't updated as well and so has lacked a model and compatible > entry ever since. > > Fixes: a1a57abaaf82 ("ARM: dts: omap4-panda: Fix model and SoC family details") > Signed-off-by: Tom Rini > --- Checkpatch tells me: WARNING: DT compatible string "ti,omap4-panda-a4" appears un-documented So I think Documentation/devicetree/bindings/arm/ti/omap.yaml needs an update too. (and note the binding update should be a separate patch[1]) > Given how long this has been broken it's entirely plausible no a4 > hardware even exists anymore and so dropping this file instead makes > sense. I only found this because scripts/make_fit.py crashed on these > properties being missing. If keeping it is just this binding update, then I'd say we keep it, but if it gets any more paninful to maintain, I'm also not going to argue very hard to keep it. Kevin [1] From: Documentation/devicetree/bindings/submitting-patches.rst 1) The Documentation/ and include/dt-bindings/ portion of the patch should be a separate patch. The preferred subject prefix for binding patches is:: "dt-bindings: : ..." > Cc: Nishanth Menon > Cc: Tony Lindgren > Cc: Rob Herring > Cc: Krzysztof Kozlowski > Cc: Conor Dooley > Cc: linux-omap@vger.kernel.org > Cc: devicetree@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > arch/arm/boot/dts/ti/omap/omap4-panda-a4.dts | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/arch/arm/boot/dts/ti/omap/omap4-panda-a4.dts b/arch/arm/boot/dts/ti/omap/omap4-panda-a4.dts > index 8fd076e5d1b0..4b8bfd0188ad 100644 > --- a/arch/arm/boot/dts/ti/omap/omap4-panda-a4.dts > +++ b/arch/arm/boot/dts/ti/omap/omap4-panda-a4.dts > @@ -7,6 +7,11 @@ > #include "omap443x.dtsi" > #include "omap4-panda-common.dtsi" > > +/ { > + model = "TI OMAP4 PandaBoard (A4)"; > + compatible = "ti,omap4-panda-a4", "ti,omap4-panda", "ti,omap4430", "ti,omap4"; > +}; > + > /* Pandaboard Rev A4+ have external pullups on SCL & SDA */ > &dss_hdmi_pins { > pinctrl-single,pins = < > -- > 2.43.0