Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1246377
| From | Borun Fu <borun.fu@linux.intel.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs |
| Date | 2015-10-14 09:50 +0200 |
| Message-ID | <qjoYW-ib-23@gated-at.bofh.it> (permalink) |
| References | <qiFyN-em-1@gated-at.bofh.it> <qj2Yr-Lw-35@gated-at.bofh.it> <qjcEq-6q9-19@gated-at.bofh.it> <qjome-7X8-13@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
So the patch will be refined to remove the comment as follows. Any else comments?
@@ -572,6 +587,11 @@ static struct mfd_cell axp288_cells[] = {
.resources = axp288_fuel_gauge_resources,
},
{
+ .name = "axp20x-pek",
+ .num_resources = ARRAY_SIZE(axp288_power_button_resources),
+ .resources = axp288_power_button_resources,
+ },
+ {
.name = "axp288_pmic_acpi",
},
};
On 10/14/2015 15:08, Lee Jones wrote:
> On Tue, 13 Oct 2015, Yang, Fei wrote:
>
>>>> static struct resource axp288_fuel_gauge_resources[] = {
>>>> {
>>>> .start = AXP288_IRQ_QWBTU,
>>>> @@ -572,6 +587,11 @@ static struct mfd_cell axp288_cells[] = {
>>>> .resources = axp288_fuel_gauge_resources,
>>>> },
>>>> {
>>>> + .name = "axp20x-pek", /* axp20x-pek works for AXP288 as well */
>>>
>>> Doesn't this change insinuate that?
>>>
>>> If axp20x-pek didn't work for AXP288 you wouldn't have put it in this structure right?
>>
>> Are you suggesting to remove the comment? Just want to make sure I understand you correctly.
>
> I am.
>
>>>> + .num_resources = ARRAY_SIZE(axp288_power_button_resources),
>>>> + .resources = axp288_power_button_resources,
>>>> + },
>>>> + {
>>>> .name = "axp288_pmic_acpi",
>>>> },
>>>> };
>>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs Borun Fu <borun.fu@linux.intel.com> - 2015-10-12 09:20 +0200
Re: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs Lee Jones <lee.jones@linaro.org> - 2015-10-13 10:20 +0200
RE: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs "Yang, Fei" <fei.yang@intel.com> - 2015-10-13 20:40 +0200
Re: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs Lee Jones <lee.jones@linaro.org> - 2015-10-14 09:10 +0200
Re: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs Borun Fu <borun.fu@linux.intel.com> - 2015-10-14 09:50 +0200
Re: [PATCH] mfd: axp20x: Add a cell for the power button part of the,axp288 PMICs Lee Jones <lee.jones@linaro.org> - 2015-10-14 10:10 +0200
csiph-web