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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9CC59C4332F for ; Thu, 29 Dec 2022 10:24:15 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233296AbiL2KYO (ORCPT ); Thu, 29 Dec 2022 05:24:14 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52980 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233260AbiL2KXl (ORCPT ); Thu, 29 Dec 2022 05:23:41 -0500 Received: from mail-lf1-x132.google.com (mail-lf1-x132.google.com [IPv6:2a00:1450:4864:20::132]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id CFC9F6315 for ; Thu, 29 Dec 2022 02:19:12 -0800 (PST) Received: by mail-lf1-x132.google.com with SMTP id z26so26933725lfu.8 for ; Thu, 29 Dec 2022 02:19:12 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=vMKLMjJpe5LSVosq/kgKiCUoGwlkMRk7EHg56MOXRG0=; b=W0/mzJ+BVYSeYNkqjUy3oTgPwLK1Bx4J2+308QC0ISlojj5vA7qUfYge3guPCW0vHR b4uo//YVXlquaTDtBEvxbnRLbSNe2ihsWAdvxUgYoZl+UxYKxRxRXRGdEQEh7KaHZttC xpqJPTr4MDV22D3FtYZIe7qJK+0g+cgNpJuRko0WgzLNja/E9UpMy3KX18bIVuCgwKiL V8rVp2b+G80ZWFuy43MTMYfn63OXdXuXo5NEd6S8hbKUe/B6ad/PPDyK0m7G68Rp9Ud+ WY5lvy6RWe3ZNXefsOkQFswi8xsP9eDLrPYQdvUe7Z+JZCEz5+l1pwcPfVV/fNbCcnr1 3AAg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=vMKLMjJpe5LSVosq/kgKiCUoGwlkMRk7EHg56MOXRG0=; b=uykNCqT81OmggUCycm717UTIP3rvi3XK7HO8eKxz4k334eC8QKUOl62R9XITHx0KTg Tw/JpHcWZxvpdIRD7424p1kodmHn5Tz6hNsTb/pWJ+Ywc4RlpqL4NsCsoPtYn45kBIhD u+w2z52FlpVRoXgxo8lcigoZOr3+51XoRIHNUNFX1jIWIVf1uli8PYCpDbBNUe6cP1k1 5x6LIe7r1AOu9raV7uTDJcbgmg+RNrZa+If4m6wtnGEY0KZ8XsUgXksf9I2IsS1RUGHx lW1D8TrIFoFUd+oUgnDvF2VxCAUrKfTrq4WMBmM4BbYPfwzzWj/a24M6qLBIGuGZEC/D JMQQ== X-Gm-Message-State: AFqh2kpt+nlRo7ZhzXyJ4BqAXGSceOOyHzHh8URnuIgIxJs1vPiyKiiB pfnk3OuQ1DQTcSLIzQmeR3+ahA== X-Google-Smtp-Source: AMrXdXvLe1i7tZ7mGvfACMbmYCAO3F40RJhpPtndbkRc12jHOJYE7rKTNP/9+510BhvQHTQwfbG2xg== X-Received: by 2002:a05:6512:3b2c:b0:4a4:68b7:deab with SMTP id f44-20020a0565123b2c00b004a468b7deabmr11677612lfv.7.1672309151197; Thu, 29 Dec 2022 02:19:11 -0800 (PST) Received: from [192.168.0.20] (088156142067.dynamic-2-waw-k-3-2-0.vectranet.pl. [88.156.142.67]) by smtp.gmail.com with ESMTPSA id d2-20020ac241c2000000b004a05402c5c3sm3024922lfi.93.2022.12.29.02.19.09 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 29 Dec 2022 02:19:10 -0800 (PST) Message-ID: Date: Thu, 29 Dec 2022 11:19:09 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.6.1 Subject: Re: [RFC PATCH v2 2/3] dt-bindings: usb: generic-xhci: add Samsung Exynos compatible To: Daehwan Jung , Greg Kroah-Hartman , Rob Herring , Krzysztof Kozlowski , Thinh Nguyen , Mathias Nyman , Felipe Balbi Cc: "open list:USB SUBSYSTEM" , "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" , open list , sc.suh@samsung.com, taehyun.cho@samsung.com, jh0801.jung@samsung.com, eomji.oh@samsung.com References: <1672307866-25839-1-git-send-email-dh10.jung@samsung.com> <1672307866-25839-3-git-send-email-dh10.jung@samsung.com> Content-Language: en-US From: Krzysztof Kozlowski In-Reply-To: <1672307866-25839-3-git-send-email-dh10.jung@samsung.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 29/12/2022 10:57, Daehwan Jung wrote: > Add compatible for Samsung Exynos SOCs Missing full stop. Please explain here in details the hardware. Otherwise it looks it is not for any hardware and patch should be dropped. Also, missing DTS. I am going to keep NAK-ing this till you provide the user. NAK. > > Signed-off-by: Daehwan Jung > --- > Documentation/devicetree/bindings/usb/generic-xhci.yaml | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/generic-xhci.yaml b/Documentation/devicetree/bindings/usb/generic-xhci.yaml > index db841589fc33..f54aff477637 100644 > --- a/Documentation/devicetree/bindings/usb/generic-xhci.yaml > +++ b/Documentation/devicetree/bindings/usb/generic-xhci.yaml > @@ -29,6 +29,8 @@ properties: > enum: > - brcm,xhci-brcm-v2 > - brcm,bcm7445-xhci > + - description: Samsung Exynos SoCs with xHCI > + const: samsung,exynos-xhci Missing fallback. > - description: Generic xHCI device> const: xhci-platform > deprecated: true Best regards, Krzysztof