Re: [PATCH v2] staging: media: Switch back to struct platform_driver::remove()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Oct 11, 2024 at 11:46 PM Uwe Kleine-König
<u.kleine-koenig@xxxxxxxxxxxx> wrote:
>
> Hello Sergio,
>
> On Fri, Oct 11, 2024 at 09:39:09AM +0200, Sergio Paracuellos wrote:
> > On Fri, Oct 11, 2024 at 9:09 AM Uwe Kleine-König
> > <u.kleine-koenig@xxxxxxxxxxxx> wrote:
> > >
> > > After commit 0edb555a65d1 ("platform: Make platform_driver::remove()
> > > return void") .remove() is (again) the right callback to implement for
> > > platform drivers.
> > >
> > > Convert all platform drivers below drivers/staging/media/ to use
> > > .remove(), with the eventual goal to drop struct
> > > platform_driver::remove_new(). As .remove() and .remove_new() have the
> > > same prototypes, conversion is done by just changing the structure
> > > member name in the driver initializer.
> > >
> > > Signed-off-by: Uwe Kleine-König <u.kleine-koenig@xxxxxxxxxxxx>
> >
> > I sent a patch the other day to convert all staging drivers to use
> > remove which was applied yesterday.
> >
> > See: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/staging.git/commit/?h=staging-next&id=c1a5060ec80020ce879fa5b2a16875bd9a5ab930
>
> If you really must grab this quest from me, please coordinate, or at
> least Cc: me on your patches.

I did not know you already have patches for staging and only sent this
to the staging list itself. Sorry for that. If you already have
patches for remaining stuff I stop sending patches about this. Sorry
for the inconvenience and let me know if you need my help in any way.

Best regards,
    Sergio Paracuellos
>
> Uwe





[Index of Archives]     [Linux Driver Development]     [Linux Driver Backports]     [DMA Engine]     [Linux GPIO]     [Linux SPI]     [Video for Linux]     [Linux USB Devel]     [Linux Coverity]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux