[PATCH v2 17/23] staging: comedi: addi_apci_2200: use addi_watchdog module
Greg Kroah-Hartman
gregkh at linuxfoundation.org
Fri Jan 18 21:16:13 UTC 2013
On Fri, Jan 18, 2013 at 03:03:24PM -0600, H Hartley Sweeten wrote:
> On Friday, January 18, 2013 1:55 PM, Greg Kroah-Hartman wrote:
> > On Thu, Jan 17, 2013 at 05:41:44PM -0700, H Hartley Sweeten wrote:
> >> The current watchdog support in hwdrv_apci2200.c does not conform
> >> to the comedi API.
> >>
> >> Use the addi_watchdog module to provide support for the watchdog
> >> subdevice and remove the now unneeded hwdrv_apci2200.c file.
> >>
> >> Signed-off-by: H Hartley Sweeten <hsweeten at visionengravers.com>
> >> Cc: Ian Abbott <abbotti at mev.co.uk>
> >> ---
> >> drivers/staging/comedi/Kconfig | 1 +
> >> .../comedi/drivers/addi-data/hwdrv_apci2200.c | 176 ---------------------
> >> drivers/staging/comedi/drivers/addi_apci_2200.c | 36 ++---
> >> 3 files changed, 10 insertions(+), 203 deletions(-)
> >> delete mode 100644 drivers/staging/comedi/drivers/addi-data/hwdrv_apci2200.c
> >
> > Are you sure you made this patch correctly? I get the following error
> > when applying it:
> >
> > patching file drivers/staging/comedi/Kconfig
> > patching file drivers/staging/comedi/drivers/addi-data/hwdrv_apci2200.c
> > Reversed (or previously applied) patch detected! Assume -R? [n]
> > Apply anyway? [n]
> > Skipping patch.
> > 1 out of 1 hunk ignored -- saving rejects to file drivers/staging/comedi/drivers/addi-data/hwdrv_apci2200.c.rej
> > patching file drivers/staging/comedi/drivers/addi_apci_2200.c
> >
> > So I've stopped applying this series here, care to resend the 17-23 patches
> > again after fixing this up?
>
> That's strange. All I did for this patch was:
>
> edit drivers/staging/comedi/Kconfig
> edit drivers/staging/comedi/drivers/addi_apci_2200.c
> git rm drivers/staging/comedi/drivers/addi-data/hwdrv_apci2200.c
> git commit .
>
> The diff --stat above looks right.
>
> I did the same basic sequence for [PATCH 19/22] that you ran into a problem with.
>
> I'll look into it and see where is screwed up...
Both of them ended up encoded in mime types, so something might be odd
with that?
greg k-h
More information about the devel
mailing list