| Message ID | 20230112042104.4107253-1-treapking@chromium.org (mailing list archive) |
|---|---|
| Headers | show |
| Series | Register Type-C mode-switch in DP bridge endpoints | expand |
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(-) >
Hi Dmitry, On Thu, Jan 12, 2023 at 12:34 PM Dmitry Baryshkov <dmitry.baryshkov@linaro.org> wrote: > > 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. Sorry about that. I'll prevent that in the future series. > > > > > 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 > Thanks and regards, Pin-yen
On Thu, Jan 12, 2023 at 06:34:52AM +0200, Dmitry Baryshkov wrote: > 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. Sorry but this seems a bit outdated requirement nowadays. Why? Because we have `b4` in each famous Linux distro and nice lore.kernel.org. It's really easy for the experienced maintainer to catch up the whole thread. Putting _all_ people on all patches may be an overkill. What people should really get is the cover letter (assuming that it explains well the structure of the series). The rest depends.
On Wed, Jan 11, 2023 at 10:21 PM Pin-yen Lin <treapking@chromium.org> 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. > > 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 4 versions in a week! Please slow down your pace. When you send a new version, you move to the end of my review queue. IIRC, these 2 chips are a bit different in what the mode switch or muxing looks like. One had a built-in mux and the other doesn't? Do I have to go research this again? No, you need to explain all this in this series. Rob
Hi Rob, On Fri, Jan 13, 2023 at 6:44 AM Rob Herring <robh+dt@kernel.org> wrote: > > On Wed, Jan 11, 2023 at 10:21 PM Pin-yen Lin <treapking@chromium.org> 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. > > > > 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 > > 4 versions in a week! Please slow down your pace. When you send a new > version, you move to the end of my review queue. I see. I'll keep this in mind in the future series. > > IIRC, these 2 chips are a bit different in what the mode switch or > muxing looks like. One had a built-in mux and the other doesn't? Do I > have to go research this again? No, you need to explain all this in > this series. Yes, anx7625 has a built-in mux while it6505 doesn't, but it's for another use case. IIUC the built-in mux in anx7625 is designed for automatically switching between two orientations of a single Type-C connector, and in that case we might need to register an orientation switch. But we don't have hardware for this use case. The use case this series aimed is having two downstreams for the bridges, and registering two mode switches to switch between them. In this use case, the built-in mux of anx7625 is not used and the behavior of the switches is the same as it6505. Explanations and TODOs have been added in the anx7625 driver change. I can also mention this in the cover letter in the future series and please let me know if anything is not clear for you. > > Rob Thanks and regards, Pin-yen