On Sat, Jan 4, 2025 at 1:02 AM Andi Shyti <andi.shyti@xxxxxxxxxx> wrote:
>
> > > > -
> > > > -/* default platform data to use if not supplied in the platform_device */
> > > > -static struct davinci_i2c_platform_data davinci_i2c_platform_data_default = {
> > > > - .bus_freq = 100,
> > > > - .bus_delay = 0,
> > >
> > > what happened to bus_delay?
> > >
> >
> > bus_delay is not set by means other than platform data and it defaults
> > to 0 so it's safe to just remove it.
>
> yes, but how is it related to this patch? Can we put it on a
> different patch?
>
No, why? This patch removes platform data and all bits and pieces
associated with it. Splitting it into two just to first remove
bus_delay and then the rest doesn't make much sense IMO. The argument
that it's already not used would be incorrect as it IS used by
platform data (even though it itself is no longer defined anywhere).
I'd keep it as is.
Bart
