lkml.org 
[lkml]   [2022]   [Nov]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v6 2/7] platform/chrome: cros_ec_typec: Purge blocking switch devlinks
    Hi Andy,

    Thanks for taking a look at this patch.

    Pin-Yen beat me to the punch with comment responses, but I'll add mine anyway.

    On Thu, Nov 24, 2022 at 4:25 AM Andy Shevchenko
    <andriy.shevchenko@linux.intel.com> wrote:
    > ...
    >
    > > + /*
    > > + * OF graph may have set up some device links with switches, since connectors have their
    > > + * own compatible. Purge these to avoid a deadlock in switch probe (the switch mistakenly
    > > + * assumes the connector is a supplier).
    > > + */
    >
    > A bit too long lines...

    They are within the 100 character limit [1] which is followed
    elsewhere in the driver; has something
    changed recently to make that invalid?

    >
    > > + if (dev->of_node)
    >
    > Why do you need this check?

    This issue only arises when using DT for this device. So the rationale
    is we don't need to
    perform this step on systems that don't use DT.

    Best regards,

    -Prashant

    [1] https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=bdc48fa11e46f867ea4d75fa59ee87a7f48be144

    \
     
     \ /
      Last update: 2022-11-25 06:54    [W:2.662 / U:0.012 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site