Hi Uwe, On Fri, Jan 13, 2023 at 8:39 AM Lad, Prabhakar <prabhakar.csengg@xxxxxxxxx> wrote: > > Hi Uwe, > > On Tue, Dec 13, 2022 at 3:36 PM Uwe Kleine-König > <u.kleine-koenig@xxxxxxxxxxxxxx> wrote: > > > > Hello, > > > > this series removes all platform remove functions that only return zero > > below drivers/media. There is no reason to have these, as the only > > caller is platform core code doing: > > > > if (drv->remove) { > > int ret = drv->remove(dev); > > > > if (ret) > > dev_warn(...) > > } > > > > (in platform_remove()) and so having no remove function is both > > equivalent and simpler. > > > > Uwe Kleine-König (4): > > media: ti/davinci: vpbe_osd: Drop empty platform remove function > > media: ti/davinci: vpbe_venc: Drop empty platform remove function > Ive reviewed the above two patches. I believe Hans will pick this up > soon for v6.3. > I totally forgot davinci drivers are being dropped now [0]. [0] https://lore.kernel.org/lkml/20221019152947.3857217-15-arnd@xxxxxxxxxx/ Cheers, Prabhakar