From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Magnus Damm <magnus.damm@gmail.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
SH-Linux <linux-sh@vger.kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Simon Horman <horms+renesas@verge.net.au>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 08/08] ARM: shmobile: Remove CMT renesas,channels-mask from DTS
Date: Thu, 25 Jun 2015 21:56:00 +0000 [thread overview]
Message-ID: <1618432.qSqOq2tHsD@avalon> (raw)
In-Reply-To: <CANqRtoRcv4VpF3vAgYKTOL1FyYURHx13-5-L9fH1HrJ-QvPymQ@mail.gmail.com>
Hi Magnus,
On Tuesday 23 June 2015 11:46:45 Magnus Damm wrote:
> On Tue, Jun 23, 2015 at 7:40 AM, Laurent Pinchart wrote:
> > On Sunday 21 June 2015 17:10:33 Magnus Damm wrote:
> >> From: Magnus Damm <damm+renesas@opensource.se>
> >>
> >> Update the DTS to remove the now deprecated "renesas,channels-mask"
> >> property.
> >>
> >> The channel information is now kept in the device driver and can easily
> >> be determined based on the compat string.
> >
> > Unless I've missed something (which is entirely possibly given the late
> > hour), the channel mask is only stored in the driver for the Gen2 CMT
> > instances.
>
> The case for sh73a0 and r8a7740 is also covered by some of the earlier
> patches in this series.
My bad, I've missed that. We should be good then.
> So unless I'm mistaken the series should cover all current in-tree CMT DT
> users that we have. I'm not aware of any out-of-tree users.
>
> You are right that there are more unused compat strings documented, so
> those will break. I would be tempted in phasing those out somehow.
> Also, I did not spend much time cleaning up the non R-Car Gen2 cases,
> so in general that needs some love.
--
Regards,
Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: Magnus Damm <magnus.damm@gmail.com>
Cc: linux-kernel <linux-kernel@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
Laurent Pinchart <laurent.pinchart+renesas@ideasonboard.com>,
Geert Uytterhoeven <geert+renesas@glider.be>,
SH-Linux <linux-sh@vger.kernel.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Simon Horman <horms+renesas@verge.net.au>,
Thomas Gleixner <tglx@linutronix.de>
Subject: Re: [PATCH 08/08] ARM: shmobile: Remove CMT renesas,channels-mask from DTS
Date: Fri, 26 Jun 2015 00:56 +0300 [thread overview]
Message-ID: <1618432.qSqOq2tHsD@avalon> (raw)
In-Reply-To: <CANqRtoRcv4VpF3vAgYKTOL1FyYURHx13-5-L9fH1HrJ-QvPymQ@mail.gmail.com>
Hi Magnus,
On Tuesday 23 June 2015 11:46:45 Magnus Damm wrote:
> On Tue, Jun 23, 2015 at 7:40 AM, Laurent Pinchart wrote:
> > On Sunday 21 June 2015 17:10:33 Magnus Damm wrote:
> >> From: Magnus Damm <damm+renesas@opensource.se>
> >>
> >> Update the DTS to remove the now deprecated "renesas,channels-mask"
> >> property.
> >>
> >> The channel information is now kept in the device driver and can easily
> >> be determined based on the compat string.
> >
> > Unless I've missed something (which is entirely possibly given the late
> > hour), the channel mask is only stored in the driver for the Gen2 CMT
> > instances.
>
> The case for sh73a0 and r8a7740 is also covered by some of the earlier
> patches in this series.
My bad, I've missed that. We should be good then.
> So unless I'm mistaken the series should cover all current in-tree CMT DT
> users that we have. I'm not aware of any out-of-tree users.
>
> You are right that there are more unused compat strings documented, so
> those will break. I would be tempted in phasing those out somehow.
> Also, I did not spend much time cleaning up the non R-Car Gen2 cases,
> so in general that needs some love.
--
Regards,
Laurent Pinchart
next prev parent reply other threads:[~2015-06-25 21:56 UTC|newest]
Thread overview: 53+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-21 8:09 [PATCH 00/08] clocksource: sh_cmt: DT binding rework Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` [PATCH 01/08] devicetree: bindings: Remove sh7372 CMT binding Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-22 9:38 ` Geert Uytterhoeven
2015-06-22 9:38 ` Geert Uytterhoeven
2015-06-22 9:38 ` Geert Uytterhoeven
2015-06-22 9:41 ` Magnus Damm
2015-06-22 9:41 ` Magnus Damm
2015-06-22 9:41 ` Magnus Damm
2015-06-21 8:09 ` [PATCH 02/08] clocksource: sh_cmt: Use 0x3f mask for SH_CMT_48BIT case Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-22 9:43 ` Geert Uytterhoeven
2015-06-22 9:43 ` Geert Uytterhoeven
2015-06-22 9:43 ` Geert Uytterhoeven
2015-06-22 9:45 ` Magnus Damm
2015-06-22 9:45 ` Magnus Damm
2015-06-22 9:45 ` Magnus Damm
2015-06-21 8:09 ` [PATCH 03/08] devicetree: bindings: R-Car Gen2 CMT0 and CMT1 bindings Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` [PATCH 04/08] clocksource: sh_cmt: Support separate R-Car Gen2 CMT0/1 Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:09 ` Magnus Damm
2015-06-21 8:10 ` [PATCH 05/08] devicetree: bindings: r8a73a4 and R-Car Gen2 CMT bindings Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` [PATCH 06/08] ARM: shmobile: Update CMT compat string users in DTS Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` [PATCH 07/08] devicetree: bindings: Deprecate property, update example Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` [PATCH 08/08] ARM: shmobile: Remove CMT renesas,channels-mask from DTS Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-21 8:10 ` Magnus Damm
2015-06-22 22:40 ` Laurent Pinchart
2015-06-22 22:40 ` Laurent Pinchart
2015-06-22 22:40 ` Laurent Pinchart
2015-06-23 2:46 ` Magnus Damm
2015-06-23 2:46 ` Magnus Damm
2015-06-23 2:46 ` Magnus Damm
2015-06-25 21:56 ` Laurent Pinchart [this message]
2015-06-25 21:56 ` Laurent Pinchart
2015-06-22 9:49 ` [PATCH 00/08] clocksource: sh_cmt: DT binding rework Geert Uytterhoeven
2015-06-22 9:49 ` Geert Uytterhoeven
2015-06-22 9:49 ` Geert Uytterhoeven
2015-06-22 22:45 ` Laurent Pinchart
2015-06-22 22:45 ` Laurent Pinchart
2015-06-22 22:45 ` Laurent Pinchart
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1618432.qSqOq2tHsD@avalon \
--to=laurent.pinchart@ideasonboard.com \
--cc=daniel.lezcano@linaro.org \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=horms+renesas@verge.net.au \
--cc=laurent.pinchart+renesas@ideasonboard.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.com \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.