{"push_id":1341861264,"size":12,"distinct_size":12,"ref":"refs/heads/master","head":"32dc2a19bb6a91862b414fdd6eec07154e5e4308","before":"5093ea33209a8b72ca43f1b1eac08629f060a634","commits":[{"sha":"a452b83cd9eac584073fa19eb76e133de913ef52","author":{"name":"Cyril Bur","email":"b6cbb1f61c178071c01e56fffbbb40d234be3176@au1.ibm.com"},"message":"libflash/blocklevel: Allow double ecc protecting a region\n\nCurrently the policy for calling ECC protecting a range at the\nblocklevel layer is that the requested region be completely\nunprotected otherwise the call will return an error. It turns out that\nduplicate calls to ffs_init() with true as the last parameter (for the\nsame blocklevel structure) will cause duplicate attempts to\necc_protect() ranges.\n\nChange the policy within blocklevel to allow duplicate protecting.\nIn fact the new policy almost guarantees no failure (baring something\nodd like malloc() failing). It will detect that the range is currently\nalready fully protected and do nothing, detect that part of the range\nis or is not and extend the existing range or detect that a range fits\nperfectly between two ranges in which case it will merge the ranges.\n\nAlso adjust tests to match the new policy.\n\nSigned-off-by: Cyril Bur <cyril.bur@au1.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/a452b83cd9eac584073fa19eb76e133de913ef52"},{"sha":"adebaf3e0e621240c2041ed8f1ce91f70795cfa9","author":{"name":"Cyril Bur","email":"b6cbb1f61c178071c01e56fffbbb40d234be3176@au1.ibm.com"},"message":"flash: Size NVRAM based on ECC for OpenPOWER platforms\n\nIf NVRAM has ECC (as per the ffs header) then the actual size of the\npartition is less than reported by the ffs header in the PNOR then the\nactual size of the partition is less than reported by the ffs header.\n\nSigned-off-by: Cyril Bur <cyril.bur@au1.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/adebaf3e0e621240c2041ed8f1ce91f70795cfa9"},{"sha":"d4a75dc289f268002339cab75a6f4a1d11fcc466","author":{"name":"Alistair Popple","email":"282a5e7894e60a826b2ef1473b1a0536b4a642df@popple.id.au"},"message":"flash: Use blocklevel to do ECC reads\n\nflash_read_corrected() assumes the passed blocklevel device is an\nactual flash device. However the blocklevel flash abstraction supports\nautomatically reading ECC protected data so use that instead.\n\nSigned-off-by: Alistair Popple <alistair@popple.id.au>\n[Cyril added true to last param to ffs_init() in flash_register()]\n[Cyril rebased onto 46c006f core/console: use char literals instead of\nnumeric]\nSigned-off-by: Cyril Bur <cyril.bur@au1.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/d4a75dc289f268002339cab75a6f4a1d11fcc466"},{"sha":"d29c25372037e61a1c8c3dd2f04ff8e31c72db97","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"platforms/astbmc: Fix coding style in slot_table_get_slot_info()\n\nRemove the unnecessary space before @pd argument.\n\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/d29c25372037e61a1c8c3dd2f04ff8e31c72db97"},{"sha":"121814d718272ee7ed743aad9d919c8365aeb6dd","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"platforms/astbmc: ibm, slot-label not depend on ibm, slot-location-code\n\n\"ibm,slot-label\" should not depend on \"ibm,slot-location-code\". The\nlater one can not be populted because of oversized \"ibm,slot-label\"\nor PHB's base location code.\n\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/121814d718272ee7ed743aad9d919c8365aeb6dd"},{"sha":"4295424a6ecfebc683106cf9d2c68007f10620d7","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"core/pci: Print CRS retry times\n\nIt's useful to know the CRS retry times before the PCI device is\ndetected successfully. In PCI hot add case, it usually indicates\ntime consumed for the adapter's firmware to be partially ready\n(responsive PCI config space).\n\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/4295424a6ecfebc683106cf9d2c68007f10620d7"},{"sha":"afadf2e4ddfd548cd6664aa32064d45402c75d08","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"core/pci: Fix initial PCIe slot power state\n\nThe initial PCIe slot power state should be retrieved from the PCIe\nSlot Control register (offset: +0x18), instead of having the fixed\nstate (power-off) wrongly. Otherwise, we possibly have mismatched\nstates by software and hardware. One side-effect is PCIe slot can\nnot be powered off from hardware for the first time.\n\nThis fixes above issue by fetching the initial PCIe slot power state\nfrom hardware (PCIe Slot Control register) if the power control is\nsupported on the slot. Otherwise, it doesn't matter on what we have\nfor the initial PCIe slot power state.\n\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\n[stewart@linux.vnet.ibm.com: use PCI_SLOT_POWER_ON|OFF rather than 1/0]\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/afadf2e4ddfd548cd6664aa32064d45402c75d08"},{"sha":"60ce59ccd0e978b0a2bc05e9048d03378c79a85c","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"hw/phb3: Disable ECRC on Broadcom adapter behind PMC switch\n\nThe ECRC generation and check can't be enabled on Broadcom's NIC\n(14e4:168a) when it seats behind PMC PCIe switch downstream port\n(11f8:8546). Otherwise, the NIC's config space can not be accessed\nand returns 0xFF's on read because of EEH error even after the error\nis cleared. The issue is reported from Firestone.\n\nThis disables ECRC generation and check on Broadcom's NIC when it\nseats behind PMC PCIe switch downstream port. With this applied,\nthe NIC can be detected successfully.\n\nReported-by: Li Meng <shlimeng@cn.ibm.com>\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\n[stewart@linux.vnet.ibm.com: add description of device workaround is for]\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/60ce59ccd0e978b0a2bc05e9048d03378c79a85c"},{"sha":"de7ffb4e49f831fc30aea9e82fcd1f68d61059d6","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"platforms/astbmc: Introduce slot_init_info() helper function\n\nThis moves the logic initializing PCI slot to helper function\nslot_info_info() so that it can be reused by subsequent patch\nsupporting dynamic PCI slot creation. No functional changes\nintroduced.\n\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/de7ffb4e49f831fc30aea9e82fcd1f68d61059d6"},{"sha":"5b43f79e3bebf4f504013c1155c2d23856324d91","author":{"name":"Gavin Shan","email":"bb32b15705e6e40930aad50252d4e3af8ca61d1b@linux.vnet.ibm.com"},"message":"platforms/astbmc: Support dynamic PCI slot\n\nWe might insert a PCIe switch to PHB direct slot and the downstream\nports of the PCIe switch supports PCI hotplug. This creates dynamic\nPCI slots for the downstream ports in the scenario:\n\n  * The dynamic PCI slot's label has fixed encoding: \"S<domain><bus_num>\".\n  * No associated platform slot.\n  * The management on dynamic PCI slot relies on the generic layer\n    implemented in pcie-slot.c.\n\nRequested-by: Li Meng <shlimeng@cn.ibm.com>\nSigned-off-by: Gavin Shan <gwshan@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/5b43f79e3bebf4f504013c1155c2d23856324d91"},{"sha":"edf57684bd6821d4d01515b6e599f477e04ca01d","author":{"name":"Vasant Hegde","email":"513eb5fb05f7c55c5c3a477d83590ec3919f7090@linux.vnet.ibm.com"},"message":"core/pel: Validate 'system-id' DT property before using\n\nPlatforms like mambo doesn't populate 'system-id' DT property. Validate\nthese properties before using.\n\nSigned-off-by: Vasant Hegde <hegdevasant@linux.vnet.ibm.com>\n[stewart@linux.vnet.ibm.com, memset(0) for not found dt props, fix unit test]\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/edf57684bd6821d4d01515b6e599f477e04ca01d"},{"sha":"32dc2a19bb6a91862b414fdd6eec07154e5e4308","author":{"name":"Mukesh Ojha","email":"d4fe1e4b4fe41f7c2e2f51e1cce8a466349ab70f@linux.vnet.ibm.com"},"message":"libc : Changes variable data type where return value of strlen() stored from int to size_t\n\nReason of the change as integer value may overflow, and it can give negative\nvalue for the length.\n\nThis patch also changes the data type of variable which is compared with\nstrlen() as the comparison also has to be done on the same level.\n\nSigned-off-by: Mukesh Ojha <mukesh02@linux.vnet.ibm.com>\nSigned-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>","distinct":true,"url":"https://api.github.com/repos/stewart-ibm/skiboot/commits/32dc2a19bb6a91862b414fdd6eec07154e5e4308"}]}
{"actor":{"display_login":"stewart-ibm"}}