linux-leds.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] leds: s3c24xx: Remove unused machine header include
@ 2020-08-03  9:19 Krzysztof Kozlowski
  2020-08-05 21:57 ` Pavel Machek
  0 siblings, 1 reply; 5+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-03  9:19 UTC (permalink / raw)
  To: Kukjin Kim, Krzysztof Kozlowski, Jacek Anaszewski, Pavel Machek,
	Dan Murphy, linux-arm-kernel, linux-samsung-soc, linux-leds,
	linux-kernel

The driver includes machine header for GPIO registers but actually does
not use them.

Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
---
 drivers/leds/leds-s3c24xx.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/drivers/leds/leds-s3c24xx.c b/drivers/leds/leds-s3c24xx.c
index 9b5e67664ba3..3c0c7aa63b8c 100644
--- a/drivers/leds/leds-s3c24xx.c
+++ b/drivers/leds/leds-s3c24xx.c
@@ -16,8 +16,6 @@
 #include <linux/module.h>
 #include <linux/platform_data/leds-s3c24xx.h>
 
-#include <mach/regs-gpio.h>
-
 /* our context */
 
 struct s3c24xx_gpio_led {
-- 
2.17.1


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

* Re: [PATCH] leds: s3c24xx: Remove unused machine header include
  2020-08-03  9:19 [PATCH] leds: s3c24xx: Remove unused machine header include Krzysztof Kozlowski
@ 2020-08-05 21:57 ` Pavel Machek
  2020-08-20 16:08   ` Krzysztof Kozlowski
  0 siblings, 1 reply; 5+ messages in thread
From: Pavel Machek @ 2020-08-05 21:57 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Kukjin Kim, Jacek Anaszewski, Dan Murphy, linux-arm-kernel,
	linux-samsung-soc, linux-leds, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 367 bytes --]

On Mon 2020-08-03 11:19:36, Krzysztof Kozlowski wrote:
> The driver includes machine header for GPIO registers but actually does
> not use them.
> 
> Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>

Thanks, applied.
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

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

* Re: [PATCH] leds: s3c24xx: Remove unused machine header include
  2020-08-05 21:57 ` Pavel Machek
@ 2020-08-20 16:08   ` Krzysztof Kozlowski
  2020-08-20 16:10     ` Krzysztof Kozlowski
  2020-08-20 21:51     ` Pavel Machek
  0 siblings, 2 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-20 16:08 UTC (permalink / raw)
  To: Pavel Machek
  Cc: Kukjin Kim, Jacek Anaszewski, Dan Murphy, linux-arm-kernel,
	linux-samsung-soc, linux-leds, linux-kernel

On Wed, Aug 05, 2020 at 11:57:30PM +0200, Pavel Machek wrote:
> On Mon 2020-08-03 11:19:36, Krzysztof Kozlowski wrote:
> > The driver includes machine header for GPIO registers but actually does
> > not use them.
> > 
> > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> 
> Thanks, applied.

Hi Pavel,

I am confused because I had impression this will go into v5.9-rc1 but
it's not there.  My further S3C cleanup depends on it [1] and now
applying of the patches stuck in the middle.

I could take the patch through samsung-soc along with the rest of the
series or pull from stable tag (but tag with only that patch...).

Best regards,
Krzysztof


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

* Re: [PATCH] leds: s3c24xx: Remove unused machine header include
  2020-08-20 16:08   ` Krzysztof Kozlowski
@ 2020-08-20 16:10     ` Krzysztof Kozlowski
  2020-08-20 21:51     ` Pavel Machek
  1 sibling, 0 replies; 5+ messages in thread
From: Krzysztof Kozlowski @ 2020-08-20 16:10 UTC (permalink / raw)
  To: Pavel Machek
  Cc: Kukjin Kim, Jacek Anaszewski, Dan Murphy, linux-arm-kernel,
	linux-samsung-soc, linux-leds, linux-kernel

On Thu, Aug 20, 2020 at 06:08:16PM +0200, Krzysztof Kozlowski wrote:
> On Wed, Aug 05, 2020 at 11:57:30PM +0200, Pavel Machek wrote:
> > On Mon 2020-08-03 11:19:36, Krzysztof Kozlowski wrote:
> > > The driver includes machine header for GPIO registers but actually does
> > > not use them.
> > > 
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > 
> > Thanks, applied.
> 
> Hi Pavel,
> 
> I am confused because I had impression this will go into v5.9-rc1 but
> it's not there.  My further S3C cleanup depends on it [1] and now
> applying of the patches stuck in the middle.
> 
> I could take the patch through samsung-soc along with the rest of the
> series or pull from stable tag (but tag with only that patch...).

... and the missing link:
[1] https://marc.info/?l=dri-devel&m=159793916332137&w=2

Best regards,
Krzysztof


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

* Re: [PATCH] leds: s3c24xx: Remove unused machine header include
  2020-08-20 16:08   ` Krzysztof Kozlowski
  2020-08-20 16:10     ` Krzysztof Kozlowski
@ 2020-08-20 21:51     ` Pavel Machek
  1 sibling, 0 replies; 5+ messages in thread
From: Pavel Machek @ 2020-08-20 21:51 UTC (permalink / raw)
  To: Krzysztof Kozlowski
  Cc: Kukjin Kim, Jacek Anaszewski, Dan Murphy, linux-arm-kernel,
	linux-samsung-soc, linux-leds, linux-kernel

[-- Attachment #1: Type: text/plain, Size: 823 bytes --]

On Thu 2020-08-20 18:08:16, Krzysztof Kozlowski wrote:
> On Wed, Aug 05, 2020 at 11:57:30PM +0200, Pavel Machek wrote:
> > On Mon 2020-08-03 11:19:36, Krzysztof Kozlowski wrote:
> > > The driver includes machine header for GPIO registers but actually does
> > > not use them.
> > > 
> > > Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org>
> > 
> > Thanks, applied.
> 
> Hi Pavel,
> 
> I am confused because I had impression this will go into v5.9-rc1 but
> it's not there.  My further S3C cleanup depends on it [1] and now
> applying of the patches stuck in the middle.

It was too late for -rc1 at that moment. It is on my for-next branch
now.

									Pavel
-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 181 bytes --]

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

end of thread, other threads:[~2020-08-20 21:52 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-03  9:19 [PATCH] leds: s3c24xx: Remove unused machine header include Krzysztof Kozlowski
2020-08-05 21:57 ` Pavel Machek
2020-08-20 16:08   ` Krzysztof Kozlowski
2020-08-20 16:10     ` Krzysztof Kozlowski
2020-08-20 21:51     ` Pavel Machek

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).