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 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).