Dear David,

 Thank you for reviewing.
 I'll update patchset according to your comment.

 Sincerely,
 Taku Izumi

> -----Original Message-----
> From: David Miller [mailto:da...@davemloft.net]
> Sent: Friday, August 21, 2015 7:49 AM
> To: Izumi, Taku/泉 拓
> Cc: netdev@vger.kernel.org; platform-driver-...@vger.kernel.org; 
> dvh...@infradead.org; rk...@redhat.com;
> alexander.h.du...@redhat.com; linux-a...@vger.kernel.org; j...@perches.com; 
> sergei.shtyl...@cogentembedded.com;
> step...@networkplumber.org; yasu.isim...@gmail.com
> Subject: Re: [PATCH v2.2 01/22] fjes: Introduce FUJITSU Extended Socket 
> Network Device driver
> 
> From: Taku Izumi <izumi.t...@jp.fujitsu.com>
> Date: Thu, 20 Aug 2015 17:46:05 +0900
> 
> > +obj-$(CONFIG_FUJITSU_ES) += fjes.o
> > +
> > +fjes-objs := fjes_main.o
> > +
> 
> Please do not have trailing empty lines in any files you add
> or edit, 'git' warns about this even when applying patches.
> 
> > +static int fjes_acpi_add(struct acpi_device *device)
> > +{
> > +   acpi_status status;
> > +   struct acpi_buffer buffer = { ACPI_ALLOCATE_BUFFER, NULL};
> > +   union acpi_object *str;
> > +   char str_buf[sizeof(FJES_ACPI_SYMBOL) + 1];
> > +   int result;
> > +   struct platform_device *plat_dev;
> 
> Please order your local variables in "reverse christmas tree" order, which
> means longer lines come before shorter ones.
> 
> Please correct this problem in your entire submission, as I am not going
> to point out each and every other place where this problem exists.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to