{"id":816153,"url":"http://patchwork.ozlabs.org/api/patches/816153/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/20170920111408.29711-5-jonas.gorski@gmail.com/","project":{"id":37,"url":"http://patchwork.ozlabs.org/api/projects/37/?format=json","name":"Devicetree Bindings","link_name":"devicetree-bindings","list_id":"devicetree.vger.kernel.org","list_email":"devicetree@vger.kernel.org","web_url":"","scm_url":"","webscm_url":"","list_archive_url":"","list_archive_url_format":"","commit_url_format":""},"msgid":"<20170920111408.29711-5-jonas.gorski@gmail.com>","list_archive_url":null,"date":"2017-09-20T11:14:04","name":"[V2,4/8] tty/bcm63xx_uart: allow naming clock in device tree","commit_ref":null,"pull_url":null,"state":"not-applicable","archived":true,"hash":"39bc0868426472fa77bbef3322be985504685c5d","submitter":{"id":7941,"url":"http://patchwork.ozlabs.org/api/people/7941/?format=json","name":"Jonas Gorski","email":"jonas.gorski@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/20170920111408.29711-5-jonas.gorski@gmail.com/mbox/","series":[{"id":4083,"url":"http://patchwork.ozlabs.org/api/series/4083/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=4083","date":"2017-09-20T11:14:00","name":"MIPS: BCM63XX: add and use clkdev lookup support","version":2,"mbox":"http://patchwork.ozlabs.org/series/4083/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/816153/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/816153/checks/","tags":{},"related":[],"headers":{"Return-Path":"","X-Original-To":"incoming-dt@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-dt@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=devicetree-owner@vger.kernel.org; receiver=)","ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=gmail.com header.i=@gmail.com\n\theader.b=\"s3+KtIu2\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xxxy770yhz9ryT\n\tfor ;\n\tWed, 20 Sep 2017 21:16:15 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751599AbdITLQO (ORCPT\n\t);\n\tWed, 20 Sep 2017 07:16:14 -0400","from mail-wm0-f67.google.com ([74.125.82.67]:36394 \"EHLO\n\tmail-wm0-f67.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751840AbdITLQN (ORCPT\n\t); Wed, 20 Sep 2017 07:16:13 -0400","by mail-wm0-f67.google.com with SMTP id r136so2119397wmf.3;\n\tWed, 20 Sep 2017 04:16:13 -0700 (PDT)","from localhost.localdomain ([2001:470:9e39::48e])\n\tby smtp.gmail.com with ESMTPSA id\n\ts12sm884513edd.25.2017.09.20.04.16.11\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tWed, 20 Sep 2017 04:16:12 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025;\n\th=from:to:cc:subject:date:message-id:in-reply-to:references;\n\tbh=aoZqJe/zJsKqD6CDebjOGOq+Cjc7ASV8t1qCFib6cac=;\n\tb=s3+KtIu2W/1tt8ZCY4noZpgLx7eLclxZrZwAJO9oVj/3qpxHnETqpbOqbGsYTrZiSk\n\tIu0fazp9VCX5H/XmXAu8Gr+P3tMgi4bs3O5JYv/9hxf7lN6M9ujSdSUSOnCjgqg2fina\n\tQBRtoU75TBmUJRwohj5z6YKi4vRuiwxyO1CyyIIcAyxljhHJF2L8Umny3zKPqdTqOVK7\n\tR8j9ByT1FRxcSi/gzvPIS7uSk2N4so1hobPu+gojDBoXf2Qrmwi85nwDZymrKU+y1NAX\n\tNmz7TRgDPtni0EK6uDl9u9614C5tjq/dmEbobveoe4E+KHr12a+4DViYwg5ydfsZsugw\n\tYOOg==","X-Google-DKIM-Signature":"v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to\n\t:references;\n\tbh=aoZqJe/zJsKqD6CDebjOGOq+Cjc7ASV8t1qCFib6cac=;\n\tb=MWHfA8CECYLqUqI/fr9FzMWjFOCJnS+CUvx43KB61aewAuOs62LHytRxt1VkdRAZyR\n\tUR+8PUw7J6gnMFpDpFxijqzjtkum0Vyasvb7tfbU23okmPksd0FMcnpZS1v1gZcrmDdQ\n\tJk9uPRJJyPCJRzvfYqRaYq6xMzQgWt3D2I6WcZQdaauD0r2hcdVayNMBgncOOi0zsvkz\n\t15BiQAgFI9DZw4D0FqlHZc5dafsi0ZaqxBMM8tRKHBnqW5QBqrJSdzfKopWfsd5ejkqP\n\t+f5MBAeBvKBzrlT0NuoOMonZBlr/TtLd8BnVUv5JOQFu/W3i5yzU1IUnAAhTE/YFs5Y7\n\tcquw==","X-Gm-Message-State":"AHPjjUhxbhVui8eRoTnz4pMC/rmrclMgUyt+yIODDvwOhJxUEzOWge1c\n\tZmigO9rcIA0ZVrZPQulD4eY=","X-Google-Smtp-Source":"AOwi7QAKV3xyf91HFB7r9ib7XtlLLbOzoLuVQYI0e3+jVyY44iSsWc4BZqjtfAzvrSZmzk8pMTMSnw==","X-Received":"by 10.80.206.11 with SMTP id y11mr4398662edi.94.1505906172634;\n\tWed, 20 Sep 2017 04:16:12 -0700 (PDT)","From":"Jonas Gorski ","To":"linux-mips@linux-mips.org, linux-arm-kernel@lists.infradead.org,\n\tlinux-serial@vger.kernel.org, devicetree@vger.kernel.org","Cc":"Greg Kroah-Hartman ,\n\tRob Herring , Mark Rutland ,\n\tRalf Baechle ,\n\tFlorian Fainelli ,\n\tbcm-kernel-feedback-list@broadcom.com,\n\tKevin Cernekee , \n\tRussell King ","Subject":"[PATCH V2 4/8] tty/bcm63xx_uart: allow naming clock in device tree","Date":"Wed, 20 Sep 2017 13:14:04 +0200","Message-Id":"<20170920111408.29711-5-jonas.gorski@gmail.com>","X-Mailer":"git-send-email 2.13.2","In-Reply-To":"<20170920111408.29711-1-jonas.gorski@gmail.com>","References":"<20170920111408.29711-1-jonas.gorski@gmail.com>","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"},"content":"Codify using a named clock for the refclk of the uart. This makes it\neasier if we might need to add a gating clock (like present on the\nBCM6345).\n\nAcked-by: Rob Herring \nAcked-by: Greg Kroah-Hartman \nReviewed-by: Florian Fainelli \nSigned-off-by: Jonas Gorski \n---\n Documentation/devicetree/bindings/serial/brcm,bcm6345-uart.txt | 6 ++++++\n drivers/tty/serial/bcm63xx_uart.c | 6 ++++--\n 2 files changed, 10 insertions(+), 2 deletions(-)","diff":"diff --git a/Documentation/devicetree/bindings/serial/brcm,bcm6345-uart.txt b/Documentation/devicetree/bindings/serial/brcm,bcm6345-uart.txt\nindex 5c52e5eef16d..8b2b0460259a 100644\n--- a/Documentation/devicetree/bindings/serial/brcm,bcm6345-uart.txt\n+++ b/Documentation/devicetree/bindings/serial/brcm,bcm6345-uart.txt\n@@ -11,6 +11,11 @@ Required properties:\n - clocks: Clock driving the hardware; used to figure out the baud rate\n divisor.\n \n+\n+Optional properties:\n+\n+- clock-names: Should be \"refclk\".\n+\n Example:\n \n \tuart0: serial@14e00520 {\n@@ -19,6 +24,7 @@ Example:\n \t\tinterrupt-parent = <&periph_intc>;\n \t\tinterrupts = <2>;\n \t\tclocks = <&periph_clk>;\n+\t\tclock-names = \"refclk\";\n \t};\n \n \tclocks {\ndiff --git a/drivers/tty/serial/bcm63xx_uart.c b/drivers/tty/serial/bcm63xx_uart.c\nindex a2b9376ec861..f227eff28d3a 100644\n--- a/drivers/tty/serial/bcm63xx_uart.c\n+++ b/drivers/tty/serial/bcm63xx_uart.c\n@@ -841,8 +841,10 @@ static int bcm_uart_probe(struct platform_device *pdev)\n \tif (!res_irq)\n \t\treturn -ENODEV;\n \n-\tclk = pdev->dev.of_node ? of_clk_get(pdev->dev.of_node, 0) :\n-\t\t\t\t clk_get(&pdev->dev, \"refclk\");\n+\tclk = clk_get(&pdev->dev, \"refclk\");\n+\tif (IS_ERR(clk) && pdev->dev.of_node)\n+\t\tclk = of_clk_get(pdev->dev.of_node, 0);\n+\n \tif (IS_ERR(clk))\n \t\treturn -ENODEV;\n \n","prefixes":["V2","4/8"]}