Hi Rasmus, On Tue, 2 Jun 2020 at 13:44, Rasmus Villemoes <rasmus.villem...@prevas.dk> wrote: > > On 02/06/2020 21.29, Simon Glass wrote: > > Hi Rasmus, > > > > On Tue, 2 Jun 2020 at 12:40, Rasmus Villemoes > > <rasmus.villem...@prevas.dk> wrote: > >> > >> Urgh. The name rtc_read() is already used for a local helper by a number > >> of rtc drivers (also rtc_write, for somewhat fewer drivers). So I can > >> still call the methods ->read and ->write, but the functions will need > >> another name. Probably dm_rtc_read/dm_rtc_write, since this is only for > >> DM-enabled drivers anyway, and matches the existing dm_rtc_get/dm_rtc_set. > > > > The conflict is OK, since at some point those drivers will be updated > > to DM or removed. I'd rather avoid the dm_ prefix if not necessary. > > There are some DM-enabled drivers that still use those names as local > helpers, e.g. rx8025.c and pt7c4338.c.
OK then they probably need a prefix of the driver name on those functions. Regards, Simon