Hi On Mon, 11 Sept 2023 at 16:13, Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> wrote: > > On 11/09/2023 12:31, Naresh Solanki wrote: > > >>> diff --git a/Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.yaml b/Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.yaml > >>> index 2d7bb998b0e9..fa73eadfdf7b 100644 > >>> --- a/Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.yaml > >>> +++ b/Documentation/devicetree/bindings/i2c/i2c-mux-pca954x.yaml > >>> @@ -71,6 +71,23 @@ properties: > >>> description: A voltage regulator supplying power to the chip. On PCA9846 > >>> the regulator supplies power to VDD2 (core logic) and optionally to VDD1. > >>> > >>> + maxim,isolate-stuck-channel: > >>> + type: boolean > >>> + description: Allows to use non faulty channels while a stuck channel is > >>> + isolated from the upstream bus. If not set all channels are isolated from > >>> + the upstream bus until the fault is cleared. > >> > >> Nothing improved here. As I said, please provide arguments or drop this > >> property. > > These features cannot be enabled by default because doing so may lead > > to unexpected behavior, such as bus disconnections(although that > > wasn't expected). > > These features should only be enabled after they have been validated > > by the board designer. > > Therefore, they cannot be enabled by default. > > And what is needed to validate them for given board? IOW, what changes > in hardware design that it can or cannot be used? Enabling PRECONNECT_TEST in my setup didn't work Although this wasn't expected. Regards, Naresh > > Best regards, > Krzysztof >
