lkml.org 
[lkml]   [2016]   [Mar]   [16]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [PATCH v10 3/9] dt-bindings: phy: tegra-xusb-padctl: Add Tegra210 support
    From
    Date
    On 03/04/2016 09:19 AM, Thierry Reding wrote:
    > From: Thierry Reding <treding@nvidia.com>
    >
    > Extend the binding to cover the set of feature found in Tegra210.

    Acked-by: Stephen Warren <swarren@nvidia.com>

    > diff --git a/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt b/Documentation/devicetree/bindings/phy/nvidia,tegra124-xusb-padctl.txt

    > + padctl@0,7009f000 {
    ...
    > + pads {
    ...
    > + };
    > +
    > + ports {
    ...
    > + };

    As a comment not affecting my ack in any way: At the top-level, we place
    all the child nodes into "array container" nodes named "pads" and
    "ports". This is nice since it separates different types of child nodes
    and allows easily adding more types of child nodes in the future without
    interference, and in a way that allows us to explicitly know what each
    node is without having to interpret its name or compatible value to do
    so. However, we haven't done this with the per-lane child nodes inside
    each pad. If we were to rev the design, I'd be tempted to suggest:

    padctl@0,7009f000 {
    pads {
    usb2 {
    lanes { // This level is new
    usb2-0 {

    \
     
     \ /
      Last update: 2016-03-16 19:41    [W:4.193 / U:1.080 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site