| Message ID | 1413363577-25969-1-git-send-email-ykaneko0929@gmail.com |
|---|---|
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: > From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > > Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> Acked-by: Simon Horman <horms+renesas@verge.net.au> Tejun, could you consider picking up this change? > --- > > This patch is against for-next branch of Tejun's libata repo. > > v2 [Yoshihiro Kaneko] > * Update the binding documentation > > Documentation/devicetree/bindings/ata/sata_rcar.txt | 3 ++- > drivers/ata/sata_rcar.c | 5 +++++ > 2 files changed, 7 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt > index 1e61113..74b1130 100644 > --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt > +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt > @@ -4,7 +4,8 @@ Required properties: > - compatible : should contain one of the following: > - "renesas,sata-r8a7779" for R-Car H1 > - "renesas,sata-r8a7790" for R-Car H2 > - - "renesas,sata-r8a7791" for R-Car M2 > + - "renesas,sata-r8a7791" for R-Car M2-W > + - "renesas,sata-r8a7793" for R-Car M2-N > - reg : address and length of the SATA registers; > - interrupts : must consist of one interrupt specifier. > > diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c > index 61eb6d7..6f8ed8e 100644 > --- a/drivers/ata/sata_rcar.c > +++ b/drivers/ata/sata_rcar.c > @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { > .compatible = "renesas,sata-r8a7791", > .data = (void *)RCAR_GEN2_SATA > }, > + { > + .compatible = "renesas,sata-r8a7793", > + .data = (void *)RCAR_GEN2_SATA > + }, > { }, > }; > MODULE_DEVICE_TABLE(of, sata_rcar_match); > @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { > { "sata-r8a7779", RCAR_GEN1_SATA }, > { "sata-r8a7790", RCAR_GEN2_SATA }, > { "sata-r8a7791", RCAR_GEN2_SATA }, > + { "sata-r8a7793", RCAR_GEN2_SATA }, > { }, > }; > MODULE_DEVICE_TABLE(platform, sata_rcar_id_table); > -- > 1.9.1 > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On 10/16/2014 2:20 PM, Simon Horman wrote: > On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: >> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >> >> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> > > Acked-by: Simon Horman <horms+renesas@verge.net.au> > > Tejun, could you consider picking up this change? Uhmm, R8A7793 SoC hadn't been supported in upstream Linux kernel yet. So, is it still OK to take this patch now ? >> --- >> >> This patch is against for-next branch of Tejun's libata repo. >> >> v2 [Yoshihiro Kaneko] >> * Update the binding documentation >> >> Documentation/devicetree/bindings/ata/sata_rcar.txt | 3 ++- >> drivers/ata/sata_rcar.c | 5 +++++ >> 2 files changed, 7 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt >> index 1e61113..74b1130 100644 >> --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt >> +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt >> @@ -4,7 +4,8 @@ Required properties: >> - compatible : should contain one of the following: >> - "renesas,sata-r8a7779" for R-Car H1 >> - "renesas,sata-r8a7790" for R-Car H2 >> - - "renesas,sata-r8a7791" for R-Car M2 >> + - "renesas,sata-r8a7791" for R-Car M2-W >> + - "renesas,sata-r8a7793" for R-Car M2-N >> - reg : address and length of the SATA registers; >> - interrupts : must consist of one interrupt specifier. >> >> diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c >> index 61eb6d7..6f8ed8e 100644 >> --- a/drivers/ata/sata_rcar.c >> +++ b/drivers/ata/sata_rcar.c >> @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { >> .compatible = "renesas,sata-r8a7791", >> .data = (void *)RCAR_GEN2_SATA >> }, >> + { >> + .compatible = "renesas,sata-r8a7793", >> + .data = (void *)RCAR_GEN2_SATA >> + }, >> { }, >> }; >> MODULE_DEVICE_TABLE(of, sata_rcar_match); >> @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { >> { "sata-r8a7779", RCAR_GEN1_SATA }, >> { "sata-r8a7790", RCAR_GEN2_SATA }, >> { "sata-r8a7791", RCAR_GEN2_SATA }, >> + { "sata-r8a7793", RCAR_GEN2_SATA }, >> { }, >> }; >> MODULE_DEVICE_TABLE(platform, sata_rcar_id_table); >> -- >> 1.9.1 >> > -- > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html >
Hi Khiem-san, On Thu, Oct 16, 2014 at 02:43:05PM +0900, Khiem Nguyen wrote: > Hi, > > On 10/16/2014 2:20 PM, Simon Horman wrote: > > On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: > >> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > >> > >> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > >> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> > > > > Acked-by: Simon Horman <horms+renesas@verge.net.au> > > > > Tejun, could you consider picking up this change? > > Uhmm, R8A7793 SoC hadn't been supported in upstream Linux kernel yet. > So, is it still OK to take this patch now ? Yes, I think so. Mainline support for the r8a7793 is in the pipeline and by adding (trivial) driver support before hand the drivers will be able to be used when the SoC code lands. This is consistent with how other are being handled for the r8a7793. > >> --- > >> > >> This patch is against for-next branch of Tejun's libata repo. > >> > >> v2 [Yoshihiro Kaneko] > >> * Update the binding documentation > >> > >> Documentation/devicetree/bindings/ata/sata_rcar.txt | 3 ++- > >> drivers/ata/sata_rcar.c | 5 +++++ > >> 2 files changed, 7 insertions(+), 1 deletion(-) > >> > >> diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt > >> index 1e61113..74b1130 100644 > >> --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt > >> +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt > >> @@ -4,7 +4,8 @@ Required properties: > >> - compatible : should contain one of the following: > >> - "renesas,sata-r8a7779" for R-Car H1 > >> - "renesas,sata-r8a7790" for R-Car H2 > >> - - "renesas,sata-r8a7791" for R-Car M2 > >> + - "renesas,sata-r8a7791" for R-Car M2-W > >> + - "renesas,sata-r8a7793" for R-Car M2-N > >> - reg : address and length of the SATA registers; > >> - interrupts : must consist of one interrupt specifier. > >> > >> diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c > >> index 61eb6d7..6f8ed8e 100644 > >> --- a/drivers/ata/sata_rcar.c > >> +++ b/drivers/ata/sata_rcar.c > >> @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { > >> .compatible = "renesas,sata-r8a7791", > >> .data = (void *)RCAR_GEN2_SATA > >> }, > >> + { > >> + .compatible = "renesas,sata-r8a7793", > >> + .data = (void *)RCAR_GEN2_SATA > >> + }, > >> { }, > >> }; > >> MODULE_DEVICE_TABLE(of, sata_rcar_match); > >> @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { > >> { "sata-r8a7779", RCAR_GEN1_SATA }, > >> { "sata-r8a7790", RCAR_GEN2_SATA }, > >> { "sata-r8a7791", RCAR_GEN2_SATA }, > >> + { "sata-r8a7793", RCAR_GEN2_SATA }, > >> { }, > >> }; > >> MODULE_DEVICE_TABLE(platform, sata_rcar_id_table); > >> -- > >> 1.9.1 > >> > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-sh" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > > -- > Best regards, > KHIEM Nguyen > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Simon-san, Thanks for your explanation. On 10/16/2014 2:55 PM, Simon Horman wrote: > Hi Khiem-san, > > On Thu, Oct 16, 2014 at 02:43:05PM +0900, Khiem Nguyen wrote: >> Hi, >> >> On 10/16/2014 2:20 PM, Simon Horman wrote: >>> On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: >>>> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >>>> >>>> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >>>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> >>> >>> Acked-by: Simon Horman <horms+renesas@verge.net.au> >>> >>> Tejun, could you consider picking up this change? >> >> Uhmm, R8A7793 SoC hadn't been supported in upstream Linux kernel yet. >> So, is it still OK to take this patch now ? > > Yes, I think so. Mainline support for the r8a7793 is in the pipeline > and by adding (trivial) driver support before hand the drivers will > be able to be used when the SoC code lands. Huhm, I think the declaration in device driver should come along with the support in SoC DTS file. Without it, I doubt how the developer makes sure the validity/correctness of the patch. (This is what I understood about development for R8A7790 and R8A7791.) > This is consistent with how other are being handled for the r8a7793. If this is our development policy, it's fine. :) >>>> --- >>>> >>>> This patch is against for-next branch of Tejun's libata repo. >>>> >>>> v2 [Yoshihiro Kaneko] >>>> * Update the binding documentation >>>> >>>> Documentation/devicetree/bindings/ata/sata_rcar.txt | 3 ++- >>>> drivers/ata/sata_rcar.c | 5 +++++ >>>> 2 files changed, 7 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt >>>> index 1e61113..74b1130 100644 >>>> --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt >>>> +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt >>>> @@ -4,7 +4,8 @@ Required properties: >>>> - compatible : should contain one of the following: >>>> - "renesas,sata-r8a7779" for R-Car H1 >>>> - "renesas,sata-r8a7790" for R-Car H2 >>>> - - "renesas,sata-r8a7791" for R-Car M2 >>>> + - "renesas,sata-r8a7791" for R-Car M2-W >>>> + - "renesas,sata-r8a7793" for R-Car M2-N >>>> - reg : address and length of the SATA registers; >>>> - interrupts : must consist of one interrupt specifier. >>>> >>>> diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c >>>> index 61eb6d7..6f8ed8e 100644 >>>> --- a/drivers/ata/sata_rcar.c >>>> +++ b/drivers/ata/sata_rcar.c >>>> @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { >>>> .compatible = "renesas,sata-r8a7791", >>>> .data = (void *)RCAR_GEN2_SATA >>>> }, >>>> + { >>>> + .compatible = "renesas,sata-r8a7793", >>>> + .data = (void *)RCAR_GEN2_SATA >>>> + }, >>>> { }, >>>> }; >>>> MODULE_DEVICE_TABLE(of, sata_rcar_match); >>>> @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { >>>> { "sata-r8a7779", RCAR_GEN1_SATA }, >>>> { "sata-r8a7790", RCAR_GEN2_SATA }, >>>> { "sata-r8a7791", RCAR_GEN2_SATA }, >>>> + { "sata-r8a7793", RCAR_GEN2_SATA }, >>>> { }, >>>> }; >>>> MODULE_DEVICE_TABLE(platform, sata_rcar_id_table); >>>> -- >>>> 1.9.1 >>>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-sh" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >> >> -- >> Best regards, >> KHIEM Nguyen >>
Hi Khiem-san, On Thu, Oct 16, 2014 at 03:20:01PM +0900, Khiem Nguyen wrote: > Hi Simon-san, > > Thanks for your explanation. > > On 10/16/2014 2:55 PM, Simon Horman wrote: > > Hi Khiem-san, > > > > On Thu, Oct 16, 2014 at 02:43:05PM +0900, Khiem Nguyen wrote: > >> Hi, > >> > >> On 10/16/2014 2:20 PM, Simon Horman wrote: > >>> On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: > >>>> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > >>>> > >>>> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > >>>> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> > >>> > >>> Acked-by: Simon Horman <horms+renesas@verge.net.au> > >>> > >>> Tejun, could you consider picking up this change? > >> > >> Uhmm, R8A7793 SoC hadn't been supported in upstream Linux kernel yet. > >> So, is it still OK to take this patch now ? > > > > Yes, I think so. Mainline support for the r8a7793 is in the pipeline > > and by adding (trivial) driver support before hand the drivers will > > be able to be used when the SoC code lands. > > Huhm, I think the declaration in device driver should come along with > the support in SoC DTS file. > Without it, I doubt how the developer makes sure the validity/correctness > of the patch. > (This is what I understood about development for R8A7790 and R8A7791.) I understand your concern and it is one that I used to hold myself. However, I believe that this patch has been verified out-of-tree and its simplicity leads me to believe that it is extremely likely to work once SoC support hits mainline. By taking the approach of merging driver changes in this way we are able to merge driver and SoC code independently which makes the whole development process rather less constrained and easier to manage. > > This is consistent with how other are being handled for the r8a7793. > If this is our development policy, it's fine. :) I believe it is. > >>>> --- > >>>> > >>>> This patch is against for-next branch of Tejun's libata repo. > >>>> > >>>> v2 [Yoshihiro Kaneko] > >>>> * Update the binding documentation > >>>> > >>>> Documentation/devicetree/bindings/ata/sata_rcar.txt | 3 ++- > >>>> drivers/ata/sata_rcar.c | 5 +++++ > >>>> 2 files changed, 7 insertions(+), 1 deletion(-) > >>>> > >>>> diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt > >>>> index 1e61113..74b1130 100644 > >>>> --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt > >>>> +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt > >>>> @@ -4,7 +4,8 @@ Required properties: > >>>> - compatible : should contain one of the following: > >>>> - "renesas,sata-r8a7779" for R-Car H1 > >>>> - "renesas,sata-r8a7790" for R-Car H2 > >>>> - - "renesas,sata-r8a7791" for R-Car M2 > >>>> + - "renesas,sata-r8a7791" for R-Car M2-W > >>>> + - "renesas,sata-r8a7793" for R-Car M2-N > >>>> - reg : address and length of the SATA registers; > >>>> - interrupts : must consist of one interrupt specifier. > >>>> > >>>> diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c > >>>> index 61eb6d7..6f8ed8e 100644 > >>>> --- a/drivers/ata/sata_rcar.c > >>>> +++ b/drivers/ata/sata_rcar.c > >>>> @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { > >>>> .compatible = "renesas,sata-r8a7791", > >>>> .data = (void *)RCAR_GEN2_SATA > >>>> }, > >>>> + { > >>>> + .compatible = "renesas,sata-r8a7793", > >>>> + .data = (void *)RCAR_GEN2_SATA > >>>> + }, > >>>> { }, > >>>> }; > >>>> MODULE_DEVICE_TABLE(of, sata_rcar_match); > >>>> @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { > >>>> { "sata-r8a7779", RCAR_GEN1_SATA }, > >>>> { "sata-r8a7790", RCAR_GEN2_SATA }, > >>>> { "sata-r8a7791", RCAR_GEN2_SATA }, > >>>> + { "sata-r8a7793", RCAR_GEN2_SATA }, > >>>> { }, > >>>> }; > >>>> MODULE_DEVICE_TABLE(platform, sata_rcar_id_table); > >>>> -- > >>>> 1.9.1 > >>>> > >>> -- > >>> To unsubscribe from this list: send the line "unsubscribe linux-sh" in > >>> the body of a message to majordomo@vger.kernel.org > >>> More majordomo info at http://vger.kernel.org/majordomo-info.html > >>> > >> > >> -- > >> Best regards, > >> KHIEM Nguyen > >> > > -- > Best regards, > KHIEM Nguyen > -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: > From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > > Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> > Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> Can you please respin the patch on top of libata/for-3.19? git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata.git for-3.19 Thanks.
Hello Tejun, 2014-10-28 0:05 GMT+09:00 Tejun Heo <tj@kernel.org>: > On Wed, Oct 15, 2014 at 05:59:37PM +0900, Yoshihiro Kaneko wrote: >> From: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >> >> Signed-off-by: Koji Matsuoka <koji.matsuoka.xm@renesas.com> >> Signed-off-by: Yoshihiro Kaneko <ykaneko0929@gmail.com> > > Can you please respin the patch on top of libata/for-3.19? Sure, will do. Thanks, Kaneko > > git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata.git for-3.19 > > Thanks. > > -- > tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/ata/sata_rcar.txt b/Documentation/devicetree/bindings/ata/sata_rcar.txt index 1e61113..74b1130 100644 --- a/Documentation/devicetree/bindings/ata/sata_rcar.txt +++ b/Documentation/devicetree/bindings/ata/sata_rcar.txt @@ -4,7 +4,8 @@ Required properties: - compatible : should contain one of the following: - "renesas,sata-r8a7779" for R-Car H1 - "renesas,sata-r8a7790" for R-Car H2 - - "renesas,sata-r8a7791" for R-Car M2 + - "renesas,sata-r8a7791" for R-Car M2-W + - "renesas,sata-r8a7793" for R-Car M2-N - reg : address and length of the SATA registers; - interrupts : must consist of one interrupt specifier. diff --git a/drivers/ata/sata_rcar.c b/drivers/ata/sata_rcar.c index 61eb6d7..6f8ed8e 100644 --- a/drivers/ata/sata_rcar.c +++ b/drivers/ata/sata_rcar.c @@ -841,6 +841,10 @@ static struct of_device_id sata_rcar_match[] = { .compatible = "renesas,sata-r8a7791", .data = (void *)RCAR_GEN2_SATA }, + { + .compatible = "renesas,sata-r8a7793", + .data = (void *)RCAR_GEN2_SATA + }, { }, }; MODULE_DEVICE_TABLE(of, sata_rcar_match); @@ -850,6 +854,7 @@ static const struct platform_device_id sata_rcar_id_table[] = { { "sata-r8a7779", RCAR_GEN1_SATA }, { "sata-r8a7790", RCAR_GEN2_SATA }, { "sata-r8a7791", RCAR_GEN2_SATA }, + { "sata-r8a7793", RCAR_GEN2_SATA }, { }, }; MODULE_DEVICE_TABLE(platform, sata_rcar_id_table);