Re: [PATCH 1/1 re-submit 1] sdio: add low level i/o functions for workarounds

2009-03-25 Thread Pierre Ossman
On Wed, 25 Mar 2009 01:43:48 -0700 (PDT)
Uri Shkolnik uri...@yahoo.com wrote:

 
 Hi Pierre,
 
 The SDIO patches are part of (at least) dozen patches needed to upgrade the 
 Siano's offering for Linux kernel.
 
 The order is -
 1) SDIO SMS interface driver and SDIO stack patch (add)
 2) SPI interface driver (add)
 3) USB interface driver (modify)
 4) IR port (add)
 5) USB v3 (modify)
 6-15(?) ) Core and Cards modifications
 

I'm not sure where the separation comes in here. So far the driver has
had a common entrypoint that calls all the different interface specific
startup routines.

 
 The order of the patches places the SDIO among the first to be submitted for 
 review (interface drivers must be patched before the core, in order to make 
 the various commits pass bisect tests).
 

I take it you're referring to your internal repo? Mainline only
contains the USB interface so splitting that up into a core and
interface driver shouldn't cause any bisect problems.

 I suggest that we'll continue the submission, and I'll cc you on ALL 
 submissions. You will be able to review, and either ask for modification 
 and/or suggest your on supplementary patches at any stage. 
 

I'd prefer to not build a huge monolith just to later take it apart,
but I can live with it.

Rgds
-- 
 -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.


signature.asc
Description: PGP signature


Re: [PATCH 1/1 re-submit 1] sdio: add low level i/o functions for workarounds

2009-03-24 Thread Pierre Ossman
On Sun, 22 Mar 2009 16:48:39 +0200
Uri Shkolnik u...@siano-ms.com wrote:

 Hi Pierre,
 
 The USB separation patches are ready, and will be committed for review
 shortly (SDIO stack workaround + Siano SDIO driver were the first to be
 re-re-re-committed, SPI will be next, and after them the core which
 includes the 'separation' code). You can view one (of many) older commit
 operations @
 http://patchwork.kernel.org/project/linux-media/list/?submitter=Uristat
 e=*
 

I see. Could you hold off on the SDIO patches and allow me to do a
final cleanup once you have the separation patches done? Then I can
send them directly to Mauro and we can have this merged quickly.

Rgds
-- 
 -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.


signature.asc
Description: PGP signature


Re: [PATCH 1/1 re-submit 1] sdio: add low level i/o functions for workarounds

2009-03-22 Thread Pierre Ossman
On Sat, 14 Mar 2009 07:42:01 -0300
Mauro Carvalho Chehab mche...@infradead.org wrote:

 Hi Pierre,
 
 Uri sent me this patchset, as part of the changes for supporting some devices
 from Siano.
 
 The changeset looks fine, although I have no experiences with MMC. Are you
 applying it on your tree, or do you prefer if I apply here?
 
 If you're applying on yours, this is my ack:
 Acked-by: Mauro Carvalho Chehab mche...@redhat.com
 

This should probably go in your tree with the patch for the Siano SDIO
driver. The problem is that that driver isn't ready yet. I was going
to do a final cleanup once the USB separations patches were done, but
those never materialised.

Rgds
-- 
 -- Pierre Ossman

  WARNING: This correspondence is being monitored by the
  Swedish government. Make sure your server uses encryption
  for SMTP traffic and consider using PGP for end-to-end
  encryption.


signature.asc
Description: PGP signature


RE: [PATCH 1/1 re-submit 1] sdio: add low level i/o functions for workarounds

2009-03-22 Thread Uri Shkolnik


-Original Message-
From: Pierre Ossman [mailto:drz...@drzeus.cx] 
Sent: Sunday, March 22, 2009 4:36 PM
To: Mauro Carvalho Chehab
Cc: Uri Shkolnik; Linux Media Mailing List
Subject: Re: [PATCH 1/1 re-submit 1] sdio: add low level i/o functions
for workarounds

On Sat, 14 Mar 2009 07:42:01 -0300
Mauro Carvalho Chehab mche...@infradead.org wrote:

 Hi Pierre,
 
 Uri sent me this patchset, as part of the changes for supporting some
devices
 from Siano.
 
 The changeset looks fine, although I have no experiences with MMC. Are
you
 applying it on your tree, or do you prefer if I apply here?
 
 If you're applying on yours, this is my ack:
 Acked-by: Mauro Carvalho Chehab mche...@redhat.com
 

This should probably go in your tree with the patch for the Siano SDIO
driver. The problem is that that driver isn't ready yet. I was going
to do a final cleanup once the USB separations patches were done, but
those never materialised.

Rgds
-- 
 -- Pierre Ossman

-

Hi Pierre,

The USB separation patches are ready, and will be committed for review
shortly (SDIO stack workaround + Siano SDIO driver were the first to be
re-re-re-committed, SPI will be next, and after them the core which
includes the 'separation' code). You can view one (of many) older commit
operations @
http://patchwork.kernel.org/project/linux-media/list/?submitter=Uristat
e=*

Please note that due the commit requirements I re-patch (re-generate)
the code patches you email me back at mid-2008 against kernel 2.6.29
(your code remain unchanged !).

Thanks,

Uri
--
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