Re: [PATCH/RFC v10 05/19] mfd: max77693: Modify flash cell name identifiers

2015-01-20 Thread Jacek Anaszewski

On 01/20/2015 12:13 PM, Lee Jones wrote:

On Fri, 09 Jan 2015, Jacek Anaszewski wrote:


Change flash cell identifiers from max77693-flash to max77693-led
to avoid confusion with NOR/NAND Flash.


This is okay by me, but aren't these ABI yet?


No, the led driver using it hasn't been merged yet.


Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
Acked-by: Kyungmin Park kyungmin.p...@samsung.com
Cc: Chanwoo Choi cw00.c...@samsung.com
Cc: Lee Jones lee.jo...@linaro.org
---
  drivers/mfd/max77693.c |4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
index a159593..cb14afa 100644
--- a/drivers/mfd/max77693.c
+++ b/drivers/mfd/max77693.c
@@ -53,8 +53,8 @@ static const struct mfd_cell max77693_devs[] = {
.of_compatible = maxim,max77693-haptic,
},
{
-   .name = max77693-flash,
-   .of_compatible = maxim,max77693-flash,
+   .name = max77693-led,
+   .of_compatible = maxim,max77693-led,
},
  };






--
Best Regards,
Jacek Anaszewski
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH/RFC v10 05/19] mfd: max77693: Modify flash cell name identifiers

2015-01-20 Thread Lee Jones
On Fri, 09 Jan 2015, Jacek Anaszewski wrote:

 Change flash cell identifiers from max77693-flash to max77693-led
 to avoid confusion with NOR/NAND Flash.

This is okay by me, but aren't these ABI yet?

 Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
 Acked-by: Kyungmin Park kyungmin.p...@samsung.com
 Cc: Chanwoo Choi cw00.c...@samsung.com
 Cc: Lee Jones lee.jo...@linaro.org
 ---
  drivers/mfd/max77693.c |4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
 index a159593..cb14afa 100644
 --- a/drivers/mfd/max77693.c
 +++ b/drivers/mfd/max77693.c
 @@ -53,8 +53,8 @@ static const struct mfd_cell max77693_devs[] = {
   .of_compatible = maxim,max77693-haptic,
   },
   {
 - .name = max77693-flash,
 - .of_compatible = maxim,max77693-flash,
 + .name = max77693-led,
 + .of_compatible = maxim,max77693-led,
   },
  };
  

-- 
Lee Jones
Linaro STMicroelectronics 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-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH/RFC v10 05/19] mfd: max77693: Modify flash cell name identifiers

2015-01-20 Thread Lee Jones
On Tue, 20 Jan 2015, Jacek Anaszewski wrote:

 On 01/20/2015 12:13 PM, Lee Jones wrote:
 On Fri, 09 Jan 2015, Jacek Anaszewski wrote:
 
 Change flash cell identifiers from max77693-flash to max77693-led
 to avoid confusion with NOR/NAND Flash.
 
 This is okay by me, but aren't these ABI yet?
 
 No, the led driver using it hasn't been merged yet.

Very well.

For my on reference:

Acked-by: Lee Jones lee.jo...@linaro.org

 Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
 Acked-by: Kyungmin Park kyungmin.p...@samsung.com
 Cc: Chanwoo Choi cw00.c...@samsung.com
 Cc: Lee Jones lee.jo...@linaro.org
 ---
   drivers/mfd/max77693.c |4 ++--
   1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
 index a159593..cb14afa 100644
 --- a/drivers/mfd/max77693.c
 +++ b/drivers/mfd/max77693.c
 @@ -53,8 +53,8 @@ static const struct mfd_cell max77693_devs[] = {
 .of_compatible = maxim,max77693-haptic,
 },
 {
 -   .name = max77693-flash,
 -   .of_compatible = maxim,max77693-flash,
 +   .name = max77693-led,
 +   .of_compatible = maxim,max77693-led,
 },
   };
 
 
 
 

-- 
Lee Jones
Linaro STMicroelectronics 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-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH/RFC v10 05/19] mfd: max77693: Modify flash cell name identifiers

2015-01-09 Thread Jacek Anaszewski
Change flash cell identifiers from max77693-flash to max77693-led
to avoid confusion with NOR/NAND Flash.

Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
Acked-by: Kyungmin Park kyungmin.p...@samsung.com
Cc: Chanwoo Choi cw00.c...@samsung.com
Cc: Lee Jones lee.jo...@linaro.org
---
 drivers/mfd/max77693.c |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/mfd/max77693.c b/drivers/mfd/max77693.c
index a159593..cb14afa 100644
--- a/drivers/mfd/max77693.c
+++ b/drivers/mfd/max77693.c
@@ -53,8 +53,8 @@ static const struct mfd_cell max77693_devs[] = {
.of_compatible = maxim,max77693-haptic,
},
{
-   .name = max77693-flash,
-   .of_compatible = maxim,max77693-flash,
+   .name = max77693-led,
+   .of_compatible = maxim,max77693-led,
},
 };
 
-- 
1.7.9.5

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


Re: [PATCH/RFC v10 05/19] mfd: max77693: Modify flash cell name identifiers

2015-01-09 Thread Pavel Machek
On Fri 2015-01-09 16:22:55, Jacek Anaszewski wrote:
 Change flash cell identifiers from max77693-flash to max77693-led
 to avoid confusion with NOR/NAND Flash.
 
 Signed-off-by: Jacek Anaszewski j.anaszew...@samsung.com
 Acked-by: Kyungmin Park kyungmin.p...@samsung.com
 Cc: Chanwoo Choi cw00.c...@samsung.com
 Cc: Lee Jones lee.jo...@linaro.org

Acked-by: Pavel Machek pa...@ucw.cz

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html
--
To unsubscribe from this list: send the line unsubscribe linux-media in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html