[U-Boot] [Patch] ./drivers/pci/pci.c

Wolfgang Denk wd at denx.de
Sun Mar 21 17:58:37 CET 2010


Dear FUJITA Kazutoshi,

In message <20100309.232841.106130457.fujita at soum.co.jp> you wrote:
>
> Subject: Re: [U-Boot] [Patch] ./drivers/pci/pci.c
> Date: Tue, 09 Mar 2010 14:27:25 +0100
> 
> > Please provide a useful commit message, and add your Signed-off-by:
> > line.
> 
> sorry, how about this?
> 
> regards,

That's actually still nort usable.

First, the Subject line would result in a commit message tile of
"./drivers/pci/pci.c" - do you think this is a meaningful description
of what this patch is doing?

Second, all the above text would become part of the commit message -
this makes no sense either.

Please use git-format-patch to format the patches, add commets _below_
the "---" line, and use git-send-email to submit the patches.

> this patch fixes to deal with PCI_BASE_ADDRESS_5 which was forgotten
> in pci_hose_config_device()
> 
> Signed-off-by: FUJITA Kazutoshi <fujita at soum.co.jp>
> 
> 
> diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
> index d6d2d6e..cd64a87 100644
> --- a/drivers/pci/pci.c
> +++ b/drivers/pci/pci.c
> @@ -388,7 +388,7 @@ int pci_hose_config_device(struct pci_controller *hose,

I edited the commit message for you, and applied the patch now.
Thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"You're just jealous." "What, of an overgrown puppy  with  a  single-
figure IQ?"                      - Terry Pratchett, _Moving Pictures_


More information about the U-Boot mailing list