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 bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 26B24C54E5D for ; Tue, 19 Mar 2024 11:37:26 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Content-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=IdmabwfuoLaWR3j8fWgiZWX74OwF4M2uVLrbOhS0LOw=; b=FUmKEuFWI8z/3D N530NeGUgwCW+sL5s93AedPKJ7YCRzR/+yxx8rDd3HcY9OI3kvDxI54R/pH4/B8dshIV5q6LpKHE9 lbVXBtZcxgiqPoEtj0/tag90KoHPqTKjA0hr/k/p31enB5RpG7NUoiuPXI0FSPY1QMZiIhLkigeRs NOxdM0QSqSXBDNDdRiJ+wSHc1J9UqwhAhQdn1G1Tr8rgKBYFjeNgBn9Ue3Q6I+aySTjoJGFqlu3hD +AaY9ETA7D5FFh03loLUR4JQbeaD1/boAKNlb2+DyLduZeOvoTCP7dH5JMRjcQB8DcZOuTs2XMWKR 5AcMe/AHkYQlgZoqFqbg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmXmE-0000000CUvs-2aMz; Tue, 19 Mar 2024 11:37:06 +0000 Received: from mail-pl1-x62c.google.com ([2607:f8b0:4864:20::62c]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmXmB-0000000CUuc-3Hly for linux-arm-kernel@lists.infradead.org; Tue, 19 Mar 2024 11:37:05 +0000 Received: by mail-pl1-x62c.google.com with SMTP id d9443c01a7336-1deefb08b9eso23487005ad.3 for ; Tue, 19 Mar 2024 04:37:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1710848222; x=1711453022; darn=lists.infradead.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=umAIavC67cazmJ9lW2ejp2jSJGxVPgTOeby3+DV2+G4=; b=jQp/x9be49C1gulztV1dKgfAtMWzvoZtL2vfG6kpc1C+LFmakUkR5MUkHdMDzaFQs/ RZvGhvPj9CZKCYX6UMK+Hn+QWpBys0MVEgDlrCN6PAIp5iD8s6DUfIY2mQadh9nEx3jV xnnITmunXSexLo3MpP+r5EaWBqU+gl+1nCx6rqhyRxHO3FuS5SkqmoRTMqSNP+3XW6pD vYfQ1LUBO4r2YOfPpXN3j9VMO5JZG8D1Dvbsc+3uooGf2ID+Rzi1xwfklGNNxbMokQ2v um8/LaoQB60tPwg96YuJLq1kjEkYFh2Hu20CpZiDtYUUI1CO5IXrbcbbmh8pSX7RndVI VF0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1710848222; x=1711453022; 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=umAIavC67cazmJ9lW2ejp2jSJGxVPgTOeby3+DV2+G4=; b=gh9/+UBj2QcKIuD/cuAFVcSjOAuR4Td8Jd5s5pG0wNrYEHBxKKTb+u/FDI9zMkjLcW bu6Hjrtw79adiye3ocwEMfGXL4ofvErgNK9XIGmuhIkg/BI4av/ip90WMRWw5dGqX7Ts uVL+iAeC0CmGCgpSu5+aejmkVDOUAmtG9L6UmgNrLksaSMnFZyt5vFr+R2YxFsos53HO d+HAfUjCrsnpsPabApuEs64dtc90l49Jrznmu0VzumgbpxzUTiTaIwlmNslShdvHY17E IKJEyeZR7XjqZMtXUN0z7cPlMUY1HJ2jIIB2IB+FIoV8G7gM6OCLueV5ABgOOXgzJ1+m /fIw== X-Forwarded-Encrypted: i=1; AJvYcCXJdLtPfGe9s+EiFEzXl+EzGSb0hENPH9mFCz9XoIPsXLBob+Tt1/X3NueExoTrDBPGfazoGvZSEX3AM7aXuIof2WlaJe4i07IoHi69mRKuxAPQ6t0= X-Gm-Message-State: AOJu0Yxs27fY8z2yojQ2Bve576Jbp4lgtAcrDbc3i7Ic/zFs8JkbIEZ6 Pc0EjQkbjFImxdnAfuv0JaBnBAK+mPsiPTPyjVCf7E2Qzi54h4/t X-Google-Smtp-Source: AGHT+IF+DsFLA4Qvhq8yiEnjGjx2GBguwAi8iI5wKdvQAedsOquaxYYW+xLfytLU6wfSwtrLpVXPfw== X-Received: by 2002:a17:902:6502:b0:1e0:3861:9e46 with SMTP id b2-20020a170902650200b001e038619e46mr2136786plk.49.1710848222071; Tue, 19 Mar 2024 04:37:02 -0700 (PDT) Received: from [172.16.116.58] ([103.15.228.94]) by smtp.gmail.com with ESMTPSA id lg7-20020a170902fb8700b001dd1d7bc0f7sm11218702plb.154.2024.03.19.04.36.47 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 19 Mar 2024 04:37:01 -0700 (PDT) Message-ID: <5a9b1cd9-05ec-4606-92b6-eadbc7af6202@gmail.com> Date: Tue, 19 Mar 2024 17:06:36 +0530 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 1/5] dt-bindings: misc: Add mikrobus-connector Content-Language: en-US To: Michael Walle , Krzysztof Kozlowski , open list Cc: jkridner@beagleboard.org, robertcnelson@beagleboard.org, lorforlinux@beagleboard.org, Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Derek Kiernan , Dragan Cvetic , Arnd Bergmann , Greg Kroah-Hartman , Vaishnav M A , Mark Brown , Johan Hovold , Alex Elder , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , "moderated list:ARM/TEXAS INSTRUMENTS K3 ARCHITECTURE" , "open list:SPI SUBSYSTEM" , "moderated list:GREYBUS SUBSYSTEM" , Vaishnav M A References: <20240317193714.403132-1-ayushdevel1325@gmail.com> <20240317193714.403132-2-ayushdevel1325@gmail.com> <0f3f56d4-3381-44f1-91bc-c126f3ced085@linaro.org> From: Ayush Singh In-Reply-To: X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240319_043703_861389_B18607A5 X-CRM114-Status: GOOD ( 20.26 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 3/19/24 15:08, Michael Walle wrote: > Hi, > >> Regardless, this patch actually does not contain any code for EEPROM >> support I have just mentioned it to give more context on why mikroBUS >> manifest is the focus of this patch instead of DT overlay or something >> else. > Right, and I think this is the crux here. Why can't you use DT > overlays? The manifest files, seem to be yet another hardware > description (method) and we already have DT. Can't we have some kind > of userspace helper that could translate them to DT overlays? That > way, you could also handle the EEPROM vs non-EEPROM case, or have > some other kind of method to load a DT overlay. > > Admittedly, I've never worked with in-kernel overlays, but AFAIK > they work with some subsystems. > > -michael So let me 1st go over 3 cases that the driver needs to support: 1. Non EEPROM boards: Using overlays should be pretty similar to current solution. If the manifest is converted to overlay in userspace, then we do not even need to do manifest parsing, setting up spi, i2c etc in the kernel driver. 2. EEPROM boards How do you propose handling these. If you are proposing storing dt overlay in EEPROM, then this raises some questions regarding support outside of Linux. The other option would be generating overlay from manifest in the kernel driver, which I'm not sure is significantly better than registering the i2c, spi, etc. interfaces separately using standard kernel APIs. 3. Over Greybus It is quite important to have mikroBUS over greybus for BeagleConnect. This is one of the major reasons why greybus manifest was chosen for the manifest format. Also, it is important to note that mikroBUS manifest is being used since 2020 now and thus manifests for a lot of boards (both supporting clickID and not supporting it exist). So I would prefer using it, unless of course there are strong reasons not to. Ayush Singh CorrectBasicCloseSpellingPossible spelling mistake found.GrabsGrey busIgnore _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel