public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file
@ 2016-10-17  1:46 Wenyou Yang
  2016-10-17  5:42 ` Heiko Schocher
  2016-10-17  5:46 ` Heiko Schocher
  0 siblings, 2 replies; 4+ messages in thread
From: Wenyou Yang @ 2016-10-17  1:46 UTC (permalink / raw)
  To: u-boot

Since the 'clk_client.h' doesn't exist, it should be 'clk.h'.

Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Andreas Bie?mann <andreas@biessmann.org>
---

Changes in v3:
 - Collect Reviewed-by tag.

Changes in v2:
 - Collect Reviewed-by tag.

 drivers/i2c/at91_i2c.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c
index 8e9c3ad..d71f75c 100644
--- a/drivers/i2c/at91_i2c.c
+++ b/drivers/i2c/at91_i2c.c
@@ -8,7 +8,7 @@
 
 #include <asm/io.h>
 #include <common.h>
-#include <clk_client.h>
+#include <clk.h>
 #include <dm.h>
 #include <errno.h>
 #include <fdtdec.h>
-- 
2.7.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file
  2016-10-17  1:46 [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file Wenyou Yang
@ 2016-10-17  5:42 ` Heiko Schocher
  2016-10-17  5:46 ` Heiko Schocher
  1 sibling, 0 replies; 4+ messages in thread
From: Heiko Schocher @ 2016-10-17  5:42 UTC (permalink / raw)
  To: u-boot

Hello Wenyou Yang,

Am 17.10.2016 um 03:46 schrieb Wenyou Yang:
> Since the 'clk_client.h' doesn't exist, it should be 'clk.h'.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Andreas Bie?mann <andreas@biessmann.org>
> ---
>
> Changes in v3:
>   - Collect Reviewed-by tag.
>
> Changes in v2:
>   - Collect Reviewed-by tag.
>
>   drivers/i2c/at91_i2c.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c
> index 8e9c3ad..d71f75c 100644
> --- a/drivers/i2c/at91_i2c.c
> +++ b/drivers/i2c/at91_i2c.c
> @@ -8,7 +8,7 @@
>
>   #include <asm/io.h>
>   #include <common.h>
> -#include <clk_client.h>
> +#include <clk.h>
>   #include <dm.h>
>   #include <errno.h>
>   #include <fdtdec.h>
>

Reviewed-by: Heiko Schocher <hs@denx.de>

bye,
Heiko
-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file
  2016-10-17  1:46 [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file Wenyou Yang
  2016-10-17  5:42 ` Heiko Schocher
@ 2016-10-17  5:46 ` Heiko Schocher
  2016-10-17  5:51   ` Wenyou.Yang at microchip.com
  1 sibling, 1 reply; 4+ messages in thread
From: Heiko Schocher @ 2016-10-17  5:46 UTC (permalink / raw)
  To: u-boot

Hello Wenyou Yang,

Am 17.10.2016 um 03:46 schrieb Wenyou Yang:
> Since the 'clk_client.h' doesn't exist, it should be 'clk.h'.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
> Reviewed-by: Andreas Bie?mann <andreas@biessmann.org>
> ---
>
> Changes in v3:
>   - Collect Reviewed-by tag.
>
> Changes in v2:
>   - Collect Reviewed-by tag.
>
>   drivers/i2c/at91_i2c.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)

Hups, I just want to apply this patch to u-boot-i2c.git ... but,
this patch is already applied in mainline:

commit 76062b9cdbe756eff75b99beaf6e94e8bb059431
Author: Wenyou Yang <wenyou.yang@atmel.com>
Date:   Tue Sep 13 10:40:31 2016 +0800

     i2c: at91_i2c: Fix the wrong include file

So, I only set your v3 version into state "Not applicable"

bye,
Heiko
>
> diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c
> index 8e9c3ad..d71f75c 100644
> --- a/drivers/i2c/at91_i2c.c
> +++ b/drivers/i2c/at91_i2c.c
> @@ -8,7 +8,7 @@
>
>   #include <asm/io.h>
>   #include <common.h>
> -#include <clk_client.h>
> +#include <clk.h>
>   #include <dm.h>
>   #include <errno.h>
>   #include <fdtdec.h>
>

-- 
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file
  2016-10-17  5:46 ` Heiko Schocher
@ 2016-10-17  5:51   ` Wenyou.Yang at microchip.com
  0 siblings, 0 replies; 4+ messages in thread
From: Wenyou.Yang at microchip.com @ 2016-10-17  5:51 UTC (permalink / raw)
  To: u-boot



> -----Original Message-----
> From: Heiko Schocher [mailto:hs at denx.de]
> Sent: 2016?10?17? 13:46
> To: Wenyou Yang - A41535 <Wenyou.Yang@microchip.com>
> Cc: U-Boot Mailing List <u-boot@lists.denx.de>; Wenyou Yang - A41535
> <Wenyou.Yang@microchip.com>; Simon Glass <sjg@chromium.org>; Andreas
> Bie?mann <andreas@biessmann.org>; Songjun Wu - A41519
> <Songjun.Wu@microchip.com>
> Subject: Re: [PATCH v3] i2c: at91_i2c: Fix the wrong include file
> 
> Hello Wenyou Yang,
> 
> Am 17.10.2016 um 03:46 schrieb Wenyou Yang:
> > Since the 'clk_client.h' doesn't exist, it should be 'clk.h'.
> >
> > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> > Reviewed-by: Simon Glass <sjg@chromium.org>
> > Reviewed-by: Andreas Bie?mann <andreas@biessmann.org>
> > ---
> >
> > Changes in v3:
> >   - Collect Reviewed-by tag.
> >
> > Changes in v2:
> >   - Collect Reviewed-by tag.
> >
> >   drivers/i2c/at91_i2c.c | 2 +-
> >   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> Hups, I just want to apply this patch to u-boot-i2c.git ... but, this patch is already
> applied in mainline:

Sorry, 

I forgot to check the mainline, I checked the master branch on git://git.denx.de/u-boot-atmel.git

> 
> commit 76062b9cdbe756eff75b99beaf6e94e8bb059431
> Author: Wenyou Yang <wenyou.yang@atmel.com>
> Date:   Tue Sep 13 10:40:31 2016 +0800
> 
>      i2c: at91_i2c: Fix the wrong include file
> 
> So, I only set your v3 version into state "Not applicable"
> 
> bye,
> Heiko
> >
> > diff --git a/drivers/i2c/at91_i2c.c b/drivers/i2c/at91_i2c.c index
> > 8e9c3ad..d71f75c 100644
> > --- a/drivers/i2c/at91_i2c.c
> > +++ b/drivers/i2c/at91_i2c.c
> > @@ -8,7 +8,7 @@
> >
> >   #include <asm/io.h>
> >   #include <common.h>
> > -#include <clk_client.h>
> > +#include <clk.h>
> >   #include <dm.h>
> >   #include <errno.h>
> >   #include <fdtdec.h>
> >
> 
> --
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany


Best Regards,
Wenyou Yang

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2016-10-17  5:51 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-17  1:46 [U-Boot] [PATCH v3] i2c: at91_i2c: Fix the wrong include file Wenyou Yang
2016-10-17  5:42 ` Heiko Schocher
2016-10-17  5:46 ` Heiko Schocher
2016-10-17  5:51   ` Wenyou.Yang at microchip.com

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox