linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: marek.vasut@gmail.com (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/3] wm97xx_batt: Remove redundant header file
Date: Sat, 12 Jun 2010 14:47:33 +0200	[thread overview]
Message-ID: <201006121447.33221.marek.vasut@gmail.com> (raw)
In-Reply-To: <AANLkTimb6TqQLLHDLI_qInwScZUQpYIDxNymE8SUh_9J@mail.gmail.com>

Dne So 12. ?ervna 2010 11:16:58 Eric Miao napsal(a):
> On Fri, Jun 4, 2010 at 9:07 AM, Marek Vasut <marek.vasut@gmail.com> wrote:
> > Signed-off-by: Marek Vasut <marek.vasut@gmail.com>
> > ---
> >  include/linux/wm97xx_batt.h |   16 ----------------
> >  1 files changed, 0 insertions(+), 16 deletions(-)
> >  delete mode 100644 include/linux/wm97xx_batt.h
> > 
> > diff --git a/include/linux/wm97xx_batt.h b/include/linux/wm97xx_batt.h
> > deleted file mode 100644
> > index a1d6419..0000000
> > --- a/include/linux/wm97xx_batt.h
> > +++ /dev/null
> > @@ -1,16 +0,0 @@
> > -#ifndef _LINUX_WM97XX_BAT_H
> > -#define _LINUX_WM97XX_BAT_H
> > -
> > -#include <linux/wm97xx.h>
> > -
> > -#warning This file will be removed soon, use wm97xx.h instead!
> > -
> > -#define wm97xx_batt_info wm97xx_batt_pdata
> > -
> > -#ifdef CONFIG_BATTERY_WM97XX
> > -void wm97xx_bat_set_pdata(struct wm97xx_batt_info *data);
> > -#else
> > -static inline void wm97xx_bat_set_pdata(struct wm97xx_batt_info *data)
> > {} -#endif
> > -
> > -#endif
> 
> A git grep of wm97xx_batt.h shows:
> 
> drivers/power/wm97xx_battery.c:/* The interface is deprecated, as well
> as linux/wm97xx_batt.h */
> 
> So you may want to remove that part as well. Otherwise looks good to me.

Thanks, good catch. But let's push this through a separate patch, shall we?

  reply	other threads:[~2010-06-12 12:47 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-04  1:07 [PATCH 1/3] pxa/colibri270: Fix typo in colibri_pxa270_flash Marek Vasut
2010-06-04  1:07 ` [PATCH 2/3] pxa: Fix all devices using wm97xx_batt.h Marek Vasut
2010-06-06 17:46   ` Robert Jarzmik
2010-06-06 18:10   ` Tomáš 'Sleep_Walker' Čech
2010-06-07  9:17   ` Mark Brown
2010-06-12  9:15     ` Eric Miao
2010-06-04  1:07 ` [PATCH 3/3] wm97xx_batt: Remove redundant header file Marek Vasut
2010-06-07  9:18   ` Mark Brown
2010-06-12  9:16   ` Eric Miao
2010-06-12 12:47     ` Marek Vasut [this message]
2010-06-04  6:27 ` [PATCH 1/3] pxa/colibri270: Fix typo in colibri_pxa270_flash Daniel Mack
2010-06-04  6:55   ` Marek Vasut

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=201006121447.33221.marek.vasut@gmail.com \
    --to=marek.vasut@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).