From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f195.google.com (mail-pl1-f195.google.com [209.85.214.195]) (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 C8E6818BBA6 for ; Thu, 12 Sep 2024 07:16:30 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.195 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726125392; cv=none; b=kHrCleCCTJHMhpt96wflfbSyBzxE+Lsm3R6UdyCxwfBNiKfzEmuH6+Vsn1iA6wbrP2E+RF+mQbGBjmzffVxEA45zUGcPH6lxQ6Dl4Knx/yUO/B09h/l+iqdGqjaItzYizBWOQ+B/Ds1DTuQZOtjh36chvUK5tq4LMgTgRMaiSv4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1726125392; c=relaxed/simple; bh=AHrcQx8LR1mijO3wHALGSJs+5RqDe43pynZ1DgmTUCg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=CS2BejQdeniLV9iChwmJkN2H6qL4hQHhdox3SBgBlhPByJC+lOjU7hImkT0il7/QmZpPA9sh4MJJ1/djOEm0273WCf2IqG8+ZL4LNj50jxEL4A902xm29PMvMPbNKQ19Oql13f02j430msJX+9acsurJxjHzSPcgSDJrGI0Z6vU= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=beagleboard.org; spf=fail smtp.mailfrom=beagleboard.org; dkim=pass (2048-bit key) header.d=beagleboard-org.20230601.gappssmtp.com header.i=@beagleboard-org.20230601.gappssmtp.com header.b=TCXeVHHh; arc=none smtp.client-ip=209.85.214.195 Authentication-Results: smtp.subspace.kernel.org; dmarc=fail (p=none dis=none) header.from=beagleboard.org Authentication-Results: smtp.subspace.kernel.org; spf=fail smtp.mailfrom=beagleboard.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=beagleboard-org.20230601.gappssmtp.com header.i=@beagleboard-org.20230601.gappssmtp.com header.b="TCXeVHHh" Received: by mail-pl1-f195.google.com with SMTP id d9443c01a7336-20543fdb7acso5340325ad.1 for ; Thu, 12 Sep 2024 00:16:30 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=beagleboard-org.20230601.gappssmtp.com; s=20230601; t=1726125390; x=1726730190; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=lh7l9YoHy4qkgvE49qc+r/MAsmVlGzV7/aids6D1De4=; b=TCXeVHHhXqMrqL4/1GuwNnEgxG30DjlYBJgBWnSuM8lE92x8U0Fw6U0i4M/MwluSwM RuoYsCOaVrBhXzUAjJKwEcFtvfQpq5HC/n1pASaqwl/c+s/BUQE7zXJiPAhvBSPyQ+Rg NQbeMBx217h4kuvrEJfA28ekfHRtFi6Z+XIjkLaG9IbYkobryXyq5SCj3wr81xDMoya/ 9M9XDMjSNH87OZJd5KE6uEJ5vbLJRlJpNWiQXrsOyYsDZtTZzTMld0tNfFg7L/tNit4P mCkJ7TFP7nGOA4Q0HrUITGJlsvgIPwM2XAp4SiTeyzuC5Sg1LBBLzI5Z/cN0cQ+kMYSq Nycg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1726125390; x=1726730190; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=lh7l9YoHy4qkgvE49qc+r/MAsmVlGzV7/aids6D1De4=; b=rhgQ4qMp9qnBJJJjbjFyLDFSUddHTa9NL7hIWtOwfarZVOwENXW/vsNHVA49gD0wZU YJ101CFXtsPr1/hVJlMW1gviQHlcVMFTuBnJX0nLBY3Rsn9ShhjLpy/2ocd6qU9e+/1K 1i3o3ujBpxNoNUN0TTX/dwEk5J2bQhnOkpgvb/O4MInlgbOl2SrUmj+1JWWjTKfwjLN4 Skpg61Bi0KJOKbLaL27nnMSlY9Pt8Fh9Q6LAZ1nsIHZOEAr3+vkrJ1FUt3SM8JaFho7x RdvMuk7lsRwZHcoGg39rBggjxDFi5Vin9XHU89WVFYvH4Y8RjkNp4unfRKLzM9JPB5Tm RbGg== X-Forwarded-Encrypted: i=1; AJvYcCU/nZ6I0coF9k+Cj239etzGJzLA4kI+04QkWQkV4Gd7r0ifUE69PyOys0+K0ZnKWbOt0mCryBX7vVTVIinFtA==@vger.kernel.org X-Gm-Message-State: AOJu0Yws/oscIvmBx+PZyj0VtW/qHb4V+mQZB9jC+sOxLCt2UV7rfo4t 3ceujHUISsa8PQRpJVzgL9LNb5s8V2oO6L9WrSywcUa//jjQ2CP4rYvCZVUo9w== X-Google-Smtp-Source: AGHT+IEgQUUi+YQfcBvAAIBX3uJLnFXqg+luxeETvddbtjlQ/UH/tax/cuWaglJLbLCWUuuqRX2lvQ== X-Received: by 2002:a17:902:da88:b0:206:b250:1e1 with SMTP id d9443c01a7336-2076e4239fdmr32667795ad.45.1726125389908; Thu, 12 Sep 2024 00:16:29 -0700 (PDT) Received: from [172.16.118.100] ([103.15.228.94]) by smtp.gmail.com with ESMTPSA id 98e67ed59e1d1-2dadb42e6fasm11951226a91.0.2024.09.12.00.16.20 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 12 Sep 2024 00:16:29 -0700 (PDT) Message-ID: <097159c7-1602-4e32-8e6f-9cd023d62238@beagleboard.org> Date: Thu, 12 Sep 2024 12:46:17 +0530 Precedence: bulk X-Mailing-List: rust-for-linux@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 8/8] addon_boards: mikrobus: Add GPS3 Click Content-Language: en-US To: Greg Kroah-Hartman , Ayush Singh Cc: fabien.parent@linaro.org, d-gole@ti.com, lorforlinux@beagleboard.org, jkridner@beagleboard.org, robertcnelson@beagleboard.org, Andrew Davis , Miguel Ojeda , Alex Gaynor , Boqun Feng , Gary Guo , =?UTF-8?Q?Bj=C3=B6rn_Roy_Baron?= , Benno Lossin , Andreas Hindborg , Alice Ryhl , Trevor Gross , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , linux-kernel@vger.kernel.org, rust-for-linux@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org References: <20240911-mikrobus-dt-v1-0-3ded4dc879e7@beagleboard.org> <20240911-mikrobus-dt-v1-8-3ded4dc879e7@beagleboard.org> <2024091149-vocalize-composite-6e48@gregkh> <44039255-159a-4284-abd8-a0f558ad006d@gmail.com> <2024091151-unworldly-dance-9a80@gregkh> From: Ayush Singh In-Reply-To: <2024091151-unworldly-dance-9a80@gregkh> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 9/12/24 01:34, Greg Kroah-Hartman wrote: > On Wed, Sep 11, 2024 at 09:26:06PM +0530, Ayush Singh wrote: >> On 9/11/24 20:28, Greg Kroah-Hartman wrote: >> >>> On Wed, Sep 11, 2024 at 07:57:25PM +0530, Ayush Singh wrote: >>>> - GPS3 Click is a UART MikroBUS addon Board >>>> >>>> Link: https://www.mikroe.com/gps-3-click >>>> >>>> Signed-off-by: Ayush Singh >>>> --- >>>> addon_boards/mikrobus/Makefile | 1 + >>>> addon_boards/mikrobus/mikroe-1714.dtso | 28 ++++++++++++++++++++++++++++ >>> Odd top-level directory for the kernel, are you sure this is correct? >>> >>> thanks, >>> >>> greg k-h >>> >> Well, it is kinda a temporary location, since well, I could not find a good >> place for board overlays but a top-level location seems better than putting >> them in any arch specific location. I am open to moving them to a more >> suitable location if we have one. > top-level location is not ok for something so tiny and "special". Just > put it where all other dtso files go. > > thanks, > > greg k-h So here are the directories where dtso files currently go: ❯ find . -type f -name "*.dtso" -printf "%h\n" | sort -u ./arch/arm64/boot/dts/amlogic ./arch/arm64/boot/dts/freescale ./arch/arm64/boot/dts/mediatek ./arch/arm64/boot/dts/qcom ./arch/arm64/boot/dts/renesas ./arch/arm64/boot/dts/rockchip ./arch/arm64/boot/dts/ti ./arch/arm64/boot/dts/xilinx ./arch/arm/boot/dts/nxp/imx ./arch/arm/boot/dts/ti/omap ./drivers/clk ./drivers/of ./drivers/of/unittest-data Out of these, `drivers/of` and `drivers/of/unittest-data` contain unittest dtso, so probably not the place. And the `arch/arm` and `arch/arm64` are for arch specific stuff. MikroBUS is supported in RISC-V boards as well (BeagleV-Ahead). So probably not the correct location either. Maybe something like `arch/common/addon_boards` would be better? Ayush Singh