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=-4.0 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_PASS autolearn=unavailable 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 31F71C282C2 for ; Wed, 13 Feb 2019 15:24:21 +0000 (UTC) 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 mail.kernel.org (Postfix) with ESMTPS id 000BD20663 for ; Wed, 13 Feb 2019 15:24:20 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="SI8n5yBj"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=baylibre-com.20150623.gappssmtp.com header.i=@baylibre-com.20150623.gappssmtp.com header.b="kLei3CGV" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 000BD20663 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=baylibre.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From: In-Reply-To:References:MIME-Version:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=DVUGYBIm95hbZw8rKglc9Mdg0gid2nuSFSlGln3rguI=; b=SI8n5yBjfEJcej pF+tuctem3VoJ2LAWIAYIXGjGwxhIV2t/+y0wwvwfak5dkvJVguwEI0rOKuxuFuJwfC0TXj5zReN9 RSmarynh/1JHA825TxGDHeBEwBXk3dhGgClmr1WH/CdL86P5jH4jnM0VhU8wy2BNbVYNU+pvdgBlG FDDn4GMhQdwXpcA0C8i9bDVFgFldrBxkopkz9M/+fUwKSOjclFbOlHmGnbhKwBQMr5EqP+ZdrpT+w HZRS5u6DiAzpI47gI5mVsEViYYc3D8ax87lDosia73qPGLT0/3th+0BW0iaJB8TGOpW9jJoUI3rhi 7bhtOByTtFfZZSAUVd4Q==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.90_1 #2 (Red Hat Linux)) id 1gtwOW-0005rJ-1W; Wed, 13 Feb 2019 15:24:16 +0000 Received: from mail-ua1-x943.google.com ([2607:f8b0:4864:20::943]) by bombadil.infradead.org with esmtps (Exim 4.90_1 #2 (Red Hat Linux)) id 1gtwOT-0005qJ-1F for linux-amlogic@lists.infradead.org; Wed, 13 Feb 2019 15:24:14 +0000 Received: by mail-ua1-x943.google.com with SMTP id v26so901053uap.4 for ; Wed, 13 Feb 2019 07:24:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=iepFje/MEMhk/3E4cnIjUK1NAURKmM59l2Ai+HFgl3Q=; b=kLei3CGVOLhHwdFp1cFntXYHYuZ7JDpvOc97rUFvIW4okfi7bSlE+J1u5TNs0A99Vq fJApk6VcQdqiToVxuzeZEunO5jlSdcikTUEc/aixA4IUjtSB0tb7HQMT47wHtJ+u582g iFeJctncDS2ZpIdUgwxsJqZ61pye1jL6q3cnBTAqFs8p4zuaWj3GiVmW+PWz6oActyCQ S/2dOc7XrKh7cZ78Q3V/BEtLluvmJLJouta4AkRhEpH5R+dcJwBRSWUmCqzAMewVzCFC OIsgFXBtW+ANZVs0p4n7n9Mwi1FhUbbbD5ySEaUTMQ8mPu6z5sMFp2/bAPI+pdOfesgx 2I1A== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=iepFje/MEMhk/3E4cnIjUK1NAURKmM59l2Ai+HFgl3Q=; b=rwl8Evjjjq65F/TMBzGECkcNiZfG28TTB64O7gKDXU7ZXlsf6A3okmGtItwnxonpIg t2lFW7lws13qfZmyqTI1jPEwUxXe0jHPOtj6Cr0PJ8dd6sRrcRX6FkGj7ju6tBFyHsqm /pfWwu4aKz8iTZ7hQJcFWq3AGKDNT8HbZwJuzb5cb3zM9inO/LCoRApog3QIdGdkFI+P PvofjavK2d2N14J4s1GP9BBrnKOWRZf+46ogYl4WwriuFPioHjXSAUDXJ8AZmam8Ewau ac9plw5Ie5yt67zbUloDKtOD7tiIoIPmWt0Y/0NpIT4ycowx107d9mxA4blO8/p5qJ2I grrQ== X-Gm-Message-State: AHQUAuaouOGK+w1CftmK8Gu5OVbVzMX95kFVPffZDmvzn9gVbvDjHD3f DNu3lRQggT0p+3PeiWm05DvFqJqrlEhsibCuoKGCcQ== X-Google-Smtp-Source: AHgI3IaFe/NYA9tBN4Y21pnU9ZVr/OXvkJenpt1XaNeKjFdFt2/TxkkTnQrYZqdO2ACGL7o+i01j2wOp/oNYdeVv8g0= X-Received: by 2002:ab0:2b93:: with SMTP id q19mr388945uar.130.1550071451959; Wed, 13 Feb 2019 07:24:11 -0800 (PST) MIME-Version: 1.0 References: <1547744074-28308-1-git-send-email-lollivier@baylibre.com> <1547744074-28308-3-git-send-email-lollivier@baylibre.com> <20190125090755.GB3691@localhost> <867eeo7pjm.fsf@baylibre.com> <20190211083349.GU4686@localhost> <86mun2v27r.fsf@baylibre.com> In-Reply-To: <86mun2v27r.fsf@baylibre.com> From: Loys Ollivier Date: Wed, 13 Feb 2019 16:24:00 +0100 Message-ID: Subject: Re: [PATCH v2 2/4] dt-bindings: gnss: add mediatek binding To: Loys Ollivier X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20190213_072413_079611_DF711715 X-CRM114-Status: GOOD ( 15.83 ) X-BeenThere: linux-amlogic@lists.infradead.org X-Mailman-Version: 2.1.21 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , Martin Blumenstingl , Kevin Hilman , Johan Hovold , linux-kernel@vger.kernel.org, Rob Herring , "moderated list:ARM/Mediatek SoC support" , "open list:ARM/Amlogic Meson..." , Linux ARM Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-amlogic" Errors-To: linux-amlogic-bounces+linux-amlogic=archiver.kernel.org@lists.infradead.org On Mon 11 Feb 2019 at 14:17, Loys Ollivier wrote: > On Mon 11 Feb 2019 at 08:33, Johan Hovold wrote: > >> On Mon, Jan 28, 2019 at 04:46:53PM +0100, Loys Ollivier wrote: >>> >>> On Fri 25 Jan 2019 at 09:07, Johan Hovold wrote: >>> >>> > On Thu, Jan 17, 2019 at 05:54:32PM +0100, Loys Ollivier wrote: >>> >> Add binding for Mediatek-based GNSS receivers. >>> >> >>> >> Signed-off-by: Loys Ollivier >>> >> --- >>> >> >>> >> v2: >>> >> Renamed bindings from Globaltop/gtop to Mediatek/mtk. >>> >> Moved current-speed as an optional propertie. >>> >> Removed the status line in the example. >>> >> Added "mediatek,mt3339" compatible. >> >>> >> +++ b/Documentation/devicetree/bindings/gnss/mediatek.txt >>> >> @@ -0,0 +1,39 @@ >>> >> +Mediatek-based GNSS Receiver DT binding >>> >> + >>> >> +Mediatek chipsets are used in GNSS-receiver modules produced by several >>> >> +vendors and can use UART, SPI or I2C interfaces. >>> >> + >>> >> +Please see Documentation/devicetree/bindings/gnss/gnss.txt for generic >>> >> +properties. >>> >> + >>> >> +Required properties: >>> >> + >>> >> +- compatible : Must be one of >>> >> + >>> >> + "globaltop,pa6h" >>> >> + "mediatek,mt3339" >>> > >>> > I'm not sure it makes sense to add the GPS chip itself as it cannot be >>> > used stand-alone AFAIU, and either not way would require a lot more >>> > resources to be defined than what you add to this binding. >>> >>> OK, I find it a bit confusing to have Mediatek bindings but no >>> compatible Mediatek SoCs. >>> But it is true that it can't be used stand-alone. I will remove this >>> reference and also remove it from the mtk.c driver code from this patch >>> serie. >> >>> >> +Required properties (I2C): >>> >> +- reg : I2C slave address >>> >> + >>> >> +Required properties (SPI): >>> >> +- reg : SPI chip select address >>> > >>> > While the mt3339 appears to support these interfaces, the globaltop one >>> > does not. >>> > >>> > Have you looked around for a small sample of mt3339-based receivers, to >>> > see whether anyone actually provides these? >>> >>> Again I'm confused by the fact that it is the Mediatek bindings. >>> Hence why I have added it and why it was not present in the v1 that was >>> only for Globaltop. >> >> This is no different from the sirfstar driver for receivers based on >> those chipsets but manufactured by various other vendors. >> > Indeed. > >>> I find it odd mentionning the chipset but not all its interfaces. >>> I will remove these sections and only list supported interfaces (i.e. >>> UART). Other interfaces can be added afterward if one finds a board that >>> provides them. >> >> Right, but please do look around for other receivers using this chipset >> so that we can make the binding generic enough to cover those as well >> (e.g. by choosing representative resource names). >> > OK will have a look. The only mt3339-based receivers I have found are: - [0] Globaltop - [1] Locosys / Pololu They both only provide the UART interface and do not list the I2C/SPI interfaces. [0] https://cdn-shop.adafruit.com/datasheets/GlobalTop-FGPMMOPA6H-Datasheet-V0A.pdf [1] https://www.pololu.com/file/0J641/LS20030~3_datasheet_v1.3.pdf > >> Johan -- -L _______________________________________________ linux-amlogic mailing list linux-amlogic@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-amlogic