* [PATCH 1/3] cosmetic: README: Fix one CONFIG name
@ 2020-04-10 14:34 Patrick Delaunay
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
` (3 more replies)
0 siblings, 4 replies; 9+ messages in thread
From: Patrick Delaunay @ 2020-04-10 14:34 UTC (permalink / raw)
To: u-boot
Only replace CONFIF_ by CONFIG_
Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
---
README | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/README b/README
index 8cfa92fac9..6cb7656fa0 100644
--- a/README
+++ b/README
@@ -1640,7 +1640,7 @@ The following options need to be configured:
- CONFIG_SYS_RCAR_I2C2_SPEED for for the speed channel 2
- CONFIG_SYS_RCAR_I2C3_BASE for setting the register channel 3
- CONFIG_SYS_RCAR_I2C3_SPEED for for the speed channel 3
- - CONFIF_SYS_RCAR_I2C_NUM_CONTROLLERS for number of i2c buses
+ - CONFIG_SYS_RCAR_I2C_NUM_CONTROLLERS for number of i2c buses
- drivers/i2c/sh_i2c.c:
- activate this driver with CONFIG_SYS_I2C_SH
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C*
2020-04-10 14:34 [PATCH 1/3] cosmetic: README: Fix one CONFIG name Patrick Delaunay
@ 2020-04-10 14:34 ` Patrick Delaunay
2020-04-10 20:47 ` Simon Glass
2020-04-26 11:27 ` Tom Rini
2020-04-10 14:34 ` [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE Patrick Delaunay
` (2 subsequent siblings)
3 siblings, 2 replies; 9+ messages in thread
From: Patrick Delaunay @ 2020-04-10 14:34 UTC (permalink / raw)
To: u-boot
Remove the references in README on CONFIG_SYS_RCAR_I2C_* not use
in U-Boot drivers/i2c/rcar_i2c.c, since commit a4d9aafadb31 ("i2c:
rcar_i2c: Remove the driver") and commit a06a0ac36d59 ("i2c: rcar_i2c:
Add DM and DT capable I2C driver")
Checked by the command: grep -r SYS_RCAR_I2C *
And these CONFIG are only defined in
arch/arm/mach-rmobile/include/mach/rcar-base.h
Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
---
README | 10 ----------
1 file changed, 10 deletions(-)
diff --git a/README b/README
index 6cb7656fa0..3cc99f0805 100644
--- a/README
+++ b/README
@@ -1632,16 +1632,6 @@ The following options need to be configured:
- activate this driver with CONFIG_SYS_I2C_RCAR
- This driver adds 4 i2c buses
- - CONFIG_SYS_RCAR_I2C0_BASE for setting the register channel 0
- - CONFIG_SYS_RCAR_I2C0_SPEED for for the speed channel 0
- - CONFIG_SYS_RCAR_I2C1_BASE for setting the register channel 1
- - CONFIG_SYS_RCAR_I2C1_SPEED for for the speed channel 1
- - CONFIG_SYS_RCAR_I2C2_BASE for setting the register channel 2
- - CONFIG_SYS_RCAR_I2C2_SPEED for for the speed channel 2
- - CONFIG_SYS_RCAR_I2C3_BASE for setting the register channel 3
- - CONFIG_SYS_RCAR_I2C3_SPEED for for the speed channel 3
- - CONFIG_SYS_RCAR_I2C_NUM_CONTROLLERS for number of i2c buses
-
- drivers/i2c/sh_i2c.c:
- activate this driver with CONFIG_SYS_I2C_SH
- This driver adds from 2 to 5 i2c buses
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread* [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C*
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
@ 2020-04-10 20:47 ` Simon Glass
2020-04-26 11:27 ` Tom Rini
1 sibling, 0 replies; 9+ messages in thread
From: Simon Glass @ 2020-04-10 20:47 UTC (permalink / raw)
To: u-boot
On Fri, 10 Apr 2020 at 08:34, Patrick Delaunay <patrick.delaunay@st.com> wrote:
>
> Remove the references in README on CONFIG_SYS_RCAR_I2C_* not use
> in U-Boot drivers/i2c/rcar_i2c.c, since commit a4d9aafadb31 ("i2c:
> rcar_i2c: Remove the driver") and commit a06a0ac36d59 ("i2c: rcar_i2c:
> Add DM and DT capable I2C driver")
>
> Checked by the command: grep -r SYS_RCAR_I2C *
> And these CONFIG are only defined in
> arch/arm/mach-rmobile/include/mach/rcar-base.h
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> ---
>
> README | 10 ----------
> 1 file changed, 10 deletions(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
Nice clean-up.
^ permalink raw reply [flat|nested] 9+ messages in thread* [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C*
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
2020-04-10 20:47 ` Simon Glass
@ 2020-04-26 11:27 ` Tom Rini
1 sibling, 0 replies; 9+ messages in thread
From: Tom Rini @ 2020-04-26 11:27 UTC (permalink / raw)
To: u-boot
On Fri, Apr 10, 2020 at 04:34:41PM +0200, Patrick Delaunay wrote:
> Remove the references in README on CONFIG_SYS_RCAR_I2C_* not use
> in U-Boot drivers/i2c/rcar_i2c.c, since commit a4d9aafadb31 ("i2c:
> rcar_i2c: Remove the driver") and commit a06a0ac36d59 ("i2c: rcar_i2c:
> Add DM and DT capable I2C driver")
>
> Checked by the command: grep -r SYS_RCAR_I2C *
> And these CONFIG are only defined in
> arch/arm/mach-rmobile/include/mach/rcar-base.h
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200426/91c50efc/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE
2020-04-10 14:34 [PATCH 1/3] cosmetic: README: Fix one CONFIG name Patrick Delaunay
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
@ 2020-04-10 14:34 ` Patrick Delaunay
2020-04-10 16:43 ` Marek Vasut
2020-04-10 20:47 ` [PATCH 1/3] cosmetic: README: Fix one CONFIG name Simon Glass
2020-04-26 11:26 ` Tom Rini
3 siblings, 1 reply; 9+ messages in thread
From: Patrick Delaunay @ 2020-04-10 14:34 UTC (permalink / raw)
To: u-boot
As this "CONFIG" are never used, CONFIG_SYS_RCAR_I2C*_BASE can
be removed without effect and the file config_whitelist.txt is
also clean-up.
Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
---
arch/arm/mach-rmobile/include/mach/rcar-base.h | 6 ------
scripts/config_whitelist.txt | 4 ----
2 files changed, 10 deletions(-)
diff --git a/arch/arm/mach-rmobile/include/mach/rcar-base.h b/arch/arm/mach-rmobile/include/mach/rcar-base.h
index a20740679f..811a1183ba 100644
--- a/arch/arm/mach-rmobile/include/mach/rcar-base.h
+++ b/arch/arm/mach-rmobile/include/mach/rcar-base.h
@@ -78,12 +78,6 @@
#define CONFIG_SYS_I2C_SH_BASE0 0xE6500000
#define CONFIG_SYS_I2C_SH_BASE1 0xE6510000
-/* RCAR-I2C */
-#define CONFIG_SYS_RCAR_I2C0_BASE 0xE6508000
-#define CONFIG_SYS_RCAR_I2C1_BASE 0xE6518000
-#define CONFIG_SYS_RCAR_I2C2_BASE 0xE6530000
-#define CONFIG_SYS_RCAR_I2C3_BASE 0xE6540000
-
/* SDHI */
#define CONFIG_SYS_SH_SDHI0_BASE 0xEE100000
diff --git a/scripts/config_whitelist.txt b/scripts/config_whitelist.txt
index 0f747ac0a3..0f9d8cb0d7 100644
--- a/scripts/config_whitelist.txt
+++ b/scripts/config_whitelist.txt
@@ -3668,10 +3668,6 @@ CONFIG_SYS_QRIO_BR_PRELIM
CONFIG_SYS_QRIO_OR_PRELIM
CONFIG_SYS_R7780MP_OLD_FLASH
CONFIG_SYS_RAMBOOT
-CONFIG_SYS_RCAR_I2C0_BASE
-CONFIG_SYS_RCAR_I2C1_BASE
-CONFIG_SYS_RCAR_I2C2_BASE
-CONFIG_SYS_RCAR_I2C3_BASE
CONFIG_SYS_RCWH_PCIHOST
CONFIG_SYS_READ_SPD
CONFIG_SYS_RESET_ADDR
--
2.17.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE
2020-04-10 14:34 ` [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE Patrick Delaunay
@ 2020-04-10 16:43 ` Marek Vasut
2020-04-20 8:42 ` Patrick DELAUNAY
0 siblings, 1 reply; 9+ messages in thread
From: Marek Vasut @ 2020-04-10 16:43 UTC (permalink / raw)
To: u-boot
On 4/10/20 4:34 PM, Patrick Delaunay wrote:
> As this "CONFIG" are never used, CONFIG_SYS_RCAR_I2C*_BASE can
> be removed without effect and the file config_whitelist.txt is
> also clean-up.
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> ---
>
> arch/arm/mach-rmobile/include/mach/rcar-base.h | 6 ------
> scripts/config_whitelist.txt | 4 ----
> 2 files changed, 10 deletions(-)
Update the README too, otherwise look good, thanks.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE
2020-04-10 16:43 ` Marek Vasut
@ 2020-04-20 8:42 ` Patrick DELAUNAY
0 siblings, 0 replies; 9+ messages in thread
From: Patrick DELAUNAY @ 2020-04-20 8:42 UTC (permalink / raw)
To: u-boot
Dear Marek,
> From: Marek Vasut <marex@denx.de>
> Sent: vendredi 10 avril 2020 18:44
>
> On 4/10/20 4:34 PM, Patrick Delaunay wrote:
> > As this "CONFIG" are never used, CONFIG_SYS_RCAR_I2C*_BASE can be
> > removed without effect and the file config_whitelist.txt is also
> > clean-up.
> >
> > Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> > ---
> >
> > arch/arm/mach-rmobile/include/mach/rcar-base.h | 6 ------
> > scripts/config_whitelist.txt | 4 ----
> > 2 files changed, 10 deletions(-)
>
> Update the README too, otherwise look good, thanks.
Done in a separate patch as I clean all the CONFIG no more used in this driver
[PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C*
I didn't add you in CC for the serie, sorry.
http://patchwork.ozlabs.org/project/uboot/list/?series=169619
Regards
Patrick
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/3] cosmetic: README: Fix one CONFIG name
2020-04-10 14:34 [PATCH 1/3] cosmetic: README: Fix one CONFIG name Patrick Delaunay
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
2020-04-10 14:34 ` [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE Patrick Delaunay
@ 2020-04-10 20:47 ` Simon Glass
2020-04-26 11:26 ` Tom Rini
3 siblings, 0 replies; 9+ messages in thread
From: Simon Glass @ 2020-04-10 20:47 UTC (permalink / raw)
To: u-boot
On Fri, 10 Apr 2020 at 08:34, Patrick Delaunay <patrick.delaunay@st.com> wrote:
>
> Only replace CONFIF_ by CONFIG_
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> ---
>
> README | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Simon Glass <sjg@chromium.org>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/3] cosmetic: README: Fix one CONFIG name
2020-04-10 14:34 [PATCH 1/3] cosmetic: README: Fix one CONFIG name Patrick Delaunay
` (2 preceding siblings ...)
2020-04-10 20:47 ` [PATCH 1/3] cosmetic: README: Fix one CONFIG name Simon Glass
@ 2020-04-26 11:26 ` Tom Rini
3 siblings, 0 replies; 9+ messages in thread
From: Tom Rini @ 2020-04-26 11:26 UTC (permalink / raw)
To: u-boot
On Fri, Apr 10, 2020 at 04:34:40PM +0200, Patrick Delaunay wrote:
> Only replace CONFIF_ by CONFIG_
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay@st.com>
> Reviewed-by: Simon Glass <sjg@chromium.org>
Applied to u-boot/master, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200426/b009e4f9/attachment.sig>
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2020-04-26 11:27 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-04-10 14:34 [PATCH 1/3] cosmetic: README: Fix one CONFIG name Patrick Delaunay
2020-04-10 14:34 ` [PATCH 2/3] README: remove references on no more used config CONFIG_SYS_RCAR_I2C* Patrick Delaunay
2020-04-10 20:47 ` Simon Glass
2020-04-26 11:27 ` Tom Rini
2020-04-10 14:34 ` [PATCH 3/3] rmobile: rcar: removed used CONFIG_SYS_RCAR_I2C*_BASE Patrick Delaunay
2020-04-10 16:43 ` Marek Vasut
2020-04-20 8:42 ` Patrick DELAUNAY
2020-04-10 20:47 ` [PATCH 1/3] cosmetic: README: Fix one CONFIG name Simon Glass
2020-04-26 11:26 ` Tom Rini
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.