From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Pin-yen Lin <treapking@chromium.org>,
Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <robert.foss@linaro.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Daniel Scally <djrscally@gmail.com>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Prashant Malani <pmalani@chromium.org>,
Benson Leung <bleung@chromium.org>,
Guenter Roeck <groeck@chromium.org>
Cc: "Stephen Boyd" <swboyd@chromium.org>,
"Nícolas F . R . A . Prado" <nfraprado@collabora.com>,
"Marek Vasut" <marex@denx.de>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
devicetree@vger.kernel.org, dri-devel@lists.freedesktop.org,
"Javier Martinez Canillas" <javierm@redhat.com>,
"Lyude Paul" <lyude@redhat.com>,
chrome-platform@lists.linux.dev, "Xin Ji" <xji@analogixsemi.com>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
linux-kernel@vger.kernel.org,
"Allen Chen" <allen.chen@ite.com.tw>,
linux-acpi@vger.kernel.org, "Hsin-Yi Wang" <hsinyi@chromium.org>,
"Douglas Anderson" <dianders@chromium.org>,
"Imre Deak" <imre.deak@intel.com>,
"Jani Nikula" <jani.nikula@intel.com>,
"José Expósito" <jose.exposito89@gmail.com>,
"Kees Cook" <keescook@chromium.org>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Ville Syrjälä" <ville.syrjala@linux.intel.com>,
"shaomin Deng" <dengshaomin@cdjrlc.com>
Subject: Re: [PATCH v10 0/9] Register Type-C mode-switch in DP bridge endpoints
Date: Thu, 12 Jan 2023 06:34:52 +0200 [thread overview]
Message-ID: <61ba2880-6784-1014-a7a2-a1eecc4e810e@linaro.org> (raw)
In-Reply-To: <20230112042104.4107253-1-treapking@chromium.org>
On 12/01/2023 06:20, Pin-yen Lin wrote:
>
> This series introduces bindings for anx7625/it6505 to register Type-C
> mode-switch in their output endpoints, and use data-lanes property to
> describe the pin connections.
Please cc everybody on all patches. Having received just a single patch
made me spend time on having to look them up on lore.
>
> The first two patch modifies fwnode_graph_devcon_matches and
> cros_typec_init_ports to enable the registration of the switches.
>
> Patch 4~6 introduce the bindings for anx7625 and the corresponding driver
> modifications.
>
> Patch 7~9 add similar bindings and driver changes for it6505.
>
> v9: https://lore.kernel.org/all/20230109084101.265664-1-treapking@chromium.org/
> v8: https://lore.kernel.org/all/20230107102231.23682-1-treapking@chromium.org/
> v7: https://lore.kernel.org/all/20230105132457.4125372-1-treapking@chromium.org/
> v6: https://lore.kernel.org/all/20221124102056.393220-1-treapking@chromium.org/
> v5: https://lore.kernel.org/linux-usb/20220622173605.1168416-1-pmalani@chromium.org/
>
> Changes in v10:
> - Collected Reviewed-by and Tested-by tags
> - Replaced "void *" with "typec_mux_set_fn_t" for mux_set callbacks
> - Print out the node name when errors on parsing DT
> - Use dev_dbg instead of dev_warn when no Type-C switch nodes available
> - Made the return path of drm_dp_register_mode_switch clearer
> - Added a TODO for implementing orientation switch for anx7625
> - Updated the commit message for the absence of orientation switch
> - Fixed typo in the commit message
>
> Changes in v9:
> - Collected Reviewed-by tag
> - Fixed subject prefix again
> - Changed the naming of the example node for it6505
>
> Changes in v8:
> - Fixed the build issue when CONFIG_TYPEC=m
> - Fixed some style issues
> - Fixed the subject prefixes for the bindings patch
> - Fixed the bindings for data-lanes properties
>
> Changes in v7:
> - Fix the long comment lines
> - Extracted the common codes to a helper function
> - Fixed style issues in anx7625 driver
> - Removed DT property validation in anx7625 driver.
> - Fixed style issues in it6505 driver
> - Removed the redundant sleep in it6505 driver
> - Removed DT property validation in it6505 driver
> - Rebased to drm-misc-next
> - Fixed indentations in bindings patches
> - Added a new patch to fix indentations in Kconfig
>
> Changes in v6:
> - Changed it6505_typec_mux_set callback function to accommodate with
> the latest drm-misc patches
> - Changed the driver implementation to accommodate with the new binding
> - Dropped typec-switch binding and use endpoints and data-lanes properties
> to describe the pin connections
> - Added new patches (patch 1,2,4) to fix probing issues
> - Changed the bindings of it6505/anx7625 and modified the drivers
> accordingly
> - Merged it6505/anx7625 driver changes into a single patch
>
> Pin-yen Lin (7):
> drm/display: Add Type-C switch helpers
> dt-bindings: display: bridge: anx7625: Add mode-switch support
> drm/bridge: anx7625: Check for Type-C during panel registration
> drm/bridge: anx7625: Register Type C mode switches
> dt-bindings: display: bridge: it6505: Add mode-switch support
> drm/bridge: it6505: Fix Kconfig indentation
> drm/bridge: it6505: Register Type C mode switches
>
> Prashant Malani (2):
> device property: Add remote endpoint to devcon matcher
> platform/chrome: cros_ec_typec: Purge blocking switch devlinks
>
> .../display/bridge/analogix,anx7625.yaml | 99 ++++++++++++-
> .../bindings/display/bridge/ite,it6505.yaml | 93 ++++++++++--
> drivers/base/property.c | 15 ++
> drivers/gpu/drm/bridge/Kconfig | 21 +--
> drivers/gpu/drm/bridge/analogix/Kconfig | 1 +
> drivers/gpu/drm/bridge/analogix/anx7625.c | 105 +++++++++++++-
> drivers/gpu/drm/bridge/analogix/anx7625.h | 13 ++
> drivers/gpu/drm/bridge/ite-it6505.c | 119 +++++++++++++++-
> drivers/gpu/drm/display/drm_dp_helper.c | 134 ++++++++++++++++++
> drivers/platform/chrome/cros_ec_typec.c | 10 ++
> include/drm/display/drm_dp_helper.h | 17 +++
> 11 files changed, 598 insertions(+), 29 deletions(-)
>
--
With best wishes
Dmitry
WARNING: multiple messages have this Message-ID (diff)
From: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
To: Pin-yen Lin <treapking@chromium.org>,
Andrzej Hajda <andrzej.hajda@intel.com>,
Neil Armstrong <neil.armstrong@linaro.org>,
Robert Foss <robert.foss@linaro.org>,
Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
Jonas Karlman <jonas@kwiboo.se>,
Jernej Skrabec <jernej.skrabec@gmail.com>,
David Airlie <airlied@gmail.com>, Daniel Vetter <daniel@ffwll.ch>,
Rob Herring <robh+dt@kernel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Daniel Scally <djrscally@gmail.com>,
Heikki Krogerus <heikki.krogerus@linux.intel.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Prashant Malani <pmalani@chromium.org>,
Benson Leung <bleung@chromium.org>,
Guenter Roeck <groeck@chromium.org>
Cc: dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org,
"Marek Vasut" <marex@denx.de>,
chrome-platform@lists.linux.dev,
"Javier Martinez Canillas" <javierm@redhat.com>,
linux-acpi@vger.kernel.org,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
devicetree@vger.kernel.org, "Kees Cook" <keescook@chromium.org>,
"Nícolas F . R . A . Prado" <nfraprado@collabora.com>,
"Jani Nikula" <jani.nikula@intel.com>,
"Allen Chen" <allen.chen@ite.com.tw>,
"Stephen Boyd" <swboyd@chromium.org>,
"Hsin-Yi Wang" <hsinyi@chromium.org>,
"Xin Ji" <xji@analogixsemi.com>,
"AngeloGioacchino Del Regno"
<angelogioacchino.delregno@collabora.com>,
"shaomin Deng" <dengshaomin@cdjrlc.com>,
"Douglas Anderson" <dianders@chromium.org>,
"Thomas Zimmermann" <tzimmermann@suse.de>,
"José Expósito" <jose.exposito89@gmail.com>
Subject: Re: [PATCH v10 0/9] Register Type-C mode-switch in DP bridge endpoints
Date: Thu, 12 Jan 2023 06:34:52 +0200 [thread overview]
Message-ID: <61ba2880-6784-1014-a7a2-a1eecc4e810e@linaro.org> (raw)
In-Reply-To: <20230112042104.4107253-1-treapking@chromium.org>
On 12/01/2023 06:20, Pin-yen Lin wrote:
>
> This series introduces bindings for anx7625/it6505 to register Type-C
> mode-switch in their output endpoints, and use data-lanes property to
> describe the pin connections.
Please cc everybody on all patches. Having received just a single patch
made me spend time on having to look them up on lore.
>
> The first two patch modifies fwnode_graph_devcon_matches and
> cros_typec_init_ports to enable the registration of the switches.
>
> Patch 4~6 introduce the bindings for anx7625 and the corresponding driver
> modifications.
>
> Patch 7~9 add similar bindings and driver changes for it6505.
>
> v9: https://lore.kernel.org/all/20230109084101.265664-1-treapking@chromium.org/
> v8: https://lore.kernel.org/all/20230107102231.23682-1-treapking@chromium.org/
> v7: https://lore.kernel.org/all/20230105132457.4125372-1-treapking@chromium.org/
> v6: https://lore.kernel.org/all/20221124102056.393220-1-treapking@chromium.org/
> v5: https://lore.kernel.org/linux-usb/20220622173605.1168416-1-pmalani@chromium.org/
>
> Changes in v10:
> - Collected Reviewed-by and Tested-by tags
> - Replaced "void *" with "typec_mux_set_fn_t" for mux_set callbacks
> - Print out the node name when errors on parsing DT
> - Use dev_dbg instead of dev_warn when no Type-C switch nodes available
> - Made the return path of drm_dp_register_mode_switch clearer
> - Added a TODO for implementing orientation switch for anx7625
> - Updated the commit message for the absence of orientation switch
> - Fixed typo in the commit message
>
> Changes in v9:
> - Collected Reviewed-by tag
> - Fixed subject prefix again
> - Changed the naming of the example node for it6505
>
> Changes in v8:
> - Fixed the build issue when CONFIG_TYPEC=m
> - Fixed some style issues
> - Fixed the subject prefixes for the bindings patch
> - Fixed the bindings for data-lanes properties
>
> Changes in v7:
> - Fix the long comment lines
> - Extracted the common codes to a helper function
> - Fixed style issues in anx7625 driver
> - Removed DT property validation in anx7625 driver.
> - Fixed style issues in it6505 driver
> - Removed the redundant sleep in it6505 driver
> - Removed DT property validation in it6505 driver
> - Rebased to drm-misc-next
> - Fixed indentations in bindings patches
> - Added a new patch to fix indentations in Kconfig
>
> Changes in v6:
> - Changed it6505_typec_mux_set callback function to accommodate with
> the latest drm-misc patches
> - Changed the driver implementation to accommodate with the new binding
> - Dropped typec-switch binding and use endpoints and data-lanes properties
> to describe the pin connections
> - Added new patches (patch 1,2,4) to fix probing issues
> - Changed the bindings of it6505/anx7625 and modified the drivers
> accordingly
> - Merged it6505/anx7625 driver changes into a single patch
>
> Pin-yen Lin (7):
> drm/display: Add Type-C switch helpers
> dt-bindings: display: bridge: anx7625: Add mode-switch support
> drm/bridge: anx7625: Check for Type-C during panel registration
> drm/bridge: anx7625: Register Type C mode switches
> dt-bindings: display: bridge: it6505: Add mode-switch support
> drm/bridge: it6505: Fix Kconfig indentation
> drm/bridge: it6505: Register Type C mode switches
>
> Prashant Malani (2):
> device property: Add remote endpoint to devcon matcher
> platform/chrome: cros_ec_typec: Purge blocking switch devlinks
>
> .../display/bridge/analogix,anx7625.yaml | 99 ++++++++++++-
> .../bindings/display/bridge/ite,it6505.yaml | 93 ++++++++++--
> drivers/base/property.c | 15 ++
> drivers/gpu/drm/bridge/Kconfig | 21 +--
> drivers/gpu/drm/bridge/analogix/Kconfig | 1 +
> drivers/gpu/drm/bridge/analogix/anx7625.c | 105 +++++++++++++-
> drivers/gpu/drm/bridge/analogix/anx7625.h | 13 ++
> drivers/gpu/drm/bridge/ite-it6505.c | 119 +++++++++++++++-
> drivers/gpu/drm/display/drm_dp_helper.c | 134 ++++++++++++++++++
> drivers/platform/chrome/cros_ec_typec.c | 10 ++
> include/drm/display/drm_dp_helper.h | 17 +++
> 11 files changed, 598 insertions(+), 29 deletions(-)
>
--
With best wishes
Dmitry
next prev parent reply other threads:[~2023-01-12 4:34 UTC|newest]
Thread overview: 74+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-01-12 4:20 [PATCH v10 0/9] Register Type-C mode-switch in DP bridge endpoints Pin-yen Lin
2023-01-12 4:20 ` Pin-yen Lin
2023-01-12 4:20 ` [PATCH v10 1/9] device property: Add remote endpoint to devcon matcher Pin-yen Lin
2023-01-12 4:20 ` Pin-yen Lin
2023-01-12 13:27 ` Heikki Krogerus
2023-01-12 13:27 ` Heikki Krogerus
2023-01-12 13:32 ` Sakari Ailus
2023-01-12 13:32 ` Sakari Ailus
2023-01-12 22:31 ` Prashant Malani
2023-01-12 22:31 ` Prashant Malani
2023-01-13 17:40 ` Andy Shevchenko
2023-01-13 17:40 ` Andy Shevchenko
2023-01-16 13:07 ` Sakari Ailus
2023-01-16 13:07 ` Sakari Ailus
2023-01-20 21:15 ` Prashant Malani
2023-01-20 21:15 ` Prashant Malani
2023-01-12 4:20 ` [PATCH v10 2/9] platform/chrome: cros_ec_typec: Purge blocking switch devlinks Pin-yen Lin
2023-01-12 4:20 ` Pin-yen Lin
2023-01-12 13:31 ` Heikki Krogerus
2023-01-12 13:31 ` Heikki Krogerus
2023-01-12 4:20 ` [PATCH v10 3/9] drm/display: Add Type-C switch helpers Pin-yen Lin
2023-01-12 4:20 ` Pin-yen Lin
2023-01-12 4:40 ` Dmitry Baryshkov
2023-01-12 4:40 ` Dmitry Baryshkov
2023-01-12 5:19 ` Pin-yen Lin
2023-01-12 5:19 ` Pin-yen Lin
2023-01-12 5:24 ` Dmitry Baryshkov
2023-01-12 5:24 ` Dmitry Baryshkov
2023-01-12 5:48 ` Pin-yen Lin
2023-01-12 5:48 ` Pin-yen Lin
2023-01-12 5:50 ` Dmitry Baryshkov
2023-01-12 5:50 ` Dmitry Baryshkov
2023-01-12 8:17 ` Pin-yen Lin
2023-01-12 8:17 ` Pin-yen Lin
2023-01-12 8:37 ` Jani Nikula
2023-01-12 8:37 ` Jani Nikula
2023-01-13 8:19 ` Pin-yen Lin
2023-01-13 8:19 ` Pin-yen Lin
2023-01-13 9:23 ` Heikki Krogerus
2023-01-13 9:23 ` Heikki Krogerus
2023-01-13 16:57 ` Andy Shevchenko
2023-01-13 16:57 ` Andy Shevchenko
2023-01-12 4:20 ` [PATCH v10 4/9] dt-bindings: display: bridge: anx7625: Add mode-switch support Pin-yen Lin
2023-01-12 4:20 ` Pin-yen Lin
2023-01-12 4:43 ` Dmitry Baryshkov
2023-01-12 4:43 ` Dmitry Baryshkov
2023-01-12 5:26 ` Pin-yen Lin
2023-01-12 5:26 ` Pin-yen Lin
2023-01-12 5:27 ` Dmitry Baryshkov
2023-01-12 5:27 ` Dmitry Baryshkov
2023-01-12 22:56 ` Rob Herring
2023-01-12 22:56 ` Rob Herring
2023-01-12 4:21 ` [PATCH v10 5/9] drm/bridge: anx7625: Check for Type-C during panel registration Pin-yen Lin
2023-01-12 4:21 ` Pin-yen Lin
2023-01-12 4:21 ` [PATCH v10 6/9] drm/bridge: anx7625: Register Type C mode switches Pin-yen Lin
2023-01-12 4:21 ` Pin-yen Lin
2023-01-12 4:21 ` [PATCH v10 7/9] dt-bindings: display: bridge: it6505: Add mode-switch support Pin-yen Lin
2023-01-12 4:21 ` Pin-yen Lin
2023-01-12 4:21 ` [PATCH v10 8/9] drm/bridge: it6505: Fix Kconfig indentation Pin-yen Lin
2023-01-12 4:21 ` Pin-yen Lin
2023-01-12 4:21 ` [PATCH v10 9/9] drm/bridge: it6505: Register Type C mode switches Pin-yen Lin
2023-01-12 4:21 ` Pin-yen Lin
2023-01-12 12:32 ` Chen-Yu Tsai
2023-01-12 12:32 ` Chen-Yu Tsai
2023-01-12 4:34 ` Dmitry Baryshkov [this message]
2023-01-12 4:34 ` [PATCH v10 0/9] Register Type-C mode-switch in DP bridge endpoints Dmitry Baryshkov
2023-01-12 5:07 ` Pin-yen Lin
2023-01-12 5:07 ` Pin-yen Lin
2023-01-12 10:36 ` Andy Shevchenko
2023-01-12 10:36 ` Andy Shevchenko
2023-01-12 22:44 ` Rob Herring
2023-01-12 22:44 ` Rob Herring
2023-01-13 8:09 ` Pin-yen Lin
2023-01-13 8:09 ` Pin-yen Lin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=61ba2880-6784-1014-a7a2-a1eecc4e810e@linaro.org \
--to=dmitry.baryshkov@linaro.org \
--cc=Laurent.pinchart@ideasonboard.com \
--cc=airlied@gmail.com \
--cc=allen.chen@ite.com.tw \
--cc=andriy.shevchenko@linux.intel.com \
--cc=andrzej.hajda@intel.com \
--cc=angelogioacchino.delregno@collabora.com \
--cc=bleung@chromium.org \
--cc=chrome-platform@lists.linux.dev \
--cc=daniel@ffwll.ch \
--cc=dengshaomin@cdjrlc.com \
--cc=devicetree@vger.kernel.org \
--cc=dianders@chromium.org \
--cc=djrscally@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=gregkh@linuxfoundation.org \
--cc=groeck@chromium.org \
--cc=heikki.krogerus@linux.intel.com \
--cc=hsinyi@chromium.org \
--cc=imre.deak@intel.com \
--cc=jani.nikula@intel.com \
--cc=javierm@redhat.com \
--cc=jernej.skrabec@gmail.com \
--cc=jonas@kwiboo.se \
--cc=jose.exposito89@gmail.com \
--cc=keescook@chromium.org \
--cc=krzysztof.kozlowski+dt@linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lyude@redhat.com \
--cc=marex@denx.de \
--cc=neil.armstrong@linaro.org \
--cc=nfraprado@collabora.com \
--cc=pmalani@chromium.org \
--cc=rafael@kernel.org \
--cc=robert.foss@linaro.org \
--cc=robh+dt@kernel.org \
--cc=sakari.ailus@linux.intel.com \
--cc=swboyd@chromium.org \
--cc=treapking@chromium.org \
--cc=tzimmermann@suse.de \
--cc=u.kleine-koenig@pengutronix.de \
--cc=ville.syrjala@linux.intel.com \
--cc=xji@analogixsemi.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.