All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Shimoda, Yoshihiro" <yoshihiro.shimoda.uh@renesas.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH v2] serial: sh-sci: modify sci_break_ctl()
Date: Wed, 11 Apr 2012 09:36:40 +0000	[thread overview]
Message-ID: <4F8550A8.30904@renesas.com> (raw)
In-Reply-To: <4F7E3FE2.4050902@renesas.com>

2012/04/11 13:28, Paul Mundt wrote:
> On Fri, Apr 06, 2012 at 02:39:00PM +0900, Shimoda, Yoshihiro wrote:
>>
>> Sorry I forgot that the SH7724 has SCIF and SCIFA.
>> The ecovec uses SCIF, and it doesn't have SCSPTR.
> 
> Well, one thing that you can do is test for the SCSPTR existence and
> simply not care about the port type. This is roughly what the generic
> sci_init_pins() does for example.
> 
> You would have to ensure that the bits you are twiddling also exist for
> the SCIx_SH2_SCIF_FIFODATA_REGTYPE, SCIx_SH2_SCIF_FIFODATA_REGTYPE, and
> SCIx_SH4_SCIF_FIFODATA_REGTYPE, though.

Thank you for the suggestion. I will modify the code.

>> I checked the ecovec schematics, but it cannot use SCIFA because
>> other functions use the multiplex pins.
>>
> You should be able to plug them in for the port and let the sh-sci driver
> try to grab the port. The port will simply be skipped if pin demux fails.
> Take a look at 50f0959ad4f9ac1c5ee208bb820de299a1b3730b for an idea of
> how to wire it up.
> 

I'm sorry, I don't understand this comment.
I think that if the sh-sci driver grab the port, other driver cannot
work correctly when other driver is working.

Best regards,
Yoshihiro Shimoda

      parent reply	other threads:[~2012-04-11  9:36 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-06  0:59 [PATCH v2] serial: sh-sci: modify sci_break_ctl() Shimoda, Yoshihiro
2012-04-06  1:57 ` Simon Horman
2012-04-06  2:13 ` Shimoda, Yoshihiro
2012-04-06  2:44 ` Simon Horman
2012-04-06  3:50 ` Shimoda, Yoshihiro
2012-04-06  5:11 ` Simon Horman
2012-04-06  5:39 ` Shimoda, Yoshihiro
2012-04-06 11:13 ` Simon Horman
2012-04-09  7:59 ` Paul Mundt
2012-04-11  4:28 ` Paul Mundt
2012-04-11  9:36 ` Shimoda, Yoshihiro [this message]

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=4F8550A8.30904@renesas.com \
    --to=yoshihiro.shimoda.uh@renesas.com \
    --cc=linux-sh@vger.kernel.org \
    /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.