Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-27 Thread Lee Jones
On Sat, 24 Aug 2013, Arnd Bergmann wrote:

> On Friday 23 August 2013, Linus Walleij wrote:
> > On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones  wrote:
> > 
> > > Signed-off-by: Lee Jones 
> > (...)
> > > /* Requires DMA bindings. */
> > > OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  _plat),
> > > -   OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  
> > > _sdi0_data),
> > > -   OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  
> > > _sdi1_data),
> > > -   OF_DEV_AUXDATA("arm,pl18x", 0x80005000, "sdi2",  
> > > _sdi2_data),
> > > -   OF_DEV_AUXDATA("arm,pl18x", 0x80114000, "sdi4",  
> > > _sdi4_data),
> > 
> > This cannot be done. Ulf H would beat me up if I applied this.
> > 
> > The platform data there in the end is actually used.
> > 
> > Especially look at mop500_sdi0_data, it contains a lot of stuff that is
> > not yet DT:ed even (a task in its own right).
> > 
> > But as long as we keep this pdata we can still delete ATAGs support (as
> > per the second patch set pending) and get a clearer picture.
> 
> Which parts of mop500_sdi0_data are not yet converted? I thought it was all 
> covered
> months ago.

Well it would have been if you hadn't have stopped my CAPs patch. ;)

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-27 Thread Lee Jones
On Sat, 24 Aug 2013, Arnd Bergmann wrote:

 On Friday 23 August 2013, Linus Walleij wrote:
  On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones lee.jo...@linaro.org wrote:
  
   Signed-off-by: Lee Jones lee.jo...@linaro.org
  (...)
   /* Requires DMA bindings. */
   OF_DEV_AUXDATA(arm,pl022, 0x80002000, ssp0,  ssp0_plat),
   -   OF_DEV_AUXDATA(arm,pl18x, 0x80126000, sdi0,  
   mop500_sdi0_data),
   -   OF_DEV_AUXDATA(arm,pl18x, 0x80118000, sdi1,  
   mop500_sdi1_data),
   -   OF_DEV_AUXDATA(arm,pl18x, 0x80005000, sdi2,  
   mop500_sdi2_data),
   -   OF_DEV_AUXDATA(arm,pl18x, 0x80114000, sdi4,  
   mop500_sdi4_data),
  
  This cannot be done. Ulf H would beat me up if I applied this.
  
  The platform data there in the end is actually used.
  
  Especially look at mop500_sdi0_data, it contains a lot of stuff that is
  not yet DT:ed even (a task in its own right).
  
  But as long as we keep this pdata we can still delete ATAGs support (as
  per the second patch set pending) and get a clearer picture.
 
 Which parts of mop500_sdi0_data are not yet converted? I thought it was all 
 covered
 months ago.

Well it would have been if you hadn't have stopped my CAPs patch. ;)

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-24 Thread Arnd Bergmann
On Friday 23 August 2013, Linus Walleij wrote:
> On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones  wrote:
> 
> > Signed-off-by: Lee Jones 
> (...)
> > /* Requires DMA bindings. */
> > OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  _plat),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  _sdi0_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  _sdi1_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80005000, "sdi2",  _sdi2_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80114000, "sdi4",  _sdi4_data),
> 
> This cannot be done. Ulf H would beat me up if I applied this.
> 
> The platform data there in the end is actually used.
> 
> Especially look at mop500_sdi0_data, it contains a lot of stuff that is
> not yet DT:ed even (a task in its own right).
> 
> But as long as we keep this pdata we can still delete ATAGs support (as
> per the second patch set pending) and get a clearer picture.

Which parts of mop500_sdi0_data are not yet converted? I thought it was all 
covered
months ago.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-24 Thread Arnd Bergmann
On Friday 23 August 2013, Linus Walleij wrote:
 On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones lee.jo...@linaro.org wrote:
 
  Signed-off-by: Lee Jones lee.jo...@linaro.org
 (...)
  /* Requires DMA bindings. */
  OF_DEV_AUXDATA(arm,pl022, 0x80002000, ssp0,  ssp0_plat),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80126000, sdi0,  mop500_sdi0_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80118000, sdi1,  mop500_sdi1_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80005000, sdi2,  mop500_sdi2_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80114000, sdi4,  mop500_sdi4_data),
 
 This cannot be done. Ulf H would beat me up if I applied this.
 
 The platform data there in the end is actually used.
 
 Especially look at mop500_sdi0_data, it contains a lot of stuff that is
 not yet DT:ed even (a task in its own right).
 
 But as long as we keep this pdata we can still delete ATAGs support (as
 per the second patch set pending) and get a clearer picture.

Which parts of mop500_sdi0_data are not yet converted? I thought it was all 
covered
months ago.

Arnd
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-23 Thread Lee Jones
On Fri, 23 Aug 2013, Linus Walleij wrote:

> On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones  wrote:
> 
> > Signed-off-by: Lee Jones 
> (...)
> > /* Requires DMA bindings. */
> > OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  _plat),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  _sdi0_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  _sdi1_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80005000, "sdi2",  _sdi2_data),
> > -   OF_DEV_AUXDATA("arm,pl18x", 0x80114000, "sdi4",  _sdi4_data),
> 
> This cannot be done. Ulf H would beat me up if I applied this.
> 
> The platform data there in the end is actually used.
> 
> Especially look at mop500_sdi0_data, it contains a lot of stuff that is
> not yet DT:ed even (a task in its own right).
> 
> But as long as we keep this pdata we can still delete ATAGs support (as
> per the second patch set pending) and get a clearer picture.

Agreed. Scrap the patch.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-23 Thread Linus Walleij
On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones  wrote:

> Signed-off-by: Lee Jones 
(...)
> /* Requires DMA bindings. */
> OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  _plat),
> -   OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  _sdi0_data),
> -   OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  _sdi1_data),
> -   OF_DEV_AUXDATA("arm,pl18x", 0x80005000, "sdi2",  _sdi2_data),
> -   OF_DEV_AUXDATA("arm,pl18x", 0x80114000, "sdi4",  _sdi4_data),

This cannot be done. Ulf H would beat me up if I applied this.

The platform data there in the end is actually used.

Especially look at mop500_sdi0_data, it contains a lot of stuff that is
not yet DT:ed even (a task in its own right).

But as long as we keep this pdata we can still delete ATAGs support (as
per the second patch set pending) and get a clearer picture.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-23 Thread Linus Walleij
On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones lee.jo...@linaro.org wrote:

 Signed-off-by: Lee Jones lee.jo...@linaro.org
(...)
 /* Requires DMA bindings. */
 OF_DEV_AUXDATA(arm,pl022, 0x80002000, ssp0,  ssp0_plat),
 -   OF_DEV_AUXDATA(arm,pl18x, 0x80126000, sdi0,  mop500_sdi0_data),
 -   OF_DEV_AUXDATA(arm,pl18x, 0x80118000, sdi1,  mop500_sdi1_data),
 -   OF_DEV_AUXDATA(arm,pl18x, 0x80005000, sdi2,  mop500_sdi2_data),
 -   OF_DEV_AUXDATA(arm,pl18x, 0x80114000, sdi4,  mop500_sdi4_data),

This cannot be done. Ulf H would beat me up if I applied this.

The platform data there in the end is actually used.

Especially look at mop500_sdi0_data, it contains a lot of stuff that is
not yet DT:ed even (a task in its own right).

But as long as we keep this pdata we can still delete ATAGs support (as
per the second patch set pending) and get a clearer picture.

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: [PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-08-23 Thread Lee Jones
On Fri, 23 Aug 2013, Linus Walleij wrote:

 On Thu, Jun 6, 2013 at 2:17 PM, Lee Jones lee.jo...@linaro.org wrote:
 
  Signed-off-by: Lee Jones lee.jo...@linaro.org
 (...)
  /* Requires DMA bindings. */
  OF_DEV_AUXDATA(arm,pl022, 0x80002000, ssp0,  ssp0_plat),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80126000, sdi0,  mop500_sdi0_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80118000, sdi1,  mop500_sdi1_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80005000, sdi2,  mop500_sdi2_data),
  -   OF_DEV_AUXDATA(arm,pl18x, 0x80114000, sdi4,  mop500_sdi4_data),
 
 This cannot be done. Ulf H would beat me up if I applied this.
 
 The platform data there in the end is actually used.
 
 Especially look at mop500_sdi0_data, it contains a lot of stuff that is
 not yet DT:ed even (a task in its own right).
 
 But as long as we keep this pdata we can still delete ATAGs support (as
 per the second patch set pending) and get a clearer picture.

Agreed. Scrap the patch.

-- 
Lee Jones
Linaro ST-Ericsson Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-06-06 Thread Lee Jones
Signed-off-by: Lee Jones 
---
 arch/arm/mach-ux500/cpu-db8500.c |4 
 1 file changed, 4 deletions(-)

diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c
index 3efb326..aa325d8 100644
--- a/arch/arm/mach-ux500/cpu-db8500.c
+++ b/arch/arm/mach-ux500/cpu-db8500.c
@@ -220,10 +220,6 @@ static struct of_dev_auxdata u8500_auxdata_lookup[] 
__initdata = {
OF_DEV_AUXDATA("arm,cortex-a9-pmu", 0, "arm-pmu", _pmu_platdata),
/* Requires DMA bindings. */
OF_DEV_AUXDATA("arm,pl022", 0x80002000, "ssp0",  _plat),
-   OF_DEV_AUXDATA("arm,pl18x", 0x80126000, "sdi0",  _sdi0_data),
-   OF_DEV_AUXDATA("arm,pl18x", 0x80118000, "sdi1",  _sdi1_data),
-   OF_DEV_AUXDATA("arm,pl18x", 0x80005000, "sdi2",  _sdi2_data),
-   OF_DEV_AUXDATA("arm,pl18x", 0x80114000, "sdi4",  _sdi4_data),
/* Requires clock name bindings. */
OF_DEV_AUXDATA("st,nomadik-i2c", 0x80004000, "nmk-i2c.0", NULL),
OF_DEV_AUXDATA("st,nomadik-i2c", 0x80122000, "nmk-i2c.1", NULL),
-- 
1.7.10.4

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


[PATCH 26/33] ARM: ux500: Remove AUXDATA relating to SDI (MMC) clock-name bindings

2013-06-06 Thread Lee Jones
Signed-off-by: Lee Jones lee.jo...@linaro.org
---
 arch/arm/mach-ux500/cpu-db8500.c |4 
 1 file changed, 4 deletions(-)

diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c
index 3efb326..aa325d8 100644
--- a/arch/arm/mach-ux500/cpu-db8500.c
+++ b/arch/arm/mach-ux500/cpu-db8500.c
@@ -220,10 +220,6 @@ static struct of_dev_auxdata u8500_auxdata_lookup[] 
__initdata = {
OF_DEV_AUXDATA(arm,cortex-a9-pmu, 0, arm-pmu, db8500_pmu_platdata),
/* Requires DMA bindings. */
OF_DEV_AUXDATA(arm,pl022, 0x80002000, ssp0,  ssp0_plat),
-   OF_DEV_AUXDATA(arm,pl18x, 0x80126000, sdi0,  mop500_sdi0_data),
-   OF_DEV_AUXDATA(arm,pl18x, 0x80118000, sdi1,  mop500_sdi1_data),
-   OF_DEV_AUXDATA(arm,pl18x, 0x80005000, sdi2,  mop500_sdi2_data),
-   OF_DEV_AUXDATA(arm,pl18x, 0x80114000, sdi4,  mop500_sdi4_data),
/* Requires clock name bindings. */
OF_DEV_AUXDATA(st,nomadik-i2c, 0x80004000, nmk-i2c.0, NULL),
OF_DEV_AUXDATA(st,nomadik-i2c, 0x80122000, nmk-i2c.1, NULL),
-- 
1.7.10.4

--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/