From: Bryan Huntsman <bryanh@codeaurora.org>
To: Andrew Morton <akpm@linux-foundation.org>,
Anirudh Ghayal <aghayal@codeaurora.org>,
Samuel Ortiz <sameo@linux.intel.com>,
tsoni@codeaurora.org, David Brown <davidb@codeaurora.org>
Cc: rtc-linux@googlegroups.com, Stephen Boyd <sboyd@codeaurora.org>,
linux-arm-msm@vger.kernel.org, Wan ZongShun <mcuos.com@gmail.com>,
Ashay Jaiswal <ashayj@codeaurora.org>
Subject: Re: [rtc-linux] [PATCH V4] drivers: rtc: Add support for Qualcomm PMIC8xxx RTC
Date: Tue, 07 Jun 2011 19:39:38 -0700 [thread overview]
Message-ID: <4DEEE0EA.9040401@codeaurora.org> (raw)
In-Reply-To: <20110607173227.953bf25c.akpm@linux-foundation.org>
On 06/07/2011 05:32 PM, Andrew Morton wrote:
> On Fri, 27 May 2011 10:26:18 +0530
> Anirudh Ghayal <aghayal@codeaurora.org> wrote:
>
>> This patch adds support for PMIC8xxx based RTC.
>> PMIC8xxx is Qualcomm's power management IC that
>> internally houses an RTC module. This driver
>> communicates with the PMIC module over SSBI bus.
>>
>
> x86_64 allmodconfig:
>
> drivers/rtc/rtc-pm8xxx.c: In function 'pm8xxx_rtc_probe':
> drivers/rtc/rtc-pm8xxx.c:400: error: implicit declaration of function 'mfd_get_data'
> drivers/rtc/rtc-pm8xxx.c:400: warning: initialization makes pointer from integer without a cast
>
> A tree-wide grep in 3.0-rc2 for "mfd_get_data" shows no definition,
> although ./drivers/input/keyboard/pmic8xxx-keypad.c and
> ./drivers/input/misc/pmic8xxx-pwrkey.c are calling it.
>
> What's happening here?
Samuel, this looks like it (92d57a7) was added to your branch after the
mfd_get_data cleanup (ba279f5). Trilok/Anirudh, would you please send
Samuel a patch to clean up this compilation error? Thanks.
- Bryan
$ git log --oneline --graph 9f1912c48ce829d24789e3e5d499de0d44d3306a
* 9f1912c Merge branch 'for-next' of
git://git.kernel.org/pub/scm/linux/kernel/git/sameo/mfd-2.6
|\
| * 0996910 regulator: Fix 88pm8607.c printk format warning
| * 92d57a7 input: Add support for Qualcomm PMIC8XXX power key
| * 39325b5 input: Add Qualcomm pm8xxx keypad controller driver
| * 7e6502d mfd: Add omap-usbhs runtime PM support
| * 74e32d1 mfd: Fix ASIC3 SD Host Controller Configuration size
| * d910774 mfd: Fix omap_usbhs_alloc_children error handling
| * 6eb6fbb mfd: Fix omap usbhs crash when rmmoding ehci or ohci
| * 13ca4f6 mfd: Add ASIC3 LED support
| * 7d9e7e9 leds: Add ASIC3 LED support
| * 4a7c00c mfd: Update twl4030-code maintainer e-mail address
| * ed13941 mfd: Correct the name and bitmask for ab8500-gpadc BTempPullUp
| * c9c9513 mfd: Add manual ab8500-gpadc batt temp activation for AB8500 3.0
| * 863dde5 mfd: Provide ab8500-core enumerators for chip cuts
| * eac78a2 mfd: Check twl4030-power remove script error condition after
i2cwrite
| * 6523b14 mfd: Fix twl6030 irq definitions
| * 521d8ec mfd: Add phoenix lite (twl6025) support to twl6030
| * 586e1a1 mfd: Avoid to use constraint name in 88pm860x regulator driver
| * ebf9988 mfd: Remove checking on max8925 regulator[0]
| * cea438d mfd: Remove unused parameter from 88pm860x API
| * f5fb758 mfd: Avoid to allocate 88pm860x static platform data
| * 008b304 mfd: Add rtc support to 88pm860x
| * db13c04 regulator: Enable max8925 i2c sequence for control
| * 2402ca5 power_supply: Use max8925 platform_data from cell
| * e6f1945 mfd: Fix tps6586x GPIO value setting
| * ea91db5 MAINTAINERS: Add patterns for pmic 8921 files to MSM subsystem
| * c013f0a mfd: Add pm8xxx irq support
| * cbdb53e mfd: Add Qualcomm PMIC 8921 core driver
| * 1305134 mfd: Group NXP PCF50633* drivers together
| * 37df003 mfd: Group TPSxxxxx power management chips together
| * ca972d1 mfd: TWL5030 version checking in twl-core
| * d7ac829 mfd: Modifying the twl4030-power macro name Main_Ref to all caps
| * 1f968ff mfd: Correct the twl4030-power warning print during script
loading
| * a5156f1 mfd: Fix build warning on 88pm860x
| * 5d76ca3 mfd: Fix wl1273 warning
| * bc86fce mfd: Continue with IRQ setup even if we don't have PMIC main IRQ
| * 0b14c22 mfd: Provide platform data for WM831x GPIO configuration
| * 8997619 mfd: Remove compatibility interface for WM831x specific IRQ API
| * ba279f5 mfd: Remove mfd_data
--
Sent by an employee of the Qualcomm Innovation Center, Inc.
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum.
next prev parent reply other threads:[~2011-06-08 2:39 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-05-27 4:56 [PATCH V4] drivers: rtc: Add support for Qualcomm PMIC8xxx RTC Anirudh Ghayal
2011-05-27 5:05 ` Anirudh Ghayal
2011-06-01 3:58 ` Anirudh Ghayal
2011-06-01 8:47 ` Samuel Ortiz
2011-06-01 8:52 ` Samuel Ortiz
2011-06-01 9:22 ` Wan ZongShun
2011-06-08 0:32 ` [rtc-linux] " Andrew Morton
2011-06-08 2:39 ` Bryan Huntsman [this message]
2011-06-08 9:11 ` Samuel Ortiz
2011-06-08 9:47 ` Anirudh Ghayal
2011-06-09 4:58 ` [PATCH V1] drivers: rtc: Do not use mfd_get_data Anirudh Ghayal
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=4DEEE0EA.9040401@codeaurora.org \
--to=bryanh@codeaurora.org \
--cc=aghayal@codeaurora.org \
--cc=akpm@linux-foundation.org \
--cc=ashayj@codeaurora.org \
--cc=davidb@codeaurora.org \
--cc=linux-arm-msm@vger.kernel.org \
--cc=mcuos.com@gmail.com \
--cc=rtc-linux@googlegroups.com \
--cc=sameo@linux.intel.com \
--cc=sboyd@codeaurora.org \
--cc=tsoni@codeaurora.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.