{"id":814493,"url":"http://patchwork.ozlabs.org/api/patches/814493/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/20170916104220.3742-2-fancer.lancer@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":"<20170916104220.3742-2-fancer.lancer@gmail.com>","list_archive_url":null,"date":"2017-09-16T10:42:12","name":"[1/9,v2] usb: usb251xb: Add USB2517i specific struct and IDs","commit_ref":null,"pull_url":null,"state":"changes-requested","archived":true,"hash":"aaa7c7512765e395139244de8c650edf60ace4f8","submitter":{"id":70038,"url":"http://patchwork.ozlabs.org/api/people/70038/?format=json","name":"Serge Semin","email":"fancer.lancer@gmail.com"},"delegate":null,"mbox":"http://patchwork.ozlabs.org/project/devicetree-bindings/patch/20170916104220.3742-2-fancer.lancer@gmail.com/mbox/","series":[{"id":3430,"url":"http://patchwork.ozlabs.org/api/series/3430/?format=json","web_url":"http://patchwork.ozlabs.org/project/devicetree-bindings/list/?series=3430","date":"2017-09-16T10:42:11","name":"usb: usb251xb: Add USB2517i hub support and fix some bugs","version":2,"mbox":"http://patchwork.ozlabs.org/series/3430/mbox/"}],"comments":"http://patchwork.ozlabs.org/api/patches/814493/comments/","check":"pending","checks":"http://patchwork.ozlabs.org/api/patches/814493/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=\"tO7s+Y7i\"; dkim-atps=neutral"],"Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xvTS31x2Zz9t32\n\tfor ;\n\tSat, 16 Sep 2017 20:45:07 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751211AbdIPKou (ORCPT\n\t);\n\tSat, 16 Sep 2017 06:44:50 -0400","from mail-lf0-f68.google.com ([209.85.215.68]:37493 \"EHLO\n\tmail-lf0-f68.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751209AbdIPKmY (ORCPT\n\t); Sat, 16 Sep 2017 06:42:24 -0400","by mail-lf0-f68.google.com with SMTP id q132so2324315lfe.4;\n\tSat, 16 Sep 2017 03:42:23 -0700 (PDT)","from linux.local ([95.79.181.62]) by smtp.gmail.com with ESMTPSA id\n\tf199sm553379lfg.85.2017.09.16.03.42.21\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128);\n\tSat, 16 Sep 2017 03:42:21 -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=juus1pduD7eDM6PT0NxoGDcrQwEwfsUhrO/8m1IUl4I=;\n\tb=tO7s+Y7ioU9+XUYxb/l85aIMqVME3R23/z3ps55WRHJmh2OSsuMp5/GLTAyj96n0Ra\n\tzrcCwKna9MHT+FPoNuGMpxXC7GbAogs+knaekuBCMAgqF/rpOdEmlt9VxWIjABvjnioN\n\tXl5hEZ9nCCpnxMVlsEA6AVZvOsMZ3mc9ndIDKzC3Jg5etegQanPN04x5X+vHCEleNNgg\n\tZ42dIBTFTC0o8oT4Dax7DcF1jCjkWM1xqHYoubFcnjR/7D2nkZnqKZDJ+iToOuxDe2A2\n\tS9aT1+DaTi5FWRDzMjh2Me1kLQkLlPiNQmHU63lhb69+y8yGc9TNiPpyDtErLSybqb0E\n\t+iag==","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=juus1pduD7eDM6PT0NxoGDcrQwEwfsUhrO/8m1IUl4I=;\n\tb=ss+tTN+j/HMx1EuAUFCWihheKQcaCMLtGBzGiO8TNW+CKme26TC9fNgm7nZI1X0jre\n\tRm7/8FJcuQmffC7NP1APknlIAuna96k1N5glX28hfe6TFx9nVY18Ik11KhHzIp8fkbSa\n\talyGJK+OcIz/8GPJOa0lXUnFaIAYLXeK+4GHUOowTYBj50Y/1+cRJLUCfy1Y9iElmSot\n\tQbiLCR/OZAxoTlPw/7mhNoEUymjx876RF43NQrXBV5lC9BlkyZFrzd3uuxZTGMjLK3pv\n\tT/ERtHyDK5iqRMKcntfczOV4nt7uOCqnpcPUYt9y2z7vVSR9Rnhnwhdq/QEZTSThBkcE\n\tbaCg==","X-Gm-Message-State":"AHPjjUhCyHpWE6zu60LI/JaM/ifkuWsLyVatvUngtCkxaIqvv91KfLQ2\n\tzQWXu7WlnFmxEKpuct8=","X-Google-Smtp-Source":"AOwi7QDIHNYfp//qByHr4wCHOpZG2+JB8acZtNwVmsMXp0DMpnbrPlGRrLsqJvM5X1E8hzHkUmUI5g==","X-Received":"by 10.25.18.71 with SMTP id h68mr828685lfi.217.1505558542554;\n\tSat, 16 Sep 2017 03:42:22 -0700 (PDT)","From":"Serge Semin ","To":"richard.leitner@skidata.com, gregkh@linuxfoundation.org,\n\trobh+dt@kernel.org, mark.rutland@arm.com","Cc":"Sergey.Semin@t-platforms.ru, linux-usb@vger.kernel.org,\n\tdevicetree@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tSerge Semin ","Subject":"[PATCH 1/9 v2] usb: usb251xb: Add USB2517i specific struct and IDs","Date":"Sat, 16 Sep 2017 13:42:12 +0300","Message-Id":"<20170916104220.3742-2-fancer.lancer@gmail.com>","X-Mailer":"git-send-email 2.12.0","In-Reply-To":"<20170916104220.3742-1-fancer.lancer@gmail.com>","References":"<20170915233113.17855-1-fancer.lancer@gmail.com>\n\t<20170916104220.3742-1-fancer.lancer@gmail.com>","Sender":"devicetree-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"devicetree@vger.kernel.org"},"content":"There are USB2517 and USB2517i hubs, which have almost the same\nregisters space as already supported USB251xbi series. The difference\nit in DIDs and in few functions. This patch adds the USB2517/i data\nstructures to the driver, so it would have different setting depending\non the device discovered on i2c-bus.\n\nSigned-off-by: Serge Semin \n---\n Documentation/devicetree/bindings/usb/usb251xb.txt | 3 ++-\n drivers/usb/misc/usb251xb.c | 21 ++++++++++++++++++++-\n 2 files changed, 22 insertions(+), 2 deletions(-)","diff":"diff --git a/Documentation/devicetree/bindings/usb/usb251xb.txt b/Documentation/devicetree/bindings/usb/usb251xb.txt\nindex 3957d4eda..1682d4087 100644\n--- a/Documentation/devicetree/bindings/usb/usb251xb.txt\n+++ b/Documentation/devicetree/bindings/usb/usb251xb.txt\n@@ -6,7 +6,8 @@ Hi-Speed Controller.\n Required properties :\n - compatible : Should be \"microchip,usb251xb\" or one of the specific types:\n \t\"microchip,usb2512b\", \"microchip,usb2512bi\", \"microchip,usb2513b\",\n-\t\"microchip,usb2513bi\", \"microchip,usb2514b\", \"microchip,usb2514bi\"\n+\t\"microchip,usb2513bi\", \"microchip,usb2514b\", \"microchip,usb2514bi\",\n+\t\"microchip,usb2517\", \"microchip,usb2517i\"\n - reset-gpios : Should specify the gpio for hub reset\n - reg : I2C address on the selected bus (default is <0x2C>)\n \ndiff --git a/drivers/usb/misc/usb251xb.c b/drivers/usb/misc/usb251xb.c\nindex 91f66d68b..96a8c20ac 100644\n--- a/drivers/usb/misc/usb251xb.c\n+++ b/drivers/usb/misc/usb251xb.c\n@@ -38,6 +38,7 @@\n #define USB251XB_DEF_PRODUCT_ID_12\t0x2512 /* USB2512B/12Bi */\n #define USB251XB_DEF_PRODUCT_ID_13\t0x2513 /* USB2513B/13Bi */\n #define USB251XB_DEF_PRODUCT_ID_14\t0x2514 /* USB2514B/14Bi */\n+#define USB251XB_DEF_PRODUCT_ID_17\t0x2517 /* USB2517i */\n \n #define USB251XB_ADDR_DEVICE_ID_LSB\t0x04\n #define USB251XB_ADDR_DEVICE_ID_MSB\t0x05\n@@ -82,7 +83,7 @@\n \n #define USB251XB_ADDR_PRODUCT_STRING_LEN\t0x14\n #define USB251XB_ADDR_PRODUCT_STRING\t\t0x54\n-#define USB251XB_DEF_PRODUCT_STRING\t\t\"USB251xB/xBi\"\n+#define USB251XB_DEF_PRODUCT_STRING\t\t\"USB251xB/xBi/7i\"\n \n #define USB251XB_ADDR_SERIAL_STRING_LEN\t\t0x15\n #define USB251XB_ADDR_SERIAL_STRING\t\t0x92\n@@ -186,6 +187,16 @@ static const struct usb251xb_data usb2514bi_data = {\n \t.product_str = \"USB2514Bi\",\n };\n \n+static const struct usb251xb_data usb2517_data = {\n+\t.product_id = 0x2517,\n+\t.product_str = \"USB2517\",\n+};\n+\n+static const struct usb251xb_data usb2517i_data = {\n+\t.product_id = 0x2517,\n+\t.product_str = \"USB2517i\",\n+};\n+\n static void usb251xb_reset(struct usb251xb *hub, int state)\n {\n \tif (!gpio_is_valid(hub->gpio_reset))\n@@ -511,6 +522,12 @@ static const struct of_device_id usb251xb_of_match[] = {\n \t\t.compatible = \"microchip,usb2514bi\",\n \t\t.data = &usb2514bi_data,\n \t}, {\n+\t\t.compatible = \"microchip,usb2517\",\n+\t\t.data = &usb2517_data,\n+\t}, {\n+\t\t.compatible = \"microchip,usb2517i\",\n+\t\t.data = &usb2517i_data,\n+\t}, {\n \t\t/* sentinel */\n \t}\n };\n@@ -574,6 +591,8 @@ static const struct i2c_device_id usb251xb_id[] = {\n \t{ \"usb2513bi\", 0 },\n \t{ \"usb2514b\", 0 },\n \t{ \"usb2514bi\", 0 },\n+\t{ \"usb2517\", 0 },\n+\t{ \"usb2517i\", 0 },\n \t{ /* sentinel */ }\n };\n MODULE_DEVICE_TABLE(i2c, usb251xb_id);\n","prefixes":["1/9","v2"]}