于 2017年4月18日 GMT+08:00 下午6:38:09, Chen-Yu Tsai <[email protected]> 写到: >On Mon, Apr 17, 2017 at 7:57 PM, Icenowy Zheng <[email protected]> wrote: >> As axp20x-regulator now supports AXP803, add a cell for it. >> >> Signed-off-by: Icenowy Zheng <[email protected]> >> --- >> Changes in v3: >> - Make the new cell one-liner. >> >> drivers/mfd/axp20x.c | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/drivers/mfd/axp20x.c b/drivers/mfd/axp20x.c >> index 1dc6235778eb..431b7f118606 100644 >> --- a/drivers/mfd/axp20x.c >> +++ b/drivers/mfd/axp20x.c >> @@ -848,7 +848,8 @@ static struct mfd_cell axp803_cells[] = { >> .name = "axp20x-pek", >> .num_resources = >ARRAY_SIZE(axp803_pek_resources), >> .resources = axp803_pek_resources, >> - } >> + }, >> + { .name = "axp20x-regulator" } > >It's best to have a trailing comma, so we don't have to change the line >again when we add more cells, like you just did with the previous line.
Should I also add it in previous mfd patch? (and remove the addition of the comma in this patch) > >Otherwise, > >Acked-by: Chen-Yu Tsai <[email protected]>
