From mboxrd@z Thu Jan  1 00:00:00 1970
From: Simon Glass <sjg@chromium.org>
Date: Thu, 9 Apr 2020 10:25:49 -0600
Subject: Calling i2c set speed twice for i2c_mux_bus
In-Reply-To: <d606a21c-3537-43e7-712f-0a32713eded9@xilinx.com>
References: <d606a21c-3537-43e7-712f-0a32713eded9@xilinx.com>
Message-ID: <CAPnjgZ2tELtmUt_bcLyxxwxiWLaypLmsLPN9=2e5djrV9dTLag@mail.gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Hi Michal,

On Thu, 9 Apr 2020 at 08:03, Michal Simek <michal.simek@xilinx.com> wrote:
>
> Hi Heiko and Simon,
>
> I have find out one bug in i2c class. I am using zcu104 revC board which
> has dts in mainline where i2c1 has i2c mux with some channels.
> In DT clock-frequency = <400000>; is specified and it is read in
> i2c_post_probe(). But i2c_mux_bus_drv is also UCLASS_I2C which means
> that post probe is called for it too. And because clock-frequency
> property is not there default 100k is used.
>
> I think that is bug and should be fixed.
> Heiko: Are you aware about this issue?
>
> Simon: I think that make sense to use some flags to skip calling
> post_probe in this case. Or any suggestion how to fix it would be great.

Ick, hope not. Can you please point me to the board config and the device tree?

Regards,
Simon